[Uml-devel] Load/Save bug in current 1.2b CVS

[Jonathan Riddell]

On Wed, 7 May 2003, P. Fleury wrote:

> Sebastian Stein wrote:
>
> >P. Fleury <fleury at users.sourceforge.net> [030506 19:11]:
> >
> >
> >>2. Add an operation to it (any method will do: list view popup menu,
> >>double-click on class, "new operation" in sequence diagram)
> >>Then, it appears twice in the list view, most of the time once as a
> >>
> >>
> >
> >I have noticed that as well!
> >
> >
> What is the idea behind the two methods of UMLListView named
> slotObjectCreated() and childObjectCreated() ? (they are not documented
> AFAICT:-)
> In the case of adding a new operation, or attribute, both get called,
> and it seems to make two children nodes.
> Also, when the operation is created with the popup menu in the list
> itself, the child gets added to the right parent (the UMLClassifier),
> but if the operation gets added through the popup menu in the diagram,
> then through the new operation dialog, the child gets attached to the
> parent of the Classifier...

childObjectCreated() is used for attributes and operations and templates

slotObjectCreated() is used for everything else (classes, interfaces etc)
but a broken switch statement ment it was also being used for attributes,
operations and templates. Hence the two entries in the list view.

Luis: please remember to document methods when adding them.

Should all be fixed now.

Jonathan Riddell