alexandria-book-collection-manager 0.7.3 → 0.7.4

data/lib/alexandria/book_providers/z3950.rb

@@ -1,55 +1,40 @@
 # frozen_string_literal: true
 
-# Copyright (C) 2005-2006 Laurent Sansonetti
-# Copyright (C) 2007 Laurent Sansonetti and Marco Costantini
-# Copyright (C) 2014, 2016 Matijs van Zuijlen
+# This file is part of Alexandria.
 #
-# Alexandria is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of the
-# License, or (at your option) any later version.
-#
-# Alexandria is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-# General Public License for more details.
-#
-# You should have received a copy of the GNU General Public
-# License along with Alexandria; see the file COPYING.  If not,
-# write to the Free Software Foundation, Inc., 51 Franklin Street,
-# Fifth Floor, Boston, MA 02110-1301 USA.
+# See the file README.md for authorship and licensing information.
 
 ## $Z3950_DEBUG = $DEBUG
 
-require 'zoom'
-require 'alexandria/book_providers/pseudomarc'
-require 'marc'
+require "zoom"
+require "alexandria/book_providers/pseudomarc"
+require "marc"
 
 module Alexandria
   class BookProviders
     class Z3950Provider < AbstractProvider
       include Logging
       include GetText
-      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: 'UTF-8')
+      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: "UTF-8")
 
-      def initialize(name = 'Z3950', fullname = 'Z39.50')
+      def initialize(name = "Z3950", fullname = "Z39.50")
         super
-        prefs.add('hostname', _('Hostname'), '')
-        prefs.add('port', _('Port'), 7090)
-        prefs.add('database', _('Database'), '')
-        prefs.add('record_syntax', _('Record syntax'), 'USMARC', ['USMARC', 'UNIMARC', 'SUTRS'])
-        prefs.add('username', _('Username'), '', nil, false)
-        prefs.add('password', _('Password'), '', nil, false)
-        prefs.add('charset', _('Charset encoding'), 'ISO-8859-1')
+        prefs.add("hostname", _("Hostname"), "")
+        prefs.add("port", _("Port"), 7090)
+        prefs.add("database", _("Database"), "")
+        prefs.add("record_syntax", _("Record syntax"), "USMARC", ["USMARC", "UNIMARC", "SUTRS"])
+        prefs.add("username", _("Username"), "", nil, false)
+        prefs.add("password", _("Password"), "", nil, false)
+        prefs.add("charset", _("Charset encoding"), "ISO-8859-1")
 
         # HACK : piggybacking support
-        prefs.add('piggyback', 'Piggyback', true, [true, false])
+        prefs.add("piggyback", "Piggyback", true, [true, false])
         prefs.read
       end
 
       def search(criterion, type)
         prefs.read
-        criterion = criterion.encode(prefs['charset'])
+        criterion = criterion.encode(prefs["charset"])
 
         # We only decode MARC at the moment.
         # SUTRS needs to be decoded separately, because each Z39.50 server has a
@@ -81,15 +66,15 @@ module Alexandria
           log.error { ex.backtrace.join("> \n") }
           begin
             marc = MARC::Record.new(marc_txt)
-          rescue StandardError => ex2
-            log.error { ex2.message }
-            log.error { ex2.backtrace.join("> \n") }
-            raise ex2
+          rescue StandardError => ex
+            log.error { ex.message }
+            log.error { ex.backtrace.join("> \n") }
+            raise ex
           end
         end
 
-        log.debug {
-          msg = 'Parsing MARC'
+        log.debug do
+          msg = "Parsing MARC"
           msg += "\n title: #{marc.title}"
           msg += "\n authors: #{marc.authors.join(', ')}"
           msg += "\n isbn: #{marc.isbn}, #{isbn}"
@@ -97,7 +82,7 @@ module Alexandria
           msg += "\n publish year: #{marc.publish_year}" if marc.respond_to?(:publish_year)
           msg += "\n edition: #{marc.edition}"
           msg
-        }
+        end
 
         return if marc.title.nil? # or marc.authors.empty?
 
@@ -106,26 +91,26 @@ module Alexandria
 
         book = Book.new(marc.title, marc.authors,
                         isbn,
-                        (marc.publisher || ''),
+                        (marc.publisher || ""),
                         marc.respond_to?(:publish_year) ? marc.publish_year.to_i : nil,
-                        (marc.edition || ''))
+                        (marc.edition || ""))
         book
       end
 
       def books_from_marc(resultset, _isbn)
         results = []
         resultset[0..9].each do |record|
-          marc_txt = record.render(prefs['charset'], 'UTF-8') # (prefs['record_syntax'], 'USMARC')
+          marc_txt = record.render(prefs["charset"], "UTF-8") # (prefs['record_syntax'], 'USMARC')
           log.debug { marc_txt }
           if $DEBUG
