[Uml-devel] [Bug 137497] New: Faulty behavior when a class inside of a package is modified

[Egbert Voigt]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137497         
           Summary: Faulty behavior when a class inside of a package is
                    modified
           Product: umbrello
           Version: unspecified
          Platform: Compiled Sources
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: umbrello-devel.kde.org
        ReportedBy: Egbert.Voigt alcatel de


Version:           1.5.51 (using KDE KDE 3.5.1)
Installed from:    Compiled From Sources
Compiler:          gcc 4.1 
OS:                Linux

Class C is part of package P. Then change something (add a comment) in C.
This leads to an additional package with the name "P.". The next change add a package "P.." and so on.

This patch fix the bug, but it could be not the best decision.

umbrello-1.5.51/umbrello/umlobject.cpp
307a308
>       QString packageName = _name.left(_name.length() - 1); // remove trailing separator, i.e. "."
309c310
<         pkgObj = umldoc->findUMLObject(_name);
---
>         pkgObj = umldoc->findUMLObject(packageName);
312,313c313,314
<                 << _name << " for " << m_Name << endl;
<             pkgObj = Import_Utils::createUMLObject(Uml::ot_Package, _name);
---
>                 << packageName << " for " << m_Name << endl;
>             pkgObj = Import_Utils::createUMLObject(Uml::ot_Package, packageName);
318c319
<                     << "existing " << _name << " is not a container" << endl;
---
>                     << "existing " << packageName << " is not a container" << endl;
321c322
<                 pkgObj = Import_Utils::createUMLObject(Uml::ot_Package, _name);
---
>                 pkgObj = Import_Utils::createUMLObject(Uml::ot_Package, packageName);