[Bug 61627] New: incorrenct target in choosetargetdialog of automake manager

[Simo]

------- 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=61627     
           Summary: incorrenct target in choosetargetdialog of automake
                    manager
           Product: kdevelop
           Version: unspecified
          Platform: Debian testing
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: kdevelop-devel at kdevelop.org
        ReportedBy: sika at iwn.fi


Version:           kdevelop CVS (24.7.2003) (using KDE KDE 3.1.2)
Installed from:    Debian testing/unstable Packages
Compiler:          gcc 3.2 
OS:          Linux

Changing back to active target after another target was selected in "Choose Target"-dialog causes the file(s) to be added in the wrong target. 
e.g:
1) create targets "t1" and "t2"
2) set active target "t1"
3) create file or a class
4) In "Choose Target"-dialog select "Choose another Target" and set it to "t2"
5) change your mind and add it active target: click "add new file to my active target"
... and the files will be added to "t2".

furthermore, if one has project structure like
subproj1/
subproj1/test/target1
subproj2/
subproj2/test/target2
then one can't insert any file to "target2" because both have same subproject name ("test").

a patch for both problems ( except long names with relativePath() is not desired UI behaviour? ):
71c71
<                               subprojectComboBox->insertItem ( SmallIcon ( "folder" ), spitem->subdir );
---
>                               subprojectComboBox->insertItem ( SmallIcon ( "folder" ), spitem->relativePath() );
142c142
<               if ( spitem->subdir == name )
---
>               if ( spitem->relativePath() == name )
248a249,254
>       if ( activeTargetRadioButton->isChecked() )
>       {
>               m_choosenTarget = m_widget->activeTarget();
>               m_choosenSubproject = m_widget->activeSubproject();
>       }
>

Hope it helps ;)