-            File.open(',marc.txt', 'wb') do |f| # DEBUG
+            File.open(",marc.txt", "wb") do |f| # DEBUG
               f.write(marc_txt)
             end
           end
           book = nil
           begin
             mappings = Alexandria::PseudoMarcParser::USMARC_MAPPINGS
-            mappings = Alexandria::PseudoMarcParser::BNF_FR_MAPPINGS if prefs['hostname'] == 'z3950.bnf.fr'
+            mappings = Alexandria::PseudoMarcParser::BNF_FR_MAPPINGS if prefs["hostname"] == "z3950.bnf.fr"
             # try pseudo-marc parser first (it seems to have more luck)
             book = Alexandria::PseudoMarcParser.marc_text_to_book(marc_txt,
                                                                   mappings)
@@ -144,26 +129,26 @@ module Alexandria
       end
 
       def marc?
-        /MARC$/.match(prefs['record_syntax'])
+        /MARC$/.match(prefs["record_syntax"])
       end
 
       def search_records(criterion, type, conn_count)
         options = {}
-        unless prefs['username'].empty? || prefs['password'].empty?
-          options['user'] = prefs['username']
-          options['password'] = prefs['password']
+        unless prefs["username"].empty? || prefs["password"].empty?
+          options["user"] = prefs["username"]
+          options["password"] = prefs["password"]
         end
-        hostname = prefs['hostname']
-        port = prefs['port'].to_i
+        hostname = prefs["hostname"]
+        port = prefs["port"].to_i
         log.debug { "hostname #{hostname} port #{port} options #{options}" }
         conn = ZOOM::Connection.new(options).connect(hostname, port)
-        conn.database_name = prefs['database']
+        conn.database_name = prefs["database"]
 
         # HACK: turn off piggybacking, just to see CMcG
         # #conn.piggyback = false
 
-        conn.preferred_record_syntax = prefs['record_syntax']
-        conn.element_set_name = 'F'
+        conn.preferred_record_syntax = prefs["record_syntax"]
+        conn.element_set_name = "F"
         conn.count = conn_count
         attr = case type
                when SEARCH_BY_ISBN     then [7]
@@ -171,25 +156,25 @@ module Alexandria
                when SEARCH_BY_AUTHORS  then [1, 1003]
                when SEARCH_BY_KEYWORD  then [1016]
                end
-        pqf = ''
+        pqf = ""
         attr.each { |att| pqf += "@attr 1=#{att} " }
         pqf += '"' + criterion.upcase + '"'
         log.debug { "pqf is #{pqf}, syntax #{prefs['record_syntax']}" }
 
         begin
-          if prefs.variable_named('piggyback')
-            unless prefs['piggyback']
-              log.debug { 'setting conn.piggyback to false' }
+          if prefs.variable_named("piggyback")
+            unless prefs["piggyback"]
+              log.debug { "setting conn.piggyback to false" }
               conn.piggyback = false
             end
           end
           conn.search(pqf)
         rescue StandardError => ex
-          if /1005/ =~ ex.message
-            if prefs.variable_named('piggyback') && prefs['piggyback']
+          if /1005/.match?(ex.message)
+            if prefs.variable_named("piggyback") && prefs["piggyback"]
               log.error { "Z39.50 search failed:: #{ex.message}" }
-              log.info { 'Turning off piggybacking for this provider' }
-              prefs.variable_named('piggyback').new_value = false
+              log.info { "Turning off piggybacking for this provider" }
+              prefs.variable_named("piggyback").new_value = false
               search_records(criterion, type, conn_count)
               # hopefully these precautions will prevent infinite loops here
             else
@@ -207,20 +192,20 @@ module Alexandria
       unabstract
 
       include GetText
-      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: 'UTF-8')
+      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: "UTF-8")
 
       def initialize
-        super('LOC', _('Library of Congress (Usa)'))
-        prefs.variable_named('hostname').default_value = 'z3950.loc.gov'
-        prefs.variable_named('port').default_value = 7090
-        prefs.variable_named('database').default_value = 'Voyager'
-        prefs.variable_named('record_syntax').default_value = 'USMARC'
-        prefs.variable_named('charset').default_value = 'ISO-8859-1'
+        super("LOC", _("Library of Congress (Usa)"))
+        prefs.variable_named("hostname").default_value = "z3950.loc.gov"
+        prefs.variable_named("port").default_value = 7090
+        prefs.variable_named("database").default_value = "Voyager"
+        prefs.variable_named("record_syntax").default_value = "USMARC"
+        prefs.variable_named("charset").default_value = "ISO-8859-1"
         prefs.read
       end
 
       def url(book)
-        'http://catalog.loc.gov/cgi-bin/Pwebrecon.cgi?DB=local&CNT=25+records+per+page&CMD=isbn+' +
+        "http://catalog.loc.gov/cgi-bin/Pwebrecon.cgi?DB=local&CNT=25+records+per+page&CMD=isbn+" +
           Library.canonicalise_isbn(book.isbn)
       rescue StandardError => ex
         log.warn { "Cannot create url for book #{book}; #{ex.message}" }
