file_scan_directory() scans subversion working base (.svn) by default

What about modifying file_scan_directory so that it doesn't traverse any directories beginning with a period?

If we add .svn, we might as well want to add .darcs. :)

Attached patch adds $omit_hidden as the final parameter to file_scan_directory(), defaulting to TRUE, in which case it ignores all files/directories beginning with a period. Modules that want to store data in hidden directories/files can set the flag to FALSE.

This would also fix hidden files with ._ prefix show on module list.

Great big +1 on hiding .* files.

Is there a compelling reason to add the $omit_hidden flag? I can't think of a viable use case for wanting to see files that are supposed to be hidden?

It seems possible that a module would want to keep security-related information in a hidden file, or to keep temporary files in a hidden directory. But removing the flag is also fine with me if that level of customization isn't needed.

I already tweaked it to do so (skip .directories) for my filebrowser extensions work.
Did so by changing the default exclusion pattern. You can overr-ride it by explicitly giving a blank exclusion pattern

-1 on the extra parameter. Why can probably always hide these.

Sounds good - I only added the parameter because I thought it would get more complaints otherwise. Here is the revised patch.

I just realized that the $omit_hidden patch neglected to pass omit_hidden to recursive calls, so we averted a problem anyway.

Extended the documentation a little and committed the patch to CVS HEAD.

backported