Page MenuHomePhabricator

Update Project for handleRequest
ClosedPublic

Authored by chad on Aug 8 2015, 4:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 9:19 AM
Unknown Object (File)
Tue, Dec 17, 3:51 AM
Unknown Object (File)
Fri, Dec 13, 4:45 PM
Unknown Object (File)
Sat, Dec 7, 3:18 PM
Unknown Object (File)
Wed, Nov 27, 6:53 PM
Unknown Object (File)
Nov 21 2024, 7:48 PM
Unknown Object (File)
Nov 21 2024, 2:25 PM
Unknown Object (File)
Nov 20 2024, 2:08 PM
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.