Page MenuHomePhabricator

Phriction - fix a corner case on renaming a project
ClosedPublic

Authored by btrahan on Jan 3 2014, 9:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 29, 10:28 PM
Unknown Object (File)
Sun, Dec 15, 12:12 PM
Unknown Object (File)
Sun, Dec 15, 12:12 PM
Unknown Object (File)
Sun, Dec 15, 7:53 AM
Unknown Object (File)
Sun, Dec 15, 7:53 AM
Unknown Object (File)
Dec 13 2024, 11:00 AM
Unknown Object (File)
Dec 10 2024, 9:22 PM
Unknown Object (File)
Dec 10 2024, 7:22 PM

Details

Summary

if you rename a project in such a way that the old slug and the new slug are the same, there are errors when the phriction document is updated. detect this case and don't bother updating the document since there is no change. Fixes Github issue 474.

Test Plan

made a project "testTest". Viewed the wiki page. Created the wiki page. Renamed the project "TestTest". Before patch, this error'd, post patch it works!

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

You might need to nuke src/.phutil_module_cache for lint, I think you're probably hitting T1486 (or I messed up D7880).