Page MenuHomePhabricator
Differential D13767

Update Notifications for handleRequest
ClosedPublic
Actions

Authored by chad on Aug 1 2015, 2:18 AM.
Tags
None
Referenced Files
F13193823: D13767.id.diff
Sun, May 12, 6:19 PM
F13186480: D13767.diff
Sat, May 11, 3:39 AM
F13181101: D13767.id33294.diff
Thu, May 9, 8:38 AM
Unknown Object (File)
Tue, May 7, 6:56 AM
Unknown Object (File)
Fri, May 3, 4:36 AM
Unknown Object (File)
Thu, Apr 25, 12:38 AM
Unknown Object (File)
Sun, Apr 14, 11:51 AM
Unknown Object (File)
Apr 10 2024, 10:54 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T8628: Modernize AphrontControllers to implement handleRequest
Commits
Restricted Diffusion Commit
rPf5be68c59c39: Update Notifications for handleRequest
Summary

Updates Notifcations for handleRequest

Test Plan

Test as much as I could, mark all, view, lists.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 33244.Aug 1 2015, 2:18 AM
chad retitled this revision from to Update Notifications for handleRequest.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 1 2015, 2:18 AM
chad added a task: T8628: Modernize AphrontControllers to implement handleRequest.Aug 1 2015, 5:02 PM
epriestley requested changes to this revision.Aug 1 2015, 10:36 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/notification/controller/PhabricatorNotificationClearController.php
10

getStr() and getURIData() are not the same.

getStr() reads values out of the query parameters, like /?chronoKey=123. getURIData() reads the route regexps.

For example, a URI like /task/edit/123/?id=456 might return 123 from getURIData(), but 456 from getInt().

This should stay as getStr().

This revision now requires changes to proceed.Aug 1 2015, 10:36 PM
chad marked an inline comment as done.Aug 2 2015, 12:08 AM

thanks for the info!

chad updated this revision to Diff 33267.Aug 2 2015, 12:09 AM
chad edited edge metadata.
  • Fix getStr
epriestley accepted this revision.Aug 4 2015, 4:32 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 4 2015, 4:32 PM
Closed by commit rPf5be68c59c39: Update Notifications for handleRequest (authored by chad, committed by chad). · Explain WhyAug 4 2015, 4:33 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 33294

View Options

src/applications/notification/controller/PhabricatorNotificationClearController.php

Loading...
View Options

src/applications/notification/controller/PhabricatorNotificationIndividualController.php

Loading...
View Options

src/applications/notification/controller/PhabricatorNotificationListController.php

Loading...
View Options

src/applications/notification/controller/PhabricatorNotificationPanelController.php

Loading...
View Options

src/applications/notification/controller/PhabricatorNotificationStatusController.php

Loading...
View Options

src/applications/notification/controller/PhabricatorNotificationTestController.php

Loading...
Phabricator · Built by Phacility