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
F15481743: D8834.id20964.diff
Tue, Apr 8, 10:37 PM
F15465682: D8834.diff
Wed, Apr 2, 9:54 PM
F15457077: D8834.id.diff
Sun, Mar 30, 2:12 PM
F15450307: D8834.id.diff
Fri, Mar 28, 2:56 PM
F15448615: D8834.id20964.diff
Fri, Mar 28, 5:56 AM
F15446938: D8834.diff
Thu, Mar 27, 8:58 PM
F15428431: D8834.diff
Sun, Mar 23, 8:20 PM
F15419219: D8834.id20964.diff
Fri, Mar 21, 4:32 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).