Page MenuHomePhabricator
Differential D13767

Update Notifications for handleRequest
ClosedPublic
Actions

Authored by chad on Aug 1 2015, 2:18 AM.
Tags
None
Referenced Files
F13292261: D13767.id33267.diff
Wed, Jun 5, 3:07 AM
F13292260: D13767.id33244.diff
Wed, Jun 5, 3:07 AM
F13292259: D13767.diff
Wed, Jun 5, 3:07 AM
F13286637: D13767.diff
Tue, Jun 4, 7:29 AM
F13264654: D13767.id33244.diff
Mon, May 27, 7:55 PM
F13257268: D13767.id33294.diff
Sat, May 25, 10:14 PM
F13253692: D13767.diff
Sat, May 25, 2:51 AM
F13241831: D13767.id33294.diff
Wed, May 22, 11:53 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