Merge lp:~mhr3/unity-lens-files/folder-fixes into lp:unity-lens-files
Proposed by
Michal Hruby
Status: | Merged |
---|---|
Merged at revision: | 190 |
Proposed branch: | lp:~mhr3/unity-lens-files/folder-fixes |
Merge into: | lp:unity-lens-files |
Diff against target: |
245 lines (+103/-45) 2 files modified
src/daemon.vala (+50/-40) src/folder.vala (+53/-5) |
To merge this branch: | bzr merge lp:~mhr3/unity-lens-files/folder-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mikkel Kamstrup Erlandsen (community) | Approve | ||
Review via email: mp+76525@code.launchpad.net |
This proposal supersedes a proposal from 2011-09-21.
Description of the change
Fixes Folder filter and bookmarks displaying deleted directories.
To post a comment you must log in.
I can confirm that it works :-) However, I'd like to refactor this a bit. The idea so far has been to keep all bookmarks logic contained in the Bookmarks class in folders.vala.
I suggest that the filtering be moved into that class an automatically taken care of when you call bookmarks. prefix_ search( ) or bookmarks.list().
Also - there's no reason to do async IO here since we have no UI to block :-). In fact async IO is to be taken cautiously inside the lens daemons because it warrants the need for reentrancy safety for fast changing queries (the daemons protect somewhat against this at a higher level, but still).
Otherwise good!