Page MenuHomePhabricator

In Applications, always redirect back to the detail page after edits
ClosedPublic

Authored by epriestley on May 14 2017, 8:42 PM.
Tags
None
Referenced Files
F15525794: D17877.id42997.diff
Mon, Apr 21, 4:56 PM
F15441855: D17877.id43003.diff
Mar 26 2025, 8:25 PM
F15441209: D17877.id.diff
Mar 26 2025, 4:55 PM
F15436420: D17877.id42997.diff
Mar 25 2025, 2:01 PM
F15429997: D17877.diff
Mar 24 2025, 5:11 AM
F15392358: D17877.id43003.diff
Mar 15 2025, 2:45 PM
Unknown Object (File)
Feb 21 2025, 3:22 PM
Unknown Object (File)
Feb 9 2025, 4:02 AM
Subscribers
None

Details

Summary

Ref T12685. When you edit an application's policies but don't make any changes, you currently get stuck on the same page. This isn't how other edit screens work, and I think it's a holdover from eras long ago.

Make it consistent with other applications.

Also, fix a missing pht().

Test Plan
  • Edited an application configuation.
  • Clicked "Save" without making changes.
  • After patch, was redirected back to detail page like in other applications.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.May 14 2017, 8:43 PM
This revision was automatically updated to reflect the committed changes.