alexandria-book-collection-manager 0.7.5 → 0.7.6

data/lib/alexandria/ui/smart_library_rule_box.rb

@@ -0,0 +1,119 @@
+# frozen_string_literal: true
+
+# This file is part of Alexandria.
+#
+# See the file README.md for authorship and licensing information.
+
+module Alexandria
+  module UI
+    class SmartLibraryRuleBox
+      attr_accessor :rule_box,
+                    :left_operand_combo, :operator_combo,
+                    :value_entry, :date_entry, :entry_label,
+                    :add_button, :remove_button
+
+      def initialize(parent)
+        @parent = parent
+
+        self.rule_box = Gtk::Box.new :horizontal
+        rule_box.spacing = 8
+
+        self.left_operand_combo = Gtk::ComboBoxText.new
+        self.operator_combo = Gtk::ComboBoxText.new
+
+        self.value_entry = Gtk::Entry.new
+
+        self.date_entry = Gtk::Entry.new.tap do |entry|
+          entry.primary_icon_name = Gtk::Stock::EDIT
+
+          entry.primary_icon_activatable = true
+          entry.signal_connect("icon-press") do |widget, primary, _icon|
+            @parent.handle_date_icon_press(widget, primary, icon)
+          end
+        end
+
+        self.entry_label = Gtk::Label.new("")
+
+        self.add_button = Gtk::Button.new(label: "").tap do |widget|
+          widget.remove(widget.children.first)
+          widget << Gtk::Image.new(stock: Gtk::Stock::ADD,
+                                   size: Gtk::IconSize::BUTTON)
+
+          widget.signal_connect("clicked") { @parent.handle_add_rule_clicked }
+        end
+
+        self.remove_button = Gtk::Button.new(label: "")
+        remove_button.remove(remove_button.children.first)
+        remove_button << Gtk::Image.new(stock: Gtk::Stock::REMOVE,
+                                        size: Gtk::IconSize::BUTTON)
+
+        remove_button.signal_connect("clicked") do |_button|
+          @parent.handle_remove_rule_clicked(self)
+        end
+
+        operands.each do |operand|
+          left_operand_combo.append_text(operand.name)
+        end
+
+        operator_combo.signal_connect("changed") do
+          handle_operator_changed
+        end
+
+        left_operand_combo.signal_connect("changed") do
+          handle_left_operand_changed
+        end
+
+        rule_box.pack_start(left_operand_combo, expand: false, fill: false)
+        rule_box.pack_start(operator_combo, expand: false, fill: false)
+        rule_box.pack_start(value_entry)
+        rule_box.pack_start(date_entry)
+        rule_box.pack_start(entry_label, expand: false, fill: false)
+        rule_box.pack_end(remove_button, expand: false, fill: false)
+        rule_box.pack_end(add_button, expand: false, fill: false)
+
+        value_entry.visible = date_entry.visible = entry_label.visible = false
+      end
+
+      def operands
+        SmartLibrary::Rule::Operands::LEFT
+      end
+
+      def handle_operator_changed
+        operand = operands[left_operand_combo.active]
+        operations = SmartLibrary::Rule.operations_for_operand(operand)
+        operation = operations[operator_combo.active]
+
+        value_entry.visible = date_entry.visible = entry_label.visible = false
+        right_operand = operation.last
+        unless right_operand.nil?
+          entry = case right_operand.klass.name
+                  when "Time"
+                    date_entry
+                  else
+                    value_entry
+                  end
+          entry.visible = true
+          unless right_operand.name.nil?
+            entry_label.text = right_operand.name
+            entry_label.visible = true
+          end
+        end
+
+        @parent.apply_smart_rule_for_rule_box(rule_box, operand, operation)
+      end
+
+      def handle_left_operand_changed
+        operand = operands[left_operand_combo.active]
+        operator_combo.freeze_notify do
+          operator_combo.remove_all
+          operations = SmartLibrary::Rule.operations_for_operand(operand)
+          operations.each do |operation|
+            operator = operation.first
+            operator_combo.append_text(operator.name)
+          end
+          operator_combo.active = 0
+        end
+      end
+    end
+  end
+end

data/lib/alexandria/ui/sound.rb