@@ -242,20 +227,20 @@ module Alexandria
       unabstract
 
       include GetText
-      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: 'UTF-8')
+      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: "UTF-8")
 
       def initialize
-        super('BL', _('British Library'))
-        prefs.variable_named('hostname').default_value = 'z3950cat.bl.uk'
-        prefs.variable_named('port').default_value = 9909
-        prefs.variable_named('database').default_value = 'BLAC'
-        prefs.variable_named('record_syntax').default_value = 'SUTRS'
-        prefs.variable_named('charset').default_value = 'ISO-8859-1'
+        super("BL", _("British Library"))
+        prefs.variable_named("hostname").default_value = "z3950cat.bl.uk"
+        prefs.variable_named("port").default_value = 9909
+        prefs.variable_named("database").default_value = "BLAC"
+        prefs.variable_named("record_syntax").default_value = "SUTRS"
+        prefs.variable_named("charset").default_value = "ISO-8859-1"
         prefs.read
       end
 
       def search(criterion, type)
-        return super unless prefs['record_syntax'] == 'SUTRS'
+        return super unless prefs["record_syntax"] == "SUTRS"
 
         prefs.read
         criterion = Library.canonicalise_isbn(criterion) if type == SEARCH_BY_ISBN
@@ -269,7 +254,7 @@ module Alexandria
       end
 
       def url(book)
-        'http://copac.ac.uk/openurl?isbn=' + Library.canonicalise_isbn(book.isbn)
+        "http://copac.ac.uk/openurl?isbn=" + Library.canonicalise_isbn(book.isbn)
       rescue StandardError => ex
         log.warn { "Cannot create url for book #{book}; #{ex.message}" }
         nil
@@ -280,7 +265,7 @@ module Alexandria
       def books_from_sutrs(resultset)
         results = []
         resultset[0..9].each do |record|
-          text = record.render(prefs['charset'], 'UTF-8')
+          text = record.render(prefs["charset"], "UTF-8")
           # File.open(',bl.marc', 'wb') {|f| f.write(text) }
           log.debug { text }
 
@@ -289,7 +274,7 @@ module Alexandria
 
           text.split(/\n/).each do |line|
             if (md = /^Title:\s+(.*)$/.match(line))
-              title = md[1].sub(/\.$/, '').squeeze(' ')
+              title = md[1].sub(/\.$/, "").squeeze(" ")
             elsif (md = /^Added Person Name:\s+(.*),[^,]+$/.match(line))
               authors << md[1]
             elsif (md = /^ME-Personal Name:\s+(.*),[^,]+$/.match(line))
@@ -301,15 +286,15 @@ module Alexandria
             end
           end
 
-          log.debug {
-            msg = 'Parsing SUTRS'
+          log.debug do
+            msg = "Parsing SUTRS"
             msg += "\n title: #{title}"
             msg += "\n authors: #{authors.join(' and ')}"
             msg += "\n isbn: #{isbn}"
             msg += "\n publisher: #{publisher}"
             msg += "\n edition: #{edition}"
             msg
-          }
+          end
 
           if title # and !authors.empty?
             book = Book.new(title, authors, isbn, (publisher || nil),
@@ -327,16 +312,16 @@ module Alexandria
       unabstract
 
       include GetText
-      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: 'UTF-8')
+      GetText.bindtextdomain(Alexandria::TEXTDOMAIN, charset: "UTF-8")
 
       def initialize
-        super('SBN', 'Servizio Bibliotecario Nazionale (Italy)')
-        prefs.variable_named('hostname').default_value = 'opac.sbn.it'
-        prefs.variable_named('port').default_value = 3950
-        prefs.variable_named('database').default_value = 'nopac'
+        super("SBN", "Servizio Bibliotecario Nazionale (Italy)")
+        prefs.variable_named("hostname").default_value = "opac.sbn.it"
+        prefs.variable_named("port").default_value = 3950
+        prefs.variable_named("database").default_value = "nopac"
         # supported 'USMARC', 'UNIMARC' , 'SUTRS'
-        prefs.variable_named('record_syntax').default_value = 'USMARC'
-        prefs.variable_named('charset').default_value = 'ISO-8859-1'
+        prefs.variable_named("record_syntax").default_value = "USMARC"
+        prefs.variable_named("charset").default_value = "ISO-8859-1"
         prefs.read
       end
 
@@ -354,14 +339,14 @@ module Alexandria
       end
 
       def url(book)
