[Uml-devel] [Bug 158699] New: [PATCH] recursive java import not working in umbrello

[Felix Möller]

------- 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=158699         
           Summary: [PATCH] recursive java import not working in umbrello
           Product: umbrello
           Version: unspecified
          Platform: Compiled Sources
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: umbrello-devel.kde.org
        ReportedBy: felix derklecks de


Version:           2.0.0 (using KDE 4.0.2)
Installed from:    Compiled From Sources
Compiler:          gcc (SUSE Linux) 4.3.0 20080221 (prerelease) [gcc-4_3-branch revision 132521] 
OS:                Linux

I am unable to import a java project into umbrello which has subdirectories.

It ends with doing things like:
umbrello(14079) NativeImportBase::parseFile: "NativeImportBase::parseFile(////////////////////////////////////////////////////////////////////////////////home/fm/eclipse/wwu.fff.or.mac/src/views/LocalGridBagLayout.java): " adding path  "////////////////////////////////////////////////////////////////////////////////home/fm/eclipse/wwu.fff.or.mac/src/views"
umbrello(14079) NativeImportBase::parseFile: "NativeImportBase::parseFile(////////////////////////////////////////////////////////////////////////////////home/fm/eclipse/wwu.fff.or.mac/src/views/LocalGridBagLayout.java): " parsing.

Applying the attached path seems to solve the problem for me:
Index: umbrello/codeimport/javaimport.cpp
===================================================================
--- umbrello/codeimport/javaimport.cpp  (Revision 781148)
+++ umbrello/codeimport/javaimport.cpp  (Arbeitskopie)
 @ -162,7 +162,7  @
         file.pop_back();
     }
     // this is now the root of any further source imports
-    QString sourceRoot = '/' + file.join("/") + '/';
+    QString sourceRoot = file.join("/") + '/';

     for (QStringList::Iterator pathIt = m_imports.begin();
                                    pathIt != m_imports.end(); ++pathIt) {

please review, as I do not understand C++.