@@ -39,9 +39,9 @@ module Alexandria
       def play(effect)
         file = File.join(@sounds_dir, "#{effect}.ogg")
         if @playing
-          puts "Already playing #{effect}." if $DEBUG
+          log.debug { "Already playing #{effect}." }
         else
-          puts "Not playing. Starting #{effect}." if $DEBUG
+          log.debug { "Not playing. Starting #{effect}." }
           @filesrc.location = file
           start_playback
         end
@@ -76,10 +76,8 @@ module Alexandria
           when Gst::MessageType::EOS
             stop_playback
           when Gst::MessageType::ERROR
-            if $DEBUG
-              puts "ERROR loop.quit"
-              p message.parse
-            end
+            log.debug { "ERROR loop.quit" }
+            log.debug { message.parse.inspect }
             stop_playback
           end
           true

data/lib/alexandria/ui/ui_manager.rb

@@ -15,6 +15,7 @@ module Alexandria
       attr_accessor :main_app, :actiongroup, :appbar, :prefs, :listview, :iconview,
                     :listview_model, :iconview_model, :filtered_model
       attr_reader :model
+
       include Logging
       include GetText
       GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: "UTF-8")
@@ -74,7 +75,7 @@ module Alexandria
         @iconview_model = Gtk::TreeModelSort.new(@filtered_model)
         @listview_manager = ListViewManager.new @listview, self
         @iconview_manager = IconViewManager.new @iconview, self
-        @sidepane_manager = SidePaneManager.new @library_listview, self
+        @sidepane_manager = SidepaneManager.new @library_listview, self
         @library_listview = @sidepane_manager.library_listview
         @listview_manager.setup_listview_columns_visibility
         @listview_manager.setup_listview_columns_width
@@ -183,15 +184,16 @@ module Alexandria
       def setup_book_providers
         log.debug { "setup_book_providers" }
         mid = @uimanager.new_merge_id
-        BookProviders.each do |provider|
+        ui_paths = ["ui/MainMenubar/ViewMenu/OnlineInformation/",
+                    "ui/BookPopup/OnlineInformation/",
+                    "ui/NoBookPopup/OnlineInformation/"]
+        BookProviders.list.each do |provider|
           name = provider.action_name
-          ["ui/MainMenubar/ViewMenu/OnlineInformation/",
-           "ui/BookPopup/OnlineInformation/",
-           "ui/NoBookPopup/OnlineInformation/"].each do |path|
-             log.debug { "Adding #{name} to #{path}" }
-             @uimanager.add_ui(mid, path, name, name,
-                               :menuitem, false)
-           end
+          ui_paths.each do |path|
+            log.debug { "Adding #{name} to #{path}" }
+            @uimanager.add_ui(mid, path, name, name,
+                              :menuitem, false)
+          end
         end
       end
 
@@ -371,25 +373,25 @@ module Alexandria
 
       def on_books_button_press_event(widget, event)
         log.debug { "books_button_press_event" }
-        if event_is_right_click event
-          widget.grab_focus
+        event_is_right_click event or return
 
-          if (path = widget.get_path_at_pos(event.x.to_i, event.y.to_i))
-            obj, path =
-              widget.is_a?(Gtk::TreeView) ? [widget.selection, path.first] : [widget, path]
+        widget.grab_focus
 
-            unless obj.path_is_selected?(path)
-              log.debug { "Select #{path}" }
-              widget.unselect_all
-              obj.select_path(path)
-            end
-          else
+        if (path = widget.get_path_at_pos(event.x.to_i, event.y.to_i))
+          obj, path =
+            widget.is_a?(Gtk::TreeView) ? [widget.selection, path.first] : [widget, path]
+
+          unless obj.path_is_selected?(path)
+            log.debug { "Select #{path}" }
             widget.unselect_all
+            obj.select_path(path)
           end
-
-          menu = selected_books.empty? ? @nobook_popup : @book_popup
-          menu.popup(nil, nil, event.button, event.time)
+        else
+          widget.unselect_all
         end
+
+        menu = selected_books.empty? ? @nobook_popup : @book_popup
+        menu.popup(nil, nil, event.button, event.time)
       end
 
       def get_library_selection_text(library)
