Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326361 - [Diagram] Tigerstripe sometimes chooses a conflicting default name for an association
Summary: [Diagram] Tigerstripe sometimes chooses a conflicting default name for an ass...
Status: VERIFIED FIXED
Alias: None
Product: Tigerstripe
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 0.5M0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-27 18:33 EDT by Navid Mehregani CLA
Modified: 2010-11-19 10:04 EST (History)
2 users (show)

See Also:


Attachments
Error log (14.89 KB, text/plain)
2010-09-27 18:34 EDT, Navid Mehregani CLA
no flags Details
Patch with bug fix (2.35 KB, patch)
2010-11-19 05:16 EST, Anton Salnik CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Navid Mehregani CLA 2010-09-27 18:33:57 EDT
Build Identifier: 

The diagrams don't take into account association class names when picking a default name for an association.

Reproducible: Always

Steps to Reproduce:
1. In an empty TS project, create to Entities
2. Create an Association Class between the Entities
3. Name the Association Class 'Association0'
4. Create a regular Association between the entities.  Notice that it picks the name 'Association0' for the newly created association!  This results in unpredictable behaviour.
Comment 1 Navid Mehregani CLA 2010-09-27 18:34:29 EDT
Created attachment 179697 [details]
Error log
Comment 2 Chris Hartley CLA 2010-09-27 23:16:37 EDT
I assume this is probably true for any artifact.

I have confirmed that if I rename an Entity Datatype0 then create a datatype I have a similar issue.

The only safe fix I can think of is for the 'default name method' to check if the FQN exists (not an issue if it exists in another package) and if it does, to increment and keep trying until it finds one not in use.
Comment 3 Anton Salnik CLA 2010-11-19 05:16:58 EST
Created attachment 183458 [details]
Patch with bug fix
Comment 4 Yuri Strot CLA 2010-11-19 06:22:40 EST
I've applied Anton's patch.
Comment 5 Navid Mehregani CLA 2010-11-19 10:04:43 EST
Verified. Thanks!