-        'http://sbnonline.sbn.it/cgi-bin/zgw/BRIEF.pl?displayquery=' \
-          '%253CB%253E%253Cfont%2520color%253D%2523000064%253E' \
-          'Codice%2520ISBN%253C%2FB%253E%253C%2Ffont%253E%2520' \
-          'contiene%2520%2522%2520%253CFONT%2520COLOR%253Dred%253E' +
+        "http://sbnonline.sbn.it/cgi-bin/zgw/BRIEF.pl?displayquery=" \
+          "%253CB%253E%253Cfont%2520color%253D%2523000064%253E" \
+          "Codice%2520ISBN%253C%2FB%253E%253C%2Ffont%253E%2520" \
+          "contiene%2520%2522%2520%253CFONT%2520COLOR%253Dred%253E" +
           canonicalise_isbn_with_dashes(book.isbn) +
-          '%253C%2FFONT%253E%2522&session=&zurl=opac&zquery=%281%3D7+4%3D2+2%3D3+5%3D100+6%3D1+3%3D3+%22' +
+          "%253C%2FFONT%253E%2522&session=&zurl=opac&zquery=%281%3D7+4%3D2+2%3D3+5%3D100+6%3D1+3%3D3+%22" +
           canonicalise_isbn_with_dashes(book.isbn) +
-          '%22%29&language=it&maxentries=10&target=0&position=1'
+          "%22%29&language=it&maxentries=10&target=0&position=1"
       rescue StandardError => ex
         log.warn { "Cannot create url for book #{book}; #{ex.message}" }
         nil
@@ -375,22 +360,22 @@ module Alexandria
 
         isbn = Alexandria::Library.canonicalise_isbn(isbn)
 
-        if isbn[0..1] == '88'
+        if isbn[0..1] == "88"
           # Italian speaking area
-          if isbn > '8895000' && (isbn <= '8899999996')
-            return isbn[0..1] + '-' + isbn[2..6] + '-' + isbn[7..8] + '-' + isbn[9..9]
-          elsif isbn > '88900000'
-            return isbn[0..1] + '-' + isbn[2..7] + '-' + isbn[8..8] + '-' + isbn[9..9]
-          elsif isbn > '8885000'
-            return isbn[0..1] + '-' + isbn[2..6] + '-' + isbn[7..8] + '-' + isbn[9..9]
-          elsif isbn > '886000'
-            return isbn[0..1] + '-' + isbn[2..5] + '-' + isbn[6..8] + '-' + isbn[9..9]
-          elsif isbn > '88200'
-            return isbn[0..1] + '-' + isbn[2..4] + '-' + isbn[5..8] + '-' + isbn[9..9]
-          elsif isbn > '8800'
-            return isbn[0..1] + '-' + isbn[2..3] + '-' + isbn[4..8] + '-' + isbn[9..9]
+          if isbn > "8895000" && (isbn <= "8899999996")
+            return isbn[0..1] + "-" + isbn[2..6] + "-" + isbn[7..8] + "-" + isbn[9..9]
+          elsif isbn > "88900000"
+            return isbn[0..1] + "-" + isbn[2..7] + "-" + isbn[8..8] + "-" + isbn[9..9]
+          elsif isbn > "8885000"
+            return isbn[0..1] + "-" + isbn[2..6] + "-" + isbn[7..8] + "-" + isbn[9..9]
+          elsif isbn > "886000"
+            return isbn[0..1] + "-" + isbn[2..5] + "-" + isbn[6..8] + "-" + isbn[9..9]
+          elsif isbn > "88200"
+            return isbn[0..1] + "-" + isbn[2..4] + "-" + isbn[5..8] + "-" + isbn[9..9]
+          elsif isbn > "8800"
+            return isbn[0..1] + "-" + isbn[2..3] + "-" + isbn[4..8] + "-" + isbn[9..9]
           else
-            raise 'Invalid ISBN'
+            raise "Invalid ISBN"
           end
 
         else

data/lib/alexandria/config.rb

@@ -2,7 +2,7 @@
 
 module Alexandria
   module Config
-    SHARE_DIR = File.expand_path('../../share', File.dirname(__FILE__))
+    SHARE_DIR = File.expand_path("../../share", File.dirname(__FILE__))
     SOUNDS_DIR = "#{SHARE_DIR}/sounds/alexandria"
     DATA_DIR = "#{SHARE_DIR}/alexandria"
     MAIN_DATA_DIR = DATA_DIR

data/lib/alexandria/console.rb

@@ -9,11 +9,11 @@ module Alexandria
     collection = Alexandria::LibraryCollection.instance
     collection.reload
     libraries = collection.all_regular_libraries
-    output_string = ''
+    output_string = ""
     @books = libraries.flatten
     @books.each do |book|
-      book_authors = book.authors.join(' & ') if authors
-      output_string += [book.title, book_authors].join(', ') + "\n"
+      book_authors = book.authors.join(" & ") if authors
+      output_string += [book.title, book_authors].join(", ") + "\n"
     end
     output_string
   end

data/lib/alexandria/export_format.rb

@@ -11,17 +11,17 @@ module Alexandria
     include GetText
     include Logging
     extend GetText
