Movie list menu: place "Import movies" before "Delete movies"

[cinaest] / src / movie-list-window.vala

diff --git a/src/movie-list-window.vala b/src/movie-list-window.vala
index bc415e1..133451a 100644 (file)
--- a/src/movie-list-window.vala
+++ b/src/movie-list-window.vala
@@ -32,8 +32,11 @@ public class MovieListWindow : StackableWindow {
        private bool search_bar_visible;
        private MovieWindow movie_window;
        private Alignment alignment;
+       private int count;
+       private MovieSource source;
 
-       public MovieListWindow (MovieSource source) {
+       public MovieListWindow (MovieSource source_) {
+               source = source_;
                set_title (source.get_description ());
 
                // View menu
@@ -64,8 +67,8 @@ public class MovieListWindow : StackableWindow {
                add_toolbar (search_bar);
 
                // Movie list - connected to menu for sorting
-               movie_list = new MovieListView (this);
-               menu.sortable = movie_list.sorted_store;
+               movie_list = new MovieListView (this, source.get_name () == _("Watched movies"));
+               menu.sortable = movie_list.store;
                store = movie_list.store;
                store.source = source;
 
@@ -91,13 +94,15 @@ public class MovieListWindow : StackableWindow {
                set_edit_toolbar (edit_toolbar);
 
                // Connect signals
-               menu.filter_changed.connect (() => { start_search (); });
+               menu.filter_changed.connect (on_filter_changed);
                edit_toolbar.button_clicked.connect (on_delete_button_clicked);
                edit_toolbar.arrow_clicked.connect (leave_edit_mode); 
                search_field.changed.connect (on_search_field_changed);
                close_button.clicked.connect (on_close_button_clicked);
                key_press_event.connect (on_key_press_event);
                movie_list.movie_activated.connect (on_movie_activated);
+               store.row_deleted.connect (on_row_deleted);
+               store.row_inserted.connect (on_row_inserted);
                store.search_finished.connect (on_search_finished);
 
                store.notify["update-running"].connect (on_update_running_changed);
@@ -127,18 +132,41 @@ public class MovieListWindow : StackableWindow {
                }
        }
 
+       private void on_filter_changed () {
+               var markup = new StringBuilder ();
+               if (filter.year_min != 0 || filter.year_max != 0)
+                       markup.append (_(" - years"));
+               if (filter.rating_min != 0)
+                       markup.append (_(" - rating"));
+               if (filter.genres.field != 0)
+                       markup.append (_(" - genres"));
+               if (markup.str.length > 0) {
+                       Gdk.Color color;
+                       this.ensure_style ();
+                       if (this.style.lookup_color ("ActiveTextColor", out color)) {
+                               markup.prepend ("<span fgcolor=\"%s\">".printf (color.to_string ()));
+                               markup.append ("</span>");
+                       }
+                       markup.prepend (source.get_description ());
+                       set_markup (markup.str);
+               } else {
+                       set_markup (source.get_description ());
+               }
+
+               start_search ();
+       }
+
        public void on_delete_movies_clicked () {
                fullscreen ();
                edit_toolbar.show ();
                movie_list.set_hildon_ui_mode (UIMode.EDIT);
 
-               var selection = movie_list.get_selection ();
-               selection.unselect_all ();
+               movie_list.unselect_all ();
        }
 
        private void on_delete_button_clicked () {
-               var selection = movie_list.get_selection ();
-               int count = selection.count_selected_rows ();
+               var movies = movie_list.get_selected_movies ();
+               int count = (int) movies.length ();
                if (count == 0) {
                        Banner.show_information (this, null, _("No movies selected"));
                        leave_edit_mode ();
@@ -149,25 +177,7 @@ public class MovieListWindow : StackableWindow {
                var res = dialog.run ();
 
                if (res == Gtk.ResponseType.OK) {
-                       weak TreeModel model;
-                       var rows = selection.get_selected_rows (out model);
-
-                       var movies = new List<Movie> ();
-
-                       // get selected movies from the store
-                       foreach (TreePath path in rows) {
-                               TreeIter iter;
-
-                               if (model.get_iter (out iter, path)) {
-                                       Movie movie;
-
-                                       model.get (iter, MovieListStore.Columns.MOVIE, out movie);
-                                       if (movie != null) {
-                                               movies.append (movie);
-                                       }
-                               }
-                       }
-                       // and remove them
+                       // Remove selected movies
                        foreach (Movie movie in movies) {
                                store.remove (movie);
                        }
@@ -242,6 +252,20 @@ public class MovieListWindow : StackableWindow {
                movie_window.show ();
        }
 
+       private void on_row_deleted (TreePath path) {
+               if (--count == 0) {
+                       no_movies.show ();
+                       movie_list.hide ();
+               }
+       }
+
+       private void on_row_inserted (TreePath path, TreeIter iter) {
+               if (count++ == 0) {
+                       no_movies.hide ();
+                       movie_list.show ();
+               }
+       }
+
        private void on_update_running_changed (GLib.Object source, ParamSpec spec) {
                TreeIter iter;
 
@@ -252,5 +276,9 @@ public class MovieListWindow : StackableWindow {
                        no_movies.show ();
                }
        }
+
+       public unowned MovieListView get_movie_list_view () {
+               return movie_list;
+       }
 }