Merge lp:~jeremywootten/pantheon-files/fix-1465155-crash-on-toggle-hiddens-in-list-view into lp:~elementary-apps/pantheon-files/trunk

Proposed by Jeremy Wootten
Status: Merged
Approved by: Cody Garver
Approved revision: 1856
Merged at revision: 1873
Proposed branch: lp:~jeremywootten/pantheon-files/fix-1465155-crash-on-toggle-hiddens-in-list-view
Merge into: lp:~elementary-apps/pantheon-files/trunk
Diff against target: 21 lines (+1/-4)
1 file modified
src/View/AbstractDirectoryView.vala (+1/-4)
To merge this branch: bzr merge lp:~jeremywootten/pantheon-files/fix-1465155-crash-on-toggle-hiddens-in-list-view
Reviewer Review Type Date Requested Status
elementary Apps team Pending
Review via email: mp+261936@code.launchpad.net

Commit message

Remove subdirectories, not reload them, when toggling show-hidden-files in List view (lp:1465155)

Description of the change

Fixes crash caused by attempting to reload subdirectories after model was cleared when toggling show-hidden-files. Also empties loaded subdirectory list before toggling to prevent reference leak.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/View/AbstractDirectoryView.vala'
--- src/View/AbstractDirectoryView.vala 2015-06-06 08:41:29 +0000
+++ src/View/AbstractDirectoryView.vala 2015-06-15 08:45:30 +0000
@@ -1221,16 +1221,13 @@
1221 /** Handle Preference changes */1221 /** Handle Preference changes */
1222 private void on_show_hidden_files_changed (GLib.Object prefs, GLib.ParamSpec pspec) {1222 private void on_show_hidden_files_changed (GLib.Object prefs, GLib.ParamSpec pspec) {
1223 bool show = (prefs as GOF.Preferences).show_hidden_files;1223 bool show = (prefs as GOF.Preferences).show_hidden_files;
1224 cancel ();
1224 if (!show) {1225 if (!show) {
1225 block_model ();1226 block_model ();
1226 model.clear ();1227 model.clear ();
1227 }1228 }
12281229
1229 directory_hidden_changed (slot.directory, show);1230 directory_hidden_changed (slot.directory, show);
1230 if (loaded_subdirectories != null)
1231 loaded_subdirectories.@foreach ((dir) => {
1232 directory_hidden_changed (dir, show);
1233 });
12341231
1235 if (!show)1232 if (!show)
1236 unblock_model ();1233 unblock_model ();

Subscribers

People subscribed via source and target branches

to all changes: