Page MenuHomePhabricator

Update Project for handleRequest
ClosedPublic

Authored by chad on Aug 8 2015, 4:52 PM.
Tags
None
Referenced Files
F13357332: D13829.id33395.diff
Mon, Jun 24, 11:56 PM
F13333301: D13829.diff
Tue, Jun 18, 3:45 AM
F13292323: D13829.diff
Wed, Jun 5, 3:11 AM
F13263059: D13829.id.diff
May 27 2024, 6:51 AM
F13251660: D13829.id33396.diff
May 25 2024, 12:17 AM
F13191893: D13829.id33395.diff
May 12 2024, 2:00 AM
Unknown Object (File)
Apr 12 2024, 1:13 AM
Unknown Object (File)
Mar 27 2024, 7:56 AM
Subscribers

Details

Summary

Updates Projects to use handleRequest

Test Plan

New Project, Edit Project, Archive, Watch, Create Workboards, Import Workboards, Edit Column, New Column, Change Icon, Add/Remove Members

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Update Project for handleRequest.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/project/controller/PhabricatorProjectBoardImportController.php
8

By convention, consider separating words with underscores in lowercase variable names: $project_id.

src/applications/project/controller/PhabricatorProjectColumnDetailController.php
9

Yeah, this one's a little more consistent with the rest of the codebase I think.

src/applications/project/controller/PhabricatorProjectListController.php
12

Really minor, but consider $query_key for similar reasons.

This revision is now accepted and ready to land.Aug 8 2015, 4:54 PM
chad marked 3 inline comments as done.
chad edited edge metadata.
  • Update per comments
This revision was automatically updated to reflect the committed changes.