Page MenuHomePhabricator

Project - add editor-level support for detecting name conflicts
ClosedPublic

Authored by btrahan on Apr 21 2014, 10:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 25, 12:20 PM
Unknown Object (File)
Tue, Jan 21, 9:05 AM
Unknown Object (File)
Fri, Jan 17, 5:52 AM
Unknown Object (File)
Fri, Jan 17, 4:08 AM
Unknown Object (File)
Thu, Jan 16, 1:05 PM
Unknown Object (File)
Thu, Jan 16, 1:39 AM
Unknown Object (File)
Dec 22 2024, 8:25 PM
Unknown Object (File)
Dec 21 2024, 7:44 AM
Subscribers

Details

Summary

Fixes T4833. I wish there was an elegant way to catch this exception but I think the stack is written such that we really should just do this one-off query here...

Test Plan

from the "create project" link under "edit task" I received a more detailed exception than the report in T4833 post patch. I also tested editing an existing project - yay - and editing an existing project to some other existing project's name - got a nice error dialogue.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Project - add editor-level support for detecting name conflicts.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

T4021 is on my mid-term radar and I can try to give us proper hooks for this kind of thing then. This is a good step forward, though.

This revision is now accepted and ready to land.Apr 21 2014, 10:43 PM
btrahan updated this revision to Diff 20964.

Closed by commit rPc00733a292e8 (authored by @btrahan).