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
F19519730: D8834.id20962.diff
Fri, Jan 16, 7:53 AM
F19517124: D8834.id20962.diff
Thu, Jan 15, 7:56 AM
F19515075: D8834.id20962.diff
Wed, Jan 14, 7:46 AM
F18841316: D8834.id20962.diff
Oct 28 2025, 8:05 AM
F18841315: D8834.id20964.diff
Oct 28 2025, 8:05 AM
F18823667: D8834.id.diff
Oct 23 2025, 12:51 PM
F18790971: D8834.id20962.diff
Oct 15 2025, 9:55 PM
F18771034: D8834.diff
Oct 8 2025, 5:01 PM
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).