-    bindtextdomain(Alexandria::TEXTDOMAIN, charset: 'UTF-8')
+    bindtextdomain(Alexandria::TEXTDOMAIN, charset: "UTF-8")
 
     def self.all
       [
-        new(_('Archived ONIX XML'), 'onix.tbz2', :export_as_onix_xml_archive),
-        new(_('Archived Tellico XML'), 'tc', :export_as_tellico_xml_archive),
-        new(_('BibTeX'), 'bib', :export_as_bibtex),
-        new(_('CSV list'), 'csv', :export_as_csv_list),
-        new(_('ISBN List'), 'txt', :export_as_isbn_list),
-        new(_('iPod Notes'), nil, :export_as_ipod_notes),
-        new(_('HTML Web Page'), nil, :export_as_html, true)
+        new(_("Archived ONIX XML"), "onix.tbz2", :export_as_onix_xml_archive),
+        new(_("Archived Tellico XML"), "tc", :export_as_tellico_xml_archive),
+        new(_("BibTeX"), "bib", :export_as_bibtex),
+        new(_("CSV list"), "csv", :export_as_csv_list),
+        new(_("ISBN List"), "txt", :export_as_isbn_list),
+        new(_("iPod Notes"), nil, :export_as_ipod_notes),
+        new(_("HTML Web Page"), nil, :export_as_html, true)
       ]
     end
 

data/lib/alexandria/export_library.rb

@@ -4,9 +4,9 @@
 #
 # See the file README.md for authorship and licensing information.
 
-require 'csv'
-require 'image_size'
-require 'tmpdir'
+require "csv"
+require "image_size"
+require "tmpdir"
 
 module Alexandria
   class ExportLibrary
@@ -36,20 +36,20 @@ module Alexandria
     end
 
     def export_as_onix_xml_archive(filename)
-      File.open(File.join(Dir.tmpdir, 'onix.xml'), 'w') do |io|
+      File.open(File.join(Dir.tmpdir, "onix.xml"), "w") do |io|
         to_onix_document.write(io, 0)
       end
-      copy_covers(File.join(Dir.tmpdir, 'images'))
+      copy_covers(File.join(Dir.tmpdir, "images"))
       Dir.chdir(Dir.tmpdir) do
         output = `tar -cjf \"#{filename}\" onix.xml images 2>&1`
         raise output unless $CHILD_STATUS.success?
       end
-      FileUtils.rm_rf(File.join(Dir.tmpdir, 'images'))
-      FileUtils.rm(File.join(Dir.tmpdir, 'onix.xml'))
+      FileUtils.rm_rf(File.join(Dir.tmpdir, "images"))
+      FileUtils.rm(File.join(Dir.tmpdir, "onix.xml"))
     end
 
     def export_as_tellico_xml_archive(filename)
-      File.open(File.join(Dir.tmpdir, 'tellico.xml'), 'w') do |io|
+      File.open(File.join(Dir.tmpdir, "tellico.xml"), "w") do |io|
         begin
           to_tellico_document.write(io, 0)
         rescue StandardError => ex
@@ -58,19 +58,19 @@ module Alexandria
           raise ex
         end
       end
-      copy_covers(File.join(Dir.tmpdir, 'images'))
+      copy_covers(File.join(Dir.tmpdir, "images"))
       Dir.chdir(Dir.tmpdir) do
         output = `zip -q -r \"#{filename}\" tellico.xml images 2>&1`
         raise output unless $CHILD_STATUS.success?
       end
-      FileUtils.rm_rf(File.join(Dir.tmpdir, 'images'))
-      FileUtils.rm(File.join(Dir.tmpdir, 'tellico.xml'))
+      FileUtils.rm_rf(File.join(Dir.tmpdir, "images"))
+      FileUtils.rm(File.join(Dir.tmpdir, "tellico.xml"))
     end
 
     def export_as_isbn_list(filename)
-      File.open(filename, 'w') do |io|
+      File.open(filename, "w") do |io|
         each do |book|
-          io.puts((book.isbn || ''))
+          io.puts((book.isbn || ""))
         end
       end
     end
@@ -78,17 +78,17 @@ module Alexandria
     def export_as_html(filename, theme)
       FileUtils.mkdir(filename) unless File.exist?(filename)
       Dir.chdir(filename) do
-        copy_covers('pixmaps')
-        FileUtils.cp_r(theme.pixmaps_directory, 'pixmaps') if theme.has_pixmaps?
-        FileUtils.cp(theme.css_file, '.')
-        File.open('index.html', 'w') do |io|
+        copy_covers("pixmaps")
+        FileUtils.cp_r(theme.pixmaps_directory, "pixmaps") if theme.has_pixmaps?
+        FileUtils.cp(theme.css_file, ".")
+        File.open("index.html", "w") do |io|
           io << to_xhtml(File.basename(theme.css_file))
         end
       end
     end
 
     def export_as_bibtex(filename)
-      File.open(filename, 'w') do |io|
+      File.open(filename, "w") do |io|
         io << to_bibtex
       end
     end
