Review Request: Add moveFilesAndFolders to iprojectfilemanager fix ibuildsystemmanager's TargetFile usage

[Olivier Jean de Gaalon]

> On May 5, 2011, 2:19 a.m., Aleix Pol Gonzalez wrote:
> > project/interfaces/iprojectfilemanager.h, line 120
> > <http://git.reviewboard.kde.org/r/101234/diff/1/?file=15438#file15438line120>
> >
> >     You mean to remove maybe?

Yep.


> On May 5, 2011, 2:19 a.m., Aleix Pol Gonzalez wrote:
> > project/projectmodel.h, line 121
> > <http://git.reviewboard.kde.org/r/101234/diff/1/?file=15439#file15439line121>
> >
> >     I don't really see why you need to have them sorted by URL.

It's an easy way of getting rid of subitems when moving or removing, as a child will always appear after its parent. See where I used it.


- Olivier Jean de


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101234/#review3121
-----------------------------------------------------------


On April 26, 2011, 3:48 a.m., Olivier Jean de Gaalon wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101234/
> -----------------------------------------------------------
> 
> (Updated April 26, 2011, 3:48 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> Add moveFilesAndFolders function to iprojectfilemanager
> Add context-menu support for removing files from targets
> Add drag and drop support to model items
> Fix ibuildsystem manager to get rid of the TargetFile QPair
> 
> See also kdevelop patch.
> 
> 
> Diffs
> -----
> 
>   plugins/projectmanagerview/projectmanagerviewplugin.h 5f3e06f 
>   plugins/projectmanagerview/projectmanagerviewplugin.cpp b3cb948 
>   project/abstractfilemanagerplugin.h 3404744 
>   project/abstractfilemanagerplugin.cpp 6d23a15 
>   project/interfaces/ibuildsystemmanager.h eba6984 
>   project/interfaces/iprojectfilemanager.h aad8beb 
>   project/projectmodel.h d1cb3cc 
>   project/projectmodel.cpp 981bd3f 
>   shell/kross/xmltokross/dummybsm.h 5b8b0c6 
> 
> Diff: http://git.reviewboard.kde.org/r/101234/diff
> 
> 
> Testing
> -------
> 
> See kdevelop patch.
> 
> 
> Thanks,
> 
> Olivier Jean de
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110506/0024f950/attachment.html>