@@ -475,7 +477,7 @@ module Alexandria
             b = books.first
             # FIXME: Clean up endless negation in this logic
             no_urls = true
-            BookProviders.each do |provider|
+            BookProviders.list.each do |provider|
               has_no_url = true
               begin
                 has_no_url = (b.isbn.nil? || b.isbn.strip.empty? || provider.url(b).nil?)
@@ -522,43 +524,25 @@ module Alexandria
         @actiongroup["Sidepane"].active = false
       end
 
+      # TODO: Figure out why this frequently selects the wrong book!
       def select_a_book(book)
-        select_this_book = proc do |bk, view|
-          @filtered_model.refilter
-          iter = iter_from_book bk
-          next unless iter
-
-          path = iter.path
-          next unless view.model
-
-          path = view_path_to_model_path(view, path)
-          log.debug { "Path for #{bk.ident} is #{path}" }
-          selection = view.respond_to?(:selection) ? @listview.selection : @iconview
-          selection.unselect_all
-          selection.select_path(path)
-        end
-        begin
-          log.debug { "select_a_book: listview" }
-          select_this_book.call(book, @listview)
-          log.debug { "select_a_book: listview" }
-          select_this_book.call(book, @iconview)
-        rescue StandardError => ex
-          trace = ex.backtrace.join("\n> ")
-          log.warn { "Failed to automatically select book: #{ex.message} #{trace}" }
-        end
-        # TODO: Figure out why this frequently selects the wrong book!
+        log.debug { "select_a_book: listview" }
+        select_book_in_view(book, @listview)
+        log.debug { "select_a_book: iconview" }
+        select_book_in_view(book, @iconview)
       end
 
       def update(*ary)
         log.debug { "on_update #{ary}" }
         caller = ary.first
-        if caller.is_a?(UndoManager)
+        case caller
+        when UndoManager
           @actiongroup["Undo"].sensitive = caller.can_undo?
           @actiongroup["Redo"].sensitive = caller.can_redo?
-        elsif caller.is_a?(Library)
+        when Library
           handle_update_caller_library ary unless caller.updating?
         else
-          raise "unrecognized update event"
+          raise _("unrecognized update event")
         end
       end
 
@@ -584,8 +568,6 @@ module Alexandria
         end
       end
 
-      # private
-
       def open_web_browser(url)
         if url.nil?
           log.warn("Attempt to open browser with nil url")
@@ -605,7 +587,7 @@ module Alexandria
       end
 
       def load_libraries
-        log.info { "Loading libraries..." }
+        log.info { _("Loading libraries...") }
         @completion_models = CompletionModels.instance
         if @libraries
           detach_old_libraries
@@ -750,7 +732,7 @@ module Alexandria
           new_height = [ICON_HEIGHT, icon.height].min
           icon = cache_scaled_icon(icon, new_width, new_height)
         end
-        icon = icon.tag(Icons::FAVORITE_TAG) if rating == Book::MAX_RATING_STARS
+        icon = Icons.tag_icon(icon, Icons::FAVORITE_TAG) if rating == Book::MAX_RATING_STARS
         iter[Columns::COVER_ICON] = icon
         log.debug { "Full iter: " + (0..15).map { |num| iter[num].inspect }.join(", ") }
       end
@@ -891,7 +873,7 @@ module Alexandria
       def iter_from_ident(ident)
         log.debug { ident.to_s }
         iter = @model.iter_first
-        ok = true
+        ok = true if iter
         while ok
           return iter if iter[Columns::IDENT] == ident
 
@@ -971,7 +953,7 @@ module Alexandria
       end
 
       def restore_preferences
-        log.debug { "Restoring preferences..." }
+        log.debug { "Restoring preferences" }
         if @prefs.maximized
           @main_app.maximize
         else
@@ -1045,7 +1027,7 @@ module Alexandria
 
       def setup_move_actions
         @actiongroup.actions.each do |action|
-          next unless /^MoveIn/.match?(action.name)
+          next unless action.name.start_with?("MoveIn")
 
           @actiongroup.remove_action(action)
         end