@@ -97,26 +97,26 @@ module Alexandria
       FileUtils.mkdir(filename) unless File.exist?(filename)
       tempdir = Dir.getwd
       Dir.chdir(filename)
-      copy_covers('pixmaps')
-      File.open('index.linx', 'w') do |io|
-        io.puts '<TITLE>' + name + '</TITLE>'
+      copy_covers("pixmaps")
+      File.open("index.linx", "w") do |io|
+        io.puts "<TITLE>" + name + "</TITLE>"
         each do |book|
-          io.puts '<A HREF="' + book.ident + '">' + book.title + '</A>'
+          io.puts '<A HREF="' + book.ident + '">' + book.title + "</A>"
         end
         io.close
       end
       each do |book|
-        File.open(book.ident, 'w') do |io|
+        File.open(book.ident, "w") do |io|
           io.puts "<TITLE>#{book.title} </TITLE>"
           # put a link to the book's cover. only works on iPod 5G and above(?).
           if File.exist?(cover(book))
-            io.puts '<A HREF="pixmaps/' + book.ident + '.jpg' + '">' + book.title + '</A>'
+            io.puts '<A HREF="pixmaps/' + book.ident + ".jpg" + '">' + book.title + "</A>"
           else
             io.puts book.title
           end
-          io.puts book.authors.join(', ')
+          io.puts book.authors.join(", ")
           io.puts book.edition
-          io.puts((book.isbn || ''))
+          io.puts((book.isbn || ""))
           # we need to close the files so the iPod can be ejected/unmounted without us closing Alexandria
           io.close
         end
@@ -126,84 +126,83 @@ module Alexandria
     end
 
     def export_as_csv_list(filename)
-      CSV.open(filename, 'w', col_sep: ';') do |csv|
-        csv << ['Title', 'Authors', 'Publisher', 'Edition', 'ISBN', 'Year Published',
-                "Rating(#{Book::DEFAULT_RATING} to #{Book::MAX_RATING_STARS})", 'Notes',
-                'Want?', 'Read?', 'Own?', 'Tags']
+      CSV.open(filename, "w", col_sep: ";") do |csv|
+        csv << ["Title", "Authors", "Publisher", "Edition", "ISBN", "Year Published",
+                "Rating(#{Book::DEFAULT_RATING} to #{Book::MAX_RATING_STARS})", "Notes",
+                "Want?", "Read?", "Own?", "Tags"]
         each do |book|
-          csv << [book.title, book.authors.join(', '), book.publisher, book.edition, book.isbn,
+          csv << [book.title, book.authors.join(", "), book.publisher, book.edition, book.isbn,
                   book.publishing_year, book.rating, book.notes,
-                  (book.want ? '1' : '0'), (book.redd ? '1' : '0'), (book.own ? '1' : '0'),
-                  (book.tags ? book.tags.join(', ') : '')]
+                  (book.want ? "1" : "0"), (book.redd ? "1" : "0"), (book.own ? "1" : "0"),
+                  (book.tags ? book.tags.join(", ") : "")]
         end
       end
     end
 
     private
 
-    ONIX_DTD_URL = 'http://www.editeur.org/onix/2.1/reference/onix-international.dtd'
+    ONIX_DTD_URL = "http://www.editeur.org/onix/2.1/reference/onix-international.dtd"
     def to_onix_document
       doc = REXML::Document.new
       doc << REXML::XMLDecl.new
