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)
Mon, Apr 22, 11:45 AM
Unknown Object (File)
Tue, Apr 16, 1:17 AM
Unknown Object (File)
Sun, Apr 7, 12:37 PM
Unknown Object (File)
Fri, Apr 5, 12:05 PM
Unknown Object (File)
Mon, Apr 1, 4:59 PM
Unknown Object (File)
Fri, Mar 29, 6:14 PM
Unknown Object (File)
Wed, Mar 27, 7:40 PM
Unknown Object (File)
Mar 24 2024, 7:51 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).