Review Request 119607: Support for ".hidden" files

[Bruno Nova]

> On Set. 17, 2014, 7:59 p.m., David Faure wrote:
> > Thanks for the patch. A few comments:
> > 
> > Is there any sort of spec or reference implementation for ".hidden" files? Are they supposed to support wildcards, for instance?
> > 
> > Something else, it would be good to add a unittest to autotests/kdirlistertest.cpp (or autotests/kdirmodeltest.cpp).
> 
> Bruno Nova wrote:
>     OK, I'll try to add a unittest later.
>     
>     I've searched for a general specification for ".hidden" files, but have found none.
>     However, I have accidentally found, through Wikipedia, the commit that adds ".hidden" support to GNOME glib: https://git.gnome.org/browse/glib/commit/?id=510ba9b4efe1813e24c6dfa7405c3547bf9efdd7

I don't think I will be able to create the unittest.
Using Project Neon in Virtualbox, I wasn't able to run `make test` or `ctest` successfully. It always hangs in some of the tests (like the 3rd), even from the latest git commit (patch not applied), and `ctest -VV` doesn't show any error.
So, I'll need help here.

Here are some special cases that I think the unittest should test (among other cases):
* Folder with no *.hidden*
* Folder with empty *.hidden*
* *.hidden* with empty lines
* *.hidden* and filenames that contain leading and/or trailing spaces (the spaces should not be ignored)
* *.hidden* with repeated lines
* *.hidden* and filenames with special and accented characters, including `*` and `?` (if patterns in the *.hidden* files are not implemented, these characters should not be interpreted)
* And possibly more


- Bruno


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119607/#review66777
-----------------------------------------------------------


On Set. 18, 2014, 10:06 a.m., Bruno Nova wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119607/
> -----------------------------------------------------------
> 
> (Updated Set. 18, 2014, 10:06 a.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Bugs: 64740 and 246260
>     https://bugs.kde.org/show_bug.cgi?id=64740
>     https://bugs.kde.org/show_bug.cgi?id=246260
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> This adds support for *.hidden* files to KDE.
> 
> When listing a directory, the files/folders listed in the *.hidden* file will be hidden, unless the user has chosen to show hidden files.
> 
> This patch was initially based on the patch provided by Mark in Bug #246260.
> 
> 
> Diffs
> -----
> 
>   src/core/kcoredirlister.h e6ba2ac 
>   src/core/kcoredirlister.cpp a31d629 
>   src/core/kfileitem.h bebc241 
>   src/core/kfileitem.cpp 74dc069 
> 
> Diff: https://git.reviewboard.kde.org/r/119607/diff/
> 
> 
> Testing
> -------
> 
> Built and tested the patch in Project Neon.
> Dolphin was still using KDE4/Qt4 version of the library, so I only tested using the desktop folder widget, and "folder desktop".
> It worked correctly when I pointed it to "~" and "~/Desktop" (I added ".hidden" there).
> However, it didn't work when I pointed it to the "Desktop folder" (the default option, not the custom location "~/Desktop").
> More testing is required.
> 
> The version for KDE4/Qt4 submitted to Bug #246260 was tested in Kubuntu 14.04, and it worked everywhere I tested (Dolphin, open/save dialogs, folder widget and detailed/tree view in Dolphin).
> It wasn't an intensive test, though.
> 
> 
> Thanks,
> 
> Bruno Nova
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140924/153cee8f/attachment.html>