-      doc << REXML::DocType.new('ONIXMessage',
+      doc << REXML::DocType.new("ONIXMessage",
                                 "SYSTEM \"#{ONIX_DTD_URL}\"")
-      msg = doc.add_element('ONIXMessage')
-      header = msg.add_element('Header')
-      header.add_element('FromCompany').text = 'Alexandria'
-      header.add_element('FromPerson').text = Etc.getlogin
+      msg = doc.add_element("ONIXMessage")
+      header = msg.add_element("Header")
+      header.add_element("FromCompany").text = "Alexandria"
+      header.add_element("FromPerson").text = Etc.getlogin
       now = Time.now
-      header.add_element('SentDate').text = format('%.4d%.2d%.2d%.2d%.2d',
-                                                   now.year, now.month, now.day, now.hour, now.min)
-      header.add_element('MessageNote').text = name
+      header.add_element("SentDate").text = now.strftime("%Y%m%d%H%M")
+      header.add_element("MessageNote").text = name
       @sorted.each_with_index do |book, idx|
         # fields that are missing: edition and rating.
-        prod = msg.add_element('Product')
-        prod.add_element('RecordReference').text = idx
-        prod.add_element('NotificationType').text = '03'  # confirmed
-        prod.add_element('RecordSourceName').text =
-          'Alexandria ' + Alexandria::DISPLAY_VERSION
-        prod.add_element('ISBN').text = (book.isbn || '')
-        prod.add_element('ProductForm').text = 'BA'       # book
-        prod.add_element('DistinctiveTitle').text = book.title
+        prod = msg.add_element("Product")
+        prod.add_element("RecordReference").text = idx
+        prod.add_element("NotificationType").text = "03"  # confirmed
+        prod.add_element("RecordSourceName").text =
+          "Alexandria " + Alexandria::DISPLAY_VERSION
+        prod.add_element("ISBN").text = (book.isbn || "")
+        prod.add_element("ProductForm").text = "BA"       # book
+        prod.add_element("DistinctiveTitle").text = book.title
         unless book.authors.empty?
           book.authors.each do |author|
-            elem = prod.add_element('Contributor')
+            elem = prod.add_element("Contributor")
             # author
-            elem.add_element('ContributorRole').text = 'A01'
-            elem.add_element('PersonName').text = author
+            elem.add_element("ContributorRole").text = "A01"
+            elem.add_element("PersonName").text = author
           end
         end
         if book.notes && !book.notes.empty?
-          elem = prod.add_element('OtherText')
+          elem = prod.add_element("OtherText")
           # reader description
-          elem.add_element('TextTypeCode').text = '12'
-          elem.add_element('TextFormat').text = '00' # ASCII
-          elem.add_element('Text').text = book.notes
+          elem.add_element("TextTypeCode").text = "12"
+          elem.add_element("TextFormat").text = "00" # ASCII
+          elem.add_element("Text").text = book.notes
         end
         if File.exist?(cover(book))
-          elem = prod.add_element('MediaFile')
+          elem = prod.add_element("MediaFile")
           # front cover image
-          elem.add_element('MediaFileTypeCode').text = '04'
-          elem.add_element('MediaFileFormatCode').text =
-            (Library.jpeg?(cover(book)) ? '03' : '02')
+          elem.add_element("MediaFileTypeCode").text = "04"
+          elem.add_element("MediaFileFormatCode").text =
+            (Library.jpeg?(cover(book)) ? "03" : "02")
           # filename
-          elem.add_element('MediaFileLinkTypeCode').text = '06'
-          elem.add_element('MediaFileLink').text =
-            File.join('images', final_cover(book))
+          elem.add_element("MediaFileLinkTypeCode").text = "06"
+          elem.add_element("MediaFileLink").text =
+            File.join("images", final_cover(book))
         end
         if book.isbn
           BookProviders.each do |provider|
-            elem = prod.add_element('ProductWebsite')
-            elem.add_element('ProductWebsiteDescription').text =
+            elem = prod.add_element("ProductWebsite")
+            elem.add_element("ProductWebsiteDescription").text =
               provider.fullname
-            elem.add_element('ProductWebsiteLink').text =
+            elem.add_element("ProductWebsiteLink").text =
               provider.url(book)
           end
         end
-        elem = prod.add_element('Publisher')
-        elem.add_element('PublishingRole').text = '01'
-        elem.add_element('PublisherName').text = book.publisher
-        prod.add_element('PublicationDate').text = book.publishing_year
+        elem = prod.add_element("Publisher")
+        elem.add_element("PublishingRole").text = "01"
+        elem.add_element("PublisherName").text = book.publisher
+        prod.add_element("PublicationDate").text = book.publishing_year
       end
       doc
     end
@@ -213,49 +212,49 @@ module Alexandria
       # http://periapsis.org/tellico/doc/hacking.html
       doc = REXML::Document.new
       doc << REXML::XMLDecl.new
-      doc << REXML::DocType.new('tellico',
+      doc << REXML::DocType.new("tellico",
                                 'PUBLIC "-//Robby Stephenson/DTD Tellico V7.0//EN"' \
                                 ' "http://periapsis.org/tellico/dtd/v7/tellico.dtd"')
-      tellico = doc.add_element('tellico')
-      tellico.add_attribute('syntaxVersion', '7')
-      tellico.add_namespace('http://periapsis.org/tellico/')
-      collection = tellico.add_element('collection')
-      collection.add_attribute('title', name)
-      collection.add_attribute('type', '2')
-      fields = collection.add_element('fields')
-      field1 = fields.add_element('field')
+      tellico = doc.add_element("tellico")
+      tellico.add_attribute("syntaxVersion", "7")
+      tellico.add_namespace("http://periapsis.org/tellico/")
+      collection = tellico.add_element("collection")
+      collection.add_attribute("title", name)
+      collection.add_attribute("type", "2")
+      fields = collection.add_element("fields")
+      field1 = fields.add_element("field")
       # a field named _default implies adding all default book
       # collection fields
-      field1.add_attribute('name', '_default')
-      images = collection.add_element('images')
+      field1.add_attribute("name", "_default")
+      images = collection.add_element("images")
       @sorted.each_with_index do |book, idx|
-        entry = collection.add_element('entry')
+        entry = collection.add_element("entry")
         new_index = (idx + 1).to_s
-        entry.add_attribute('id', new_index)
+        entry.add_attribute("id", new_index)
         # translate the binding
-        entry.add_element('title').text = book.title
-        entry.add_element('isbn').text = (book.isbn || '')
-        entry.add_element('pub_year').text = book.publishing_year
-        entry.add_element('binding').text = book.edition
-        entry.add_element('publisher').text = book.publisher
+        entry.add_element("title").text = book.title
+        entry.add_element("isbn").text = (book.isbn || "")
+        entry.add_element("pub_year").text = book.publishing_year
+        entry.add_element("binding").text = book.edition
+        entry.add_element("publisher").text = book.publisher
         unless book.authors.empty?
-          authors = entry.add_element('authors')
+          authors = entry.add_element("authors")
           book.authors.each do |author|
-            authors.add_element('author').text = author
+            authors.add_element("author").text = author
           end
         end
-        entry.add_element('read').text = book.redd.to_s if book.redd
-        entry.add_element('loaned').text = book.loaned.to_s if book.loaned
-        entry.add_element('rating').text = book.rating unless book.rating == Book::DEFAULT_RATING
-        entry.add_element('comments').text = book.notes if book.notes && !book.notes.empty?
+        entry.add_element("read").text = book.redd.to_s if book.redd
+        entry.add_element("loaned").text = book.loaned.to_s if book.loaned
+        entry.add_element("rating").text = book.rating unless book.rating == Book::DEFAULT_RATING
+        entry.add_element("comments").text = book.notes if book.notes && !book.notes.empty?
         if File.exist?(cover(book))
-          entry.add_element('cover').text = final_cover(book)
-          image = images.add_element('image')
-          image.add_attribute('id', final_cover(book))
+          entry.add_element("cover").text = final_cover(book)
+          image = images.add_element("image")
+          image.add_attribute("id", final_cover(book))
           image_s = ImageSize.new(IO.read(cover(book)))
-          image.add_attribute('height', image_s.height.to_s)
-          image.add_attribute('width', image_s.width.to_s)
-          image.add_attribute('format', image_s.format)
+          image.add_attribute("height", image_s.height.to_s)
+          image.add_attribute("width", image_s.width.to_s)
+          image.add_attribute("format", image_s.format)
         end
       end
       doc
@@ -264,16 +263,16 @@ module Alexandria
     def xhtml_escape(str)
       escaped = str.dup
       # used to occasionally use CGI.escapeHTML
-      escaped.gsub!(/&/, '&amp;')
-      escaped.gsub!(/</, '&lt;')
-      escaped.gsub!(/>/, '&gt;')
-      escaped.gsub!(/\"/, '&quot;')
+      escaped.gsub!(/&/, "&amp;")
+      escaped.gsub!(/</, "&lt;")
+      escaped.gsub!(/>/, "&gt;")
+      escaped.gsub!(/\"/, "&quot;")
       escaped
     end
 
     def to_xhtml(css)
-      generator = 'Alexandria ' + Alexandria::DISPLAY_VERSION
-      xhtml = +''
+      generator = "Alexandria " + Alexandria::DISPLAY_VERSION
+      xhtml = +""
       xhtml << <<~EOS
         <?xml version="1.0" encoding="UTF-8"?>
         <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
@@ -326,7 +325,7 @@ module Alexandria
               <li class="book_author">#{xhtml_escape(author)}</li>
             EOS
           end
-          xhtml << '</ul>'
+          xhtml << "</ul>"
         end
 
         unless book.edition.nil?
@@ -356,15 +355,15 @@ module Alexandria
     end
 
     def to_bibtex
-      generator = 'Alexandria ' + Alexandria::DISPLAY_VERSION
-      bibtex = +''
+      generator = "Alexandria " + Alexandria::DISPLAY_VERSION
+      bibtex = +""
       bibtex << "\%Generated on #{Date.today} by: #{generator}\n"
       bibtex << "\%\n"
       bibtex << "\n"
 
       auths = Hash.new(0)
       each do |book|
-        k = (book.authors[0] || 'Anonymous').split[0]
+        k = (book.authors[0] || "Anonymous").split[0]
         if auths.key?(k)
           auths[k] += 1
         else
@@ -384,14 +383,14 @@ module Alexandria
         bibtex << "publisher = \"#{latex_escape(book.publisher)}\",\n"
         bibtex << "OPTnote = \"#{latex_escape(book.notes)}\",\n" if book.notes && !book.notes.empty?
         # year is a required field in bibtex @BOOK
-        bibtex << 'year = ' + (book.publishing_year || '"n/a"').to_s + "\n"
+        bibtex << "year = " + (book.publishing_year || '"n/a"').to_s + "\n"
         bibtex << "}\n\n"
       end
       bibtex
     end
 
     def latex_escape(str)
-      return '' if str.nil?
+      return "" if str.nil?
 
       my_str = str.dup
       my_str.gsub!(/%/, '\\%')