lib/alexandria/ui/import_dialog.rb in alexandria-book-collection-manager-0.7.4 vs lib/alexandria/ui/import_dialog.rb in alexandria-book-collection-manager-0.7.5

@@ -54,11 +54,11 @@
           puts "Added ImportFilter #{filefilter} -- #{filefilter.name}" if $DEBUG
           filters[filefilter] = filter
         end
 
         signal_connect("selection_changed") do
-          import_button.sensitive = filename && File.file?(filename)
+          import_button.sensitive = !!(filename && File.file?(filename))
         end
 
         # before adding the (hidden) progress bar, we must re-set the
         # packing of the button box (currently packed at the end),
         # because the progressbar will be *after* the button box.
@@ -117,16 +117,14 @@
 
           library = nil
           @bad_isbns = nil
           @failed_isbns = nil
           thread = Thread.start do
-            begin
-              library, @bad_isbns, @failed_isbns = filter.invoke(new_library_name,
-                                                                 filename)
-            rescue StandardError => ex
-              trace = ex.backtrace.join("\n> ")
-              log.error { "Import failed: #{ex.message} #{trace}" }
-            end
+            library, @bad_isbns, @failed_isbns = filter.invoke(new_library_name,
+                                                               filename)
+          rescue StandardError => ex
+            trace = ex.backtrace.join("\n> ")
+            log.error { "Import failed: #{ex.message} #{trace}" }
           end
 
           while thread.alive? && !@destroyed
             # puts "Thread #{thread} still alive."
             @running = true