@@ -1060,10 +1042,11 @@ module Alexandria
         @actiongroup.add_actions(actions)
         @uimanager.remove_ui(@move_mid) if @move_mid
         @move_mid = @uimanager.new_merge_id
+        ui_paths = ["ui/MainMenubar/EditMenu/Move/",
+                    "ui/BookPopup/Move/"]
         @libraries.all_regular_libraries.each do |library|
           name = library.action_name
-          ["ui/MainMenubar/EditMenu/Move/",
-           "ui/BookPopup/Move/"].each do |path|
+          ui_paths.each do |path|
             @uimanager.add_ui(@move_mid, path, name, name,
                               :menuitem, false)
           end
@@ -1147,10 +1130,10 @@ module Alexandria
       end
 
       def remove_library_separator
-        if !@library_separator_iter.nil? && @libraries.all_smart_libraries.empty?
-          @library_listview.model.remove(@library_separator_iter)
-          @library_separator_iter = nil
-        end
+        return if @library_separator_iter.nil? || @libraries.all_smart_libraries.any?
+
+        @library_listview.model.remove(@library_separator_iter)
+        @library_separator_iter = nil
       end
 
       def undoable_undelete(library, books = nil)
@@ -1190,6 +1173,21 @@ module Alexandria
 
       private
 
+      def select_book_in_view(book, view)
+        @filtered_model.refilter
+        iter = iter_from_book book
+        return unless iter
+
+        path = iter.path
+        return unless view.model
+
+        path = view_path_to_model_path(view, path)
+        log.debug { "Path for #{book.ident} is #{path}" }
+        selection = view.respond_to?(:selection) ? view.selection : view
+        selection.unselect_all
+        selection.select_path(path)
+      end
+
       def view_path_to_model_path(view, path)
         path = view.model.convert_path_to_child_path(path)
         @filtered_model.convert_path_to_child_path(path) if path

data/lib/alexandria/version.rb

@@ -5,7 +5,7 @@
 # See the file README.md for authorship and licensing information.
 
 module Alexandria
-  VERSION = "0.7.5"
+  VERSION = "0.7.6"
   DATA_VERSION = "0.6.3"
-  DISPLAY_VERSION = "0.7.5"
+  DISPLAY_VERSION = "0.7.6"
 end

data/lib/alexandria/web_themes.rb

@@ -22,6 +22,17 @@ module Alexandria
   class WebTheme
     attr_reader :name, :css_file, :preview_file, :pixmaps_directory
 
+    def initialize(css_file, preview_file, pixmaps_directory)
+      @name = File.basename(css_file, ".css").capitalize
+      @css_file = css_file
+      @preview_file = preview_file
+      @pixmaps_directory = pixmaps_directory
+    end
+
+    def has_pixmaps?
+      File.exist?(@pixmaps_directory)
+    end
+
     def self.all
       themes_dir = [
         # System dir
@@ -33,18 +44,12 @@ module Alexandria
       themes_dir.map { |x| load(x) }.flatten
     end
 
-    def has_pixmaps?
-      File.exist?(@pixmaps_directory)
-    end
-
-    private
-
     def self.load(themes_dir)
       themes = []
       if File.exist?(themes_dir)
         Dir.entries(themes_dir).each do |file|
           # ignore hidden files
-          next if /^\./.match?(file)
+          next if file.start_with?(".")
 
           # ignore non-directories
           path = File.join(themes_dir, file)
@@ -55,7 +60,9 @@ module Alexandria
           css_file = File.join(path, file + ".css")
           preview_file = File.join(path, "preview.jpg")
           [css_file, preview_file].each do |helper_file|
-            raise "#{helper_file} not found" unless File.exist?(helper_file)
+            unless File.exist?(helper_file)
+              raise format(_("File %<file>s not found"), file: helper_file)
+            end
           end
           themes << WebTheme.new(css_file, preview_file,
                                  File.join(path, file, "pixmaps"))
@@ -65,12 +72,5 @@ module Alexandria
       end
       themes
     end
-
-    def initialize(css_file, preview_file, pixmaps_directory)
-      @name = File.basename(css_file, ".css").capitalize
-      @css_file = css_file
-      @preview_file = preview_file
-      @pixmaps_directory = pixmaps_directory
-    end
   end
 end