> On Aug. 21, 2014, 7:19 a.m., David Faure wrote: > > We have never needed this until now because we can recognize such files by > > their name... So while I'm not opposed to this, I wonder, why not use if > > (u.fileName() == ".directory") in KDesktopFileActions (RR 119869) -- which > > will also make it work with existing .directory files that got created > > without a Type=Directory entry?
The simple solution is always the best one. Thanks! :) - Maarten ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119868/#review64952 ----------------------------------------------------------- On Aug. 21, 2014, 1:15 p.m., Maarten De Meyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119868/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2014, 1:15 p.m.) > > > Review request for KDE Frameworks, kdelibs and Frank Reininghaus. > > > Repository: kconfig > > > Description > ------- > > Add a hasDirectoryType method to KDesktopFile. > This is required for review request > [119869](https://git.reviewboard.kde.org/r/119869/) > > > Diffs > ----- > > src/core/kdesktopfile.h 2190051 > src/core/kdesktopfile.cpp 6278309 > > Diff: https://git.reviewboard.kde.org/r/119868/diff/ > > > Testing > ------- > > > Thanks, > > Maarten De Meyer > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel