[Apps] All file-types from Desktop get listed #137

Closed
TheGoblinsBranch opened this Issue Nov 25, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@TheGoblinsBranch

TheGoblinsBranch commented Nov 25, 2016

Version number

Keypirinha 2.9.9 for x64

Package

Apps

Issue description

With the default configuration (scan_start_menu and scan_env_path enabled) all file-types located at the Desktop get listed and not only executables (or links to executables).
I think that this behaviour is not consistent with the one for directories listed in PATHor extra_paths.
(Personally, I didn't expect Desktop to get scanned at all ^_^)

Steps for reproducing

  1. Create a .txt-file at the Desktop
  2. Use the default-configuration for Apps
  3. Search directly for the in step 1 created file

Expected behaviour: File should not be listed.
Actual behaviour: File gets listed.

Suggestion for fix/change

I think the inital problem is, that the items located at the start menu are not executable files, but windows-shortcuts. Nevertheless, we know that all these will link to executables so we can ignore scan_env_path in _catalog_startmenu.
But Desktop can contain different file-types. Therefore in this special case we should look for executables and for shortcuts, but only which one are linking to executables!
(Another way could be just not to look inside Desktop)

@TheGoblinsBranch TheGoblinsBranch changed the title from [Apps] Listing all file-types from Desktop despite "scan_env_path=yes" to [Apps] All file-types from Desktop get listed despite "scan_env_path=yes" Nov 25, 2016

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Nov 25, 2016

Member

The most obvious change that would blend best to the existing features would be to add a scan_desktop setting (boolean; enabled by default). That would not perfectly fit your wishes since the underlying request behind your comment is to have some kind of file filtering feature. At least, that's what your request implies technically. But as things are currently, the Apps package doesn't allow such thing and the KISS principle prevails here.

An ideal solution is to roll up an extra package that would allow real file filtering (i.e. wildcards, perhaps regex; with include/exclude rules). In fact this has been (partly) requested already (see #24). I have the features listing and design ready since quite a while but didn't have time to start its development. Bonus is we could leave the Apps package dealing with scanning of installed applications only.

Member

polyvertex commented Nov 25, 2016

The most obvious change that would blend best to the existing features would be to add a scan_desktop setting (boolean; enabled by default). That would not perfectly fit your wishes since the underlying request behind your comment is to have some kind of file filtering feature. At least, that's what your request implies technically. But as things are currently, the Apps package doesn't allow such thing and the KISS principle prevails here.

An ideal solution is to roll up an extra package that would allow real file filtering (i.e. wildcards, perhaps regex; with include/exclude rules). In fact this has been (partly) requested already (see #24). I have the features listing and design ready since quite a while but didn't have time to start its development. Bonus is we could leave the Apps package dealing with scanning of installed applications only.

@TheGoblinsBranch

This comment has been minimized.

Show comment
Hide comment
@TheGoblinsBranch

TheGoblinsBranch Nov 28, 2016

Oh, I see you are aware of this behaviour and have already plans regarding this! :) So I'll close this issue.

Oh, I see you are aware of this behaviour and have already plans regarding this! :) So I'll close this issue.

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Nov 29, 2016

Member

I'm reopening this until the scan_desktop setting is implemented.

Member

polyvertex commented Nov 29, 2016

I'm reopening this until the scan_desktop setting is implemented.

@polyvertex polyvertex reopened this Nov 29, 2016

@polyvertex polyvertex changed the title from [Apps] All file-types from Desktop get listed despite "scan_env_path=yes" to [Apps] All file-types from Desktop get listed Nov 29, 2016

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Jan 14, 2017

Member

scan_desktop setting available in v2.9.10

Member

polyvertex commented Jan 14, 2017

scan_desktop setting available in v2.9.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment