0.9.19

Changelog.yaml

@@ -19,6 +19,49 @@
 #   new recipes:
 #     - title: 
 
+- version: 0.9.19
+  date: 2013-02-15
+
+  new features:
+    - title: "New tool: \"Polish books\" that allows you to perform various automated cleanup actions on EPUB and AZW3 files without doing a full conversion."
+      type: major
+      description: "Polishing books is all about putting the shine of perfection on your ebook files. You can use it to subset embedded fonts, update the metadata in the book files from the metadata in the calibre library, manipulate the book jacket, etc. More features will be added in the future. To use this tool, go to Preferences->Toolbar and add the Polish books tool to the main toolbar. Then simply select the books you want to be polished and click the Polish books button. Polishing, unlike conversion, does not change the internal structure/markup of your book, it performs only the minimal set of actions needed to achieve its goals. Note that polish books is a completely new codebase, so there may well be bugs, polishing a book backs up the original as ORIGINAL_EPUB or ORIGINAL_AZW3, unless you have turned off this feature in Preferences->Tweaks, in which case you should backup your files manually. You can also use this tool from the command line with ebook-polish.exe."
+
+    - title: "Driver for the Trekstor Pyrus Mini."
+      tickets: [1124120]
+
+    - title: "E-book viewer: Add an option to change the minimum font size."
+      tickets: [1122333]
+ 
+    - title: "PDF Output: Add support for converting documents with math typesetting, as described here: http://manual.calibre-ebook.com/typesetting_math.html"
+
+    - title: "Column coloring/icons: Add more conditions when using date based columns with reference to 'today'."
+
+  bug fixes:
+    - title: "Transforming to titlecase - handle typographic hyphens in all caps phrases"
+
+    - title: "Dont ignore file open events that occur before the GUI is initialized on OS X"
+      tickets: [1122713]
+
+    - title: "News download: Handle feeds that have entries with empty ids"
+
+    - title: "Fix a regression that broke using the template editor"
+
+    - title: "Do not block startup while scanning the computer for available network interfaces. Speeds up startup time on some windows computers with lots of spurious network interfaces."
+
+  improved recipes:
+    - New Yorker
+    - Kommersant
+    - Le Monde (Subscription version)
+    - NZ Herald
+
+  new recipes:
+    - title: Navegalo 
+      author: Douglas Delgado
+
+    - title: El Guardian and More Intelligent Life
+      author: Darko Miletic
+
 - version: 0.9.18
   date: 2013-02-08
 

manual/develop.rst

@@ -39,27 +39,27 @@ All the |app| python code is in the ``calibre`` package. This package contains t
 
     * devices - All the device drivers. Just look through some of the built-in drivers to get an idea for how they work.
 
-      * For details, see: devices.interface which defines the interface supported by device drivers and devices.usbms which
+      * For details, see: devices.interface which defines the interface supported by device drivers and ``devices.usbms`` which
         defines a generic driver that connects to a USBMS device. All USBMS based drivers in |app| inherit from it.
 
     * ebooks  - All the ebook conversion/metadata code. A good starting point is ``calibre.ebooks.conversion.cli`` which is the
-      module powering the :command:`ebook-convert` command. The conversion process is controlled via conversion.plumber.
+      module powering the :command:`ebook-convert` command. The conversion process is controlled via ``conversion.plumber``.
-      The format independent code is all in ebooks.oeb and the format dependent code is in ebooks.format_name.
+      The format independent code is all in ``ebooks.oeb`` and the format dependent code is in ``ebooks.format_name``.
 
-        * Metadata reading, writing, and downloading is all in ebooks.metadata
+        * Metadata reading, writing, and downloading is all in ``ebooks.metadata``
         * Conversion happens in a pipeline, for the structure of the pipeline,
           see :ref:`conversion-introduction`. The pipeline consists of an input
           plugin, various transforms and an output plugin. The that code constructs
-          and drives the pipeline is in plumber.py. The pipeline works on a
+          and drives the pipeline is in :file:`plumber.py`. The pipeline works on a
           representation of an ebook that is like an unzipped epub, with
           manifest, spine, toc, guide, html content, etc. The
-          class that manages this representation is OEBBook in oeb/base.py. The
+          class that manages this representation is OEBBook in ``ebooks.oeb.base``. The
           various transformations that are applied to the book during
-          conversions live in `oeb/transforms/*.py`. And the input and output
+          conversions live in :file:`oeb/transforms/*.py`. And the input and output
-          plugins live in `conversion/plugins/*.py`.
+          plugins live in :file:`conversion/plugins/*.py`.
 
-    * library - The database back-end and the content server. See library.database2 for the interface to the |app| library. library.server is the |app| Content Server.
+    * library - The database back-end and the content server. See ``library.database2`` for the interface to the |app| library. ``library.server`` is the |app| Content Server.
-    * gui2 - The Graphical User Interface. GUI initialization happens in gui2.main and gui2.ui. The ebook-viewer is in gui2.viewer.
+    * gui2 - The Graphical User Interface. GUI initialization happens in ``gui2.main`` and ``gui2.ui``. The ebook-viewer is in ``gui2.viewer``.
 
 If you need help understanding the code, post in the `development forum <http://www.mobileread.com/forums/forumdisplay.php?f=240>`_
 and you will most likely get help from one of |app|'s many developers.

manual/faq.rst

@@ -250,42 +250,71 @@ If you don't want to uninstall it altogether, there are a couple of tricks you c
 simplest is to simply re-name the executable file that launches the library program. More detail
 `in the forums <http://www.mobileread.com/forums/showthread.php?t=65809>`_.
 
-How do I use |app| with my iPad/iPhone/iTouch?
+How do I use |app| with my iPad/iPhone/iPod touch?
-~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 Over the air
 ^^^^^^^^^^^^^^
 
-The easiest way to browse your |app| collection on your Apple device (iPad/iPhone/iPod) is by using the calibre content server, which makes your collection available over the net. First perform the following steps in |app|
+The easiest way to browse your |app| collection on your Apple device
+(iPad/iPhone/iPod) is by using the |app| content server, which makes your
+collection available over the net. First perform the following steps in |app|
 
-  * Set the Preferred Output Format in |app| to EPUB (The output format can be set under :guilabel:`Preferences->Interface->Behavior`)
+  * Set the Preferred Output Format in |app| to EPUB (The output format can be
-  * Set the output profile to iPad (this will work for iPhone/iPods as well), under :guilabel:`Preferences->Conversion->Common Options->Page Setup`
+    set under :guilabel:`Preferences->Interface->Behavior`)
-  * Convert the books you want to read on your iPhone to EPUB format by selecting them and clicking the Convert button.
+  * Set the output profile to iPad (this will work for iPhone/iPods as well),
-  * Turn on the Content Server in |app|'s preferences and leave |app| running.
+    under :guilabel:`Preferences->Conversion->Common Options->Page Setup`
+  * Convert the books you want to read on your iDevice to EPUB format by
+    selecting them and clicking the Convert button.
+  * Turn on the Content Server by clicking the :guilabel:`Connect/Share` button
+    and leave |app| running. You can also tell |app| to automatically start the
+    content server via :guilabel:`Preferences->Sharing over the net`.
 
-Now on your iPad/iPhone you have two choices, use either iBooks (version 1.2 and later) or Stanza (version 3.0 and later). Both are available free from the app store.
+There are many apps for your iDevice that can connect to |app|. Here we
+describe using two of them, iBooks and Stanza.
 
 Using Stanza
 ***************
 
-Now you should be able to access your books on your iPhone by opening Stanza. Go to "Get Books" and then click the "Shared" tab. Under Shared you will see an entry "Books in calibre". If you don't, make sure your iPad/iPhone is connected using the WiFi network in your house, not 3G. If the |app| catalog is still not detected in Stanza, you can add it manually in Stanza. To do this, click the "Shared" tab, then click the "Edit" button and then click "Add book source" to add a new book source. In the Add Book Source screen enter whatever name you like and in the URL field, enter the following::
+You should be able to access your books on your iPhone by opening Stanza. Go to
+"Get Books" and then click the "Shared" tab. Under Shared you will see an entry
+"Books in calibre". If you don't, make sure your iPad/iPhone is connected using
+the WiFi network in your house, not 3G. If the |app| catalog is still not
+detected in Stanza, you can add it manually in Stanza. To do this, click the
+"Shared" tab, then click the "Edit" button and then click "Add book source" to
+add a new book source. In the Add Book Source screen enter whatever name you
+like and in the URL field, enter the following::
 
     http://192.168.1.2:8080/
 
-Replace ``192.168.1.2`` with the local IP address of the computer running |app|. If you have changed the port the |app| content server is running on, you will have to change ``8080`` as well to the new port. The local IP address is the IP address you computer is assigned on your home network. A quick Google search will tell you how to find out your local IP address.   Now click "Save" and you are done.
+Replace ``192.168.1.2`` with the local IP address of the computer running
+|app|. If you have changed the port the |app| content server is running on, you
+will have to change ``8080`` as well to the new port. The local IP address is
+the IP address you computer is assigned on your home network. A quick Google
+search will tell you how to find out your local IP address.   Now click "Save"
+and you are done.
 
-If you get timeout errors while browsing the calibre catalog in Stanza, try increasing the connection timeout value in the stanza settings. Go to Info->Settings and increase the value of Download Timeout.
+If you get timeout errors while browsing the calibre catalog in Stanza, try
+increasing the connection timeout value in the stanza settings. Go to
+Info->Settings and increase the value of Download Timeout.
 
 Using iBooks
 **************
 
-Start the Safari browser and type in the IP address and port of the computer running the calibre server, like this::
+Start the Safari browser and type in the IP address and port of the computer
+running the calibre server, like this::
 
     http://192.168.1.2:8080/
 
-Replace ``192.168.1.2`` with the local IP address of the computer running |app|. If you have changed the port the |app| content server is running on, you will have to change ``8080`` as well to the new port. The local IP address is the IP address you computer is assigned on your home network. A quick Google search will tell you how to find out your local IP address.
+Replace ``192.168.1.2`` with the local IP address of the computer running
+|app|. If you have changed the port the |app| content server is running on, you
+will have to change ``8080`` as well to the new port. The local IP address is
+the IP address you computer is assigned on your home network. A quick Google
+search will tell you how to find out your local IP address.
 
-You will see a list of books in Safari, just click on the epub link for whichever book you want to read, Safari will then prompt you to open it with iBooks.
+You will see a list of books in Safari, just click on the epub link for
+whichever book you want to read, Safari will then prompt you to open it with
+iBooks.
 
 
 With the USB cable + iTunes

manual/gui.rst

@@ -537,6 +537,8 @@ Calibre has several keyboard shortcuts to save you time and mouse movement. Thes
       - Merge selected records, keeping originals
     * - :kbd:`O`
       - Open containing folder
+    * - :kbd:`P`
+      - Polish books
     * - :kbd:`S`
       - Save to Disk
     * - :kbd:`V`

recipes/adventure_zone_pl.recipe

@@ -11,7 +11,7 @@ class Adventure_zone(BasicNewsRecipe):
     max_articles_per_feed = 100
     cover_url = 'http://www.adventure-zone.info/inne/logoaz_2012.png'
     index='http://www.adventure-zone.info/fusion/'
-    use_embedded_content=False
+    use_embedded_content = False
     preprocess_regexps     = [(re.compile(r"<td class='capmain'>Komentarze</td>", re.IGNORECASE), lambda m: ''),
     (re.compile(r'</?table.*?>'), lambda match: ''),
     (re.compile(r'</?tbody.*?>'), lambda match: '')]
@@ -21,7 +21,7 @@ class Adventure_zone(BasicNewsRecipe):
     extra_css              = '.main-bg{text-align: left;}  td.capmain{ font-size: 22px; }'
     feeds          = [(u'Nowinki', u'http://www.adventure-zone.info/fusion/feeds/news.php')]
 
-    def parse_feeds (self): 
+    '''def parse_feeds (self): 
       feeds = BasicNewsRecipe.parse_feeds(self) 
       soup=self.index_to_soup(u'http://www.adventure-zone.info/fusion/feeds/news.php')
       tag=soup.find(name='channel')
@@ -34,7 +34,7 @@ class Adventure_zone(BasicNewsRecipe):
       for feed in feeds:
         for article in feed.articles[:]:
             article.title=titles[feed.articles.index(article)]
-      return feeds
+      return feeds'''
 
 
     '''def get_cover_url(self):
@@ -42,16 +42,25 @@ class Adventure_zone(BasicNewsRecipe):
         cover=soup.find(id='box_OstatninumerAZ')
         self.cover_url='http://www.adventure-zone.info/fusion/'+ cover.center.a.img['src']
         return getattr(self, 'cover_url', self.cover_url)'''
-
+    def populate_article_metadata(self, article, soup, first):
+        result = re.search('(.+) - Adventure Zone', soup.title.string)
+        if result:
+            article.title = result.group(1)
+        else:
+            result = soup.body.find('strong')
+            if result:
+                article.title = result.string
 
     def skip_ad_pages(self, soup):
         skip_tag = soup.body.find(name='td', attrs={'class':'main-bg'})
         skip_tag = skip_tag.findAll(name='a')
-        for r in skip_tag:
+        title = soup.title.string.lower()
-           if r.strong:
+        if (('zapowied' in title) or ('recenzj' in title)  or ('solucj' in title) or ('poradnik' in title)):
-                 word=r.strong.string.lower()
+            for r in skip_tag:
-                 if word and (('zapowied' in word) or ('recenzj' in word)  or ('solucj' in word) or ('poradnik' in word)):
+                if r.strong and r.strong.string:
-                   return self.index_to_soup('http://www.adventure-zone.info/fusion/print.php?type=A&item'+r['href'][r['href'].find('article_id')+7:], raw=True)
+                   word=r.strong.string.lower()
+                   if (('zapowied' in word) or ('recenzj' in word)  or ('solucj' in word) or ('poradnik' in word)):
+                       return self.index_to_soup('http://www.adventure-zone.info/fusion/print.php?type=A&item'+r['href'][r['href'].find('article_id')+7:], raw=True)
 
     def preprocess_html(self, soup):
         footer=soup.find(attrs={'class':'news-footer middle-border'})

recipes/badania_net.recipe

@@ -0,0 +1,17 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+class BadaniaNet(BasicNewsRecipe):
+    title          = u'badania.net'
+    __author__ = 'fenuks'
+    description   = u'chcesz wiedzieć więcej?'
+    category       = 'science'
+    language       = 'pl'
+    cover_url = 'http://badania.net/wp-content/badanianet_green_transparent.png'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    no_stylesheets = True
+    remove_empty_feeds = True
+    use_embedded_content = False
+    remove_tags = [dict(attrs={'class':['omc-flex-category', 'omc-comment-count', 'omc-single-tags']})]
+    remove_tags_after = dict(attrs={'class':'omc-single-tags'})
+    keep_only_tags = [dict(id='omc-full-article')]
+    feeds          = [(u'Psychologia', u'http://badania.net/category/psychologia/feed/'), (u'Technologie', u'http://badania.net/category/technologie/feed/'), (u'Biologia', u'http://badania.net/category/biologia/feed/'), (u'Chemia', u'http://badania.net/category/chemia/feed/'), (u'Zdrowie', u'http://badania.net/category/zdrowie/'), (u'Seks', u'http://badania.net/category/psychologia-ewolucyjna-tematyka-seks/feed/')]

recipes/bash_org_pl.recipe

@@ -35,8 +35,8 @@ class Bash_org_pl(BasicNewsRecipe):
              soup=self.index_to_soup(u'http://bash.org.pl/random/')
              #date=soup.find('div', attrs={'class':'right'}).string
              url=soup.find('a', attrs={'class':'qid click'})
-             title=url.string
+             title=''
-             url='http://bash.org.pl' +url['href']
+             url='http://bash.org.pl/random/'
              articles.append({'title' : title,
 	               'url'   : url,
 	               'date'  : '',
@@ -44,6 +44,8 @@ class Bash_org_pl(BasicNewsRecipe):
 	                })
          return articles
 
+    def populate_article_metadata(self, article, soup, first):
+        article.title = soup.find(attrs={'class':'qid click'}).string
 
     def parse_index(self):
          feeds          = []

recipes/discover_magazine.recipe

@@ -33,6 +33,21 @@ class DiscoverMagazine(BasicNewsRecipe):
 
     remove_tags_after = [dict(name='div', attrs={'class':'listingBar'})]
 
+    # Login stuff
+    needs_subscription = True
+    use_javascript_to_login = True
+    requires_version = (0, 9, 20)
+
+    def javascript_login(self, br, username, password):
+        br.visit('http://discovermagazine.com', timeout=120)
+        f = br.select_form('div.login.section div.form')
+        f['username'] = username
+        f['password'] = password
+        br.submit('input[id="signInButton"]', timeout=120)
+        br.run_for_a_time(20)
+    # End login stuff
+
+
     def append_page(self, soup, appendtag, position):
         pager = soup.find('span',attrs={'class':'next'})
         if pager:

recipes/ekologia_pl.recipe

@@ -15,7 +15,8 @@ class EkologiaPl(BasicNewsRecipe):
     no_stylesheets = True
     remove_empty_feeds = True
     use_embedded_content = False
-    remove_tags = [dict(attrs={'class':['ekoLogo', 'powrocArt', 'butonDrukuj']})]
+    remove_attrs = ['style']
+    remove_tags = [dict(attrs={'class':['ekoLogo', 'powrocArt', 'butonDrukuj', 'widget-social-buttons']})]
 
     feeds          = [(u'Wiadomo\u015bci', u'http://www.ekologia.pl/rss/20,53,0'), (u'\u015arodowisko', u'http://www.ekologia.pl/rss/20,56,0'), (u'Styl \u017cycia', u'http://www.ekologia.pl/rss/20,55,0')]
 

recipes/elguardian.recipe

@@ -0,0 +1,93 @@
+__license__   = 'GPL v3'
+__copyright__ = '2013, Darko Miletic <darko.miletic at gmail.com>'
+'''
+elguardian.com.ar
+'''
+
+import re
+from calibre.web.feeds.news import BasicNewsRecipe
+
+class ElGuardian(BasicNewsRecipe):
+    title                 = 'El Guardian'
+    __author__            = 'Darko Miletic'
+    description           = "Semanario con todas las tendencias de un pais"
+    publisher             = 'Editorial Apache SA'
+    category              = 'news,politics,Argentina'
+    oldest_article        = 8
+    max_articles_per_feed = 200
+    no_stylesheets        = True
+    encoding              = 'utf8'
+    use_embedded_content  = False
+    language              = 'es_AR'
+    remove_empty_feeds    = True
+    publication_type      = 'magazine'
+    issn                  = '1666-7476'
+    masthead_url          = 'http://elguardian.com.ar/application/templates/frontend/images/home/logo.png'
+    extra_css             = """
+                               body{font-family: Arial,sans-serif}
+                               img{margin-bottom: 0.4em; display:block}
+                            """
+
+    conversion_options = {
+                          'comment'   : description
+                        , 'tags'      : category
+                        , 'publisher' : publisher
+                        , 'language'  : language
+                        , 'series'    : title
+                        , 'isbn'      : issn
+                        }
+
+    keep_only_tags    = [dict(attrs={'class':['fotos', 'header_nota', 'nota']})]
+    remove_tags       = [dict(name=['meta','link','iframe','embed','object'])]
+    remove_attributes = ['lang']
+
+    feeds = [
+              (u'El Pais'       , u'http://elguardian.com.ar/RSS/el-pais.xml'       )
+             ,(u'Columnistas'   , u'http://elguardian.com.ar/RSS/columnistas.xml'   )
+             ,(u'Personajes'    , u'http://elguardian.com.ar/RSS/personajes.xml'    )
+             ,(u'Tinta roja'    , u'http://elguardian.com.ar/RSS/tinta-roja.xml'    )
+             ,(u'Yo fui'        , u'http://elguardian.com.ar/RSS/yo-fui.xml'        )
+             ,(u'Ciencia'       , u'http://elguardian.com.ar/RSS/ciencia.xml'       )
+             ,(u'Cronicas'      , u'http://elguardian.com.ar/RSS/cronicas.xml'      )
+             ,(u'Culturas'      , u'http://elguardian.com.ar/RSS/culturas.xml'      )
+             ,(u'DxT'           , u'http://elguardian.com.ar/RSS/dxt.xml'           )
+             ,(u'Fierros'       , u'http://elguardian.com.ar/RSS/fierros.xml'       )
+             ,(u'Frente fashion', u'http://elguardian.com.ar/RSS/frente-fashion.xml')
+             ,(u'Pan y vino'    , u'http://elguardian.com.ar/RSS/pan-y-vino.xml'    )
+             ,(u'Turismo'       , u'http://elguardian.com.ar/RSS/turismo.xml'       )
+            ]
+
+    def get_cover_url(self):
+        soup = self.index_to_soup('http://elguardian.com.ar/')
+        udata = soup.find('div', attrs={'class':'datosNumero'})
+        if udata:
+           sdata = udata.find('div')
+           if sdata:
+               stra = re.findall(r'\d+', self.tag_to_string(sdata))
+               self.conversion_options.update({'series_index':int(stra[1])})
+        unumero = soup.find('div', attrs={'class':'ultimoNumero'})
+        if unumero:
+           img = unumero.find('img', src=True)
+           if img:
+              return img['src']
+        return None
+
+    def preprocess_html(self, soup):
+        for item in soup.findAll(style=True):
+            del item['style']
+        for item in soup.findAll('a'):
+            limg = item.find('img')
+            if item.string is not None:
+               str = item.string
+               item.replaceWith(str)
+            else:
+               if limg:
+                  item.name = 'div'
+                  item.attrs = []
+               else:
+                   str = self.tag_to_string(item)
+                   item.replaceWith(str)
+        for item in soup.findAll('img'):
+            if not item.has_key('alt'):
+               item['alt'] = 'image'
+        return soup

recipes/eso_pl.recipe

@@ -0,0 +1,23 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+
+class ESO(BasicNewsRecipe):
+    title          = u'ESO PL'
+    __author__        = 'fenuks'
+    description   = u'ESO, Europejskie Obserwatorium Południowe, buduje i obsługuje najbardziej zaawansowane naziemne teleskopy astronomiczne na świecie'
+    category       = 'astronomy'
+    language       = 'pl'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    no_stylesheets = True
+    remove_empty_feeds = True
+    use_embedded_content = False
+    cover_url = 'https://twimg0-a.akamaihd.net/profile_images/1922519424/eso-twitter-logo.png'
+    keep_only_tags = [dict(attrs={'class':'subcl'})]
+    remove_tags = [dict(id='lang_row'), dict(attrs={'class':['pr_typeid', 'pr_news_feature_link', 'outreach_usage', 'hidden']})]
+    feeds          = [(u'Wiadomo\u015bci', u'http://www.eso.org/public/poland/news/feed/'), (u'Og\u0142oszenia', u'http://www.eso.org/public/poland/announcements/feed/'), (u'Zdj\u0119cie tygodnia', u'http://www.eso.org/public/poland/images/potw/feed/')]
+
+    def preprocess_html(self, soup):
+        for a in soup.findAll('a', href=True):
+            if a['href'].startswith('/'):
+                a['href'] = 'http://www.eso.org' + a['href']
+        return soup

recipes/informacje_usa.recipe

@@ -1,5 +1,4 @@
 from calibre.web.feeds.news import BasicNewsRecipe
-import re
 class Informacje_USA(BasicNewsRecipe):
     title          = u'Informacje USA'
     oldest_article = 7
@@ -8,11 +7,10 @@ class Informacje_USA(BasicNewsRecipe):
     description   = u'portal wiadomości amerykańskich'
     category       = 'news'
     language       = 'pl'
-    masthead_url= 'http://www.informacjeusa.com/wp-content/add_images/top_logo_5_2010.jpg'
+    cover_url='http://www.informacjeusa.com/wp-content/uploads/2013/01/V3BANNER420-90new.jpg'
-    cover_url='http://www.informacjeusa.com/wp-content/add_images/top_logo_5_2010.jpg'
     no_stylesheets = True
-    preprocess_regexps = [(re.compile(ur'<p>Zobacz:.*?</p>', re.DOTALL), lambda match: ''), (re.compile(ur'<p><a href=".*?Zobacz także:.*?</a></p>', re.DOTALL), lambda match: ''), (re.compile(ur'<p><p>Zobacz też:.*?</a></p>', re.DOTALL), lambda match: '')]
+    use_embedded_content = False
-    keep_only_tags=[dict(name='div', attrs={'class':'box box-single'})]
+    keep_only_tags=[dict(id='post-area')]
-    remove_tags_after= dict(attrs={'class':'tags'})
+    remove_tags_after= dict(id='content-area')
-    remove_tags= [dict(attrs={'class':['postmetadata', 'tags', 'banner']}), dict(name='a', attrs={'title':['Drukuj', u'Wyślij']})]
+    remove_tags= [dict(attrs={'class':['breadcrumb']}), dict(id=['social-box', 'social-box-vert'])]
     feeds          = [(u'Informacje', u'http://www.informacjeusa.com/feed/')]

recipes/kdefamily_pl.recipe

@@ -0,0 +1,14 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+
+class KDEFamilyPl(BasicNewsRecipe):
+    title          = u'KDEFamily.pl'
+    __author__        = 'fenuks'
+    description   = u'KDE w Polsce'
+    category       = 'open source, KDE'
+    language       = 'pl'
+    cover_url = 'http://www.mykde.home.pl/kdefamily/wp-content/uploads/2012/07/logotype-e1341585198616.jpg'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    no_stylesheets = True
+    use_embedded_content = True
+    feeds          = [(u'Wszystko', u'http://kdefamily.pl/feed/')]

recipes/kommersant.recipe

@@ -1,5 +1,5 @@
 __license__   = 'GPL v3'
-__copyright__ = '2010-2011, Darko Miletic <darko.miletic at gmail.com>'
+__copyright__ = '2010-2013, Darko Miletic <darko.miletic at gmail.com>'
 '''
 www.kommersant.ru
 '''
@@ -29,17 +29,20 @@ class Kommersant_ru(BasicNewsRecipe):
 							"""
 
     conversion_options = {
-                          'comment'          : description
+                          'comment'   : description
-                        , 'tags'             : category
+                        , 'tags'      : category
-                        , 'publisher'        : publisher
+                        , 'publisher' : publisher
-                        , 'language'         : language
+                        , 'language'  : language
                         }
 
     keep_only_tags = [dict(attrs={'class':['document','document_vvodka','document_text','document_authors vblock']})]
     remove_tags    = [dict(name=['iframe','object','link','img','base','meta'])]
 
-    feeds       = [(u'Articles', u'http://feeds.kommersant.ru/RSS_Export/RU/daily.xml')]
+    feeds       = [(u'Articles', u'http://dynamic.feedsportal.com/pf/438800/http://feeds.kommersant.ru/RSS_Export/RU/daily.xml')]
-
+    
+    def get_article_url(self, article):
+        return article.get('guid',  None)
+    
     def print_version(self, url):	    
         return url.replace('/doc-rss/','/Doc/') + '/Print'
 		

recipes/kurier_galicyjski.recipe

@@ -0,0 +1,56 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+from calibre.ebooks.BeautifulSoup import BeautifulSoup as bs
+class KurierGalicyjski(BasicNewsRecipe):
+    title          = u'Kurier Galicyjski'
+    __author__        = 'fenuks'
+    #description   = u''
+    category       = 'news'
+    language       = 'pl'
+    cover_url = 'http://www.duszki.pl/Kurier_galicyjski_bis2_small.gif'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    remove_empty_feeds = True
+    no_stylesheets = True
+    keep_only_tags = [dict(attrs={'class':'item-page'})]
+    remove_tags = [dict(attrs={'class':'pagenav'}), dict(attrs={'style':'border-top-width: thin; border-top-style: dashed; border-top-color: #CCC; border-bottom-width: thin; border-bottom-style: dashed; border-bottom-color: #CCC; padding-top:5px; padding-bottom:5px; text-align:right; margin-top:10px; height:20px;'})]
+    feeds          = [(u'Wydarzenia', u'http://kuriergalicyjski.com/index.php/wydarzenia?format=feed&type=atom'), (u'Publicystyka', u'http://kuriergalicyjski.com/index.php/niezwykle-historie?format=feed&type=atom'), (u'Reporta\u017ce', u'http://kuriergalicyjski.com/index.php/report?format=feed&type=atom'), (u'Rozmowy Kuriera', u'http://kuriergalicyjski.com/index.php/kuriera?format=feed&type=atom'), (u'Przegl\u0105d prasy', u'http://kuriergalicyjski.com/index.php/2012-01-05-14-08-55?format=feed&type=atom'), (u'Kultura', u'http://kuriergalicyjski.com/index.php/2011-12-02-14-26-39?format=feed&type=atom'), (u'Zabytki', u'http://kuriergalicyjski.com/index.php/2011-12-02-14-27-32?format=feed&type=atom'), (u'Polska-Ukraina', u'http://kuriergalicyjski.com/index.php/pol-ua?format=feed&type=atom'), (u'Polacy i Ukrai\u0144cy', u'http://kuriergalicyjski.com/index.php/polacy-i-ukr?format=feed&type=atom'), (u'Niezwyk\u0142e historie', u'http://kuriergalicyjski.com/index.php/niezwykle-historie?format=feed&type=atom'), (u'Polemiki', u'http://kuriergalicyjski.com/index.php/polemiki?format=feed&type=atom')]
+
+    def append_page(self, soup, appendtag):
+        pager = soup.find(id='article-index')
+        if pager:
+            pager = pager.findAll('a')[1:]
+        if pager:
+            for a in pager:
+                nexturl = 'http://www.kuriergalicyjski.com' + a['href']
+                soup2 = self.index_to_soup(nexturl)
+                pagetext = soup2.find(attrs={'class':'item-page'})
+                if pagetext.h2:
+                    pagetext.h2.extract()
+                r = pagetext.find(attrs={'class':'article-info'})
+                if r:
+                    r.extract()
+                pos = len(appendtag.contents)
+                appendtag.insert(pos, pagetext)
+                pos = len(appendtag.contents)
+            for r in appendtag.findAll(id='article-index'):
+                r.extract()
+            for r in appendtag.findAll(attrs={'class':'pagenavcounter'}):
+                r.extract()
+            for r in appendtag.findAll(attrs={'class':'pagination'}):
+                r.extract()
+            for r in appendtag.findAll(attrs={'class':'pagenav'}):
+                r.extract()
+            for r in appendtag.findAll(attrs={'style':'border-top-width: thin; border-top-style: dashed; border-top-color: #CCC; border-bottom-width: thin; border-bottom-style: dashed; border-bottom-color: #CCC; padding-top:5px; padding-bottom:5px; text-align:right; margin-top:10px; height:20px;'}):
+                r.extract()
+
+    def preprocess_html(self, soup):
+        self.append_page(soup, soup.body)
+        for r in soup.findAll(style=True):
+            del r['style']
+        for img in soup.findAll(attrs={'class':'easy_img_caption smartresize'}):
+            img.insert(len(img.contents)-1, bs('<br />'))
+            img.insert(len(img.contents), bs('<br /><br />'))
+        for a in soup.findAll('a', href=True):
+            if a['href'].startswith('/'):
+                a['href'] = 'http://kuriergalicyjski.com' + a['href']
+        return soup

recipes/le_monde_sub.recipe

@@ -1,166 +1,94 @@
-#!/usr/bin/env  python
+# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:fdm=marker:ai
-
+__author__    = 'Sylvain Durand <sylvain.durand@ponts.org>'
 __license__   = 'GPL v3'
-__copyright__ = '2012, 2013, Rémi Vanicat <vanicat at debian.org>'
-'''
-Lemonde.fr: Version abonnée
-'''
 
+import time
 
-import os, zipfile, re, time
+from calibre import strftime
-from urllib2 import HTTPError
-from calibre.constants import preferred_encoding
-
 from calibre.web.feeds.news import BasicNewsRecipe
-from calibre.ebooks.BeautifulSoup import BeautifulSoup
 from calibre.ptempfile import PersistentTemporaryFile
+from urllib2 import HTTPError
 
-class LeMondeAbonne(BasicNewsRecipe):
+class LeMonde(BasicNewsRecipe):
 
-    title                 = u'Le Monde: Édition abonnés'
+    title              = u'Le Monde: Édition abonnés'
-    __author__            = u'Rémi Vanicat'
+    __author__         = 'Sylvain Durand'
-    description           = u'Actualités'
+    description        = u'Disponible du lundi au samedi à partir de 14 heures environ, avec tous ses cahiers.'
-    category              = u'Actualités, France, Monde'
+    language           = 'fr'
-    publisher             = 'Le Monde'
+    encoding           = 'utf8'
-    language              = 'fr'
-    needs_subscription    = True
-    no_stylesheets        = True
-    smarten_punctuation   = True
-    remove_attributes     = [ 'border', 'cellspacing', 'display', 'align', 'cellpadding', 'colspan', 'valign', 'vscape', 'hspace', 'alt', 'width', 'height']
-    extra_css = ''' li{margin:6pt 0}
-                    ul{margin:0}
 
-                    div.photo img{max-width:100%; border:0px transparent solid;}
+    needs_subscription = True
-                    div.photo{font-family:inherit; color:#333; text-align:center;}
-                    div.photo p{text-align:justify;font-size:.9em; line-height:.9em;}
 
-                    @page{margin:10pt}
+    date_url           = 'http://www.lemonde.fr/journalelectronique/donnees/libre/%Y%m%d/index.html'
-                    .ar-txt {color:#000; text-align:justify;}
+    login_url          = 'http://www.lemonde.fr/web/journal_electronique/identification/1,56-0,45-0,0.html'
-                    h1{text-align:left; font-size:1.25em;}
+    journal_url        = 'http://www.lemonde.fr/journalelectronique/donnees/protege/%Y%m%d/%Y%m%d_ipad.xml'
+    masthead_url       = 'http://upload.wikimedia.org/wikipedia/fr/thumb/c/c5/Le_Monde_logo.svg/300px-Le_Monde_logo.svg.png'
+    couverture_url     = 'http://medias.lemonde.fr/abonnes/editionelectronique/%Y%m%d/html/data/img/%y%m%d01.jpg'
 
-                    .auteur{text-align:right; font-weight:bold}
+    extra_css = '''
-                    .feed{text-align:right; font-weight:bold}
+                img{max-width:100%}
-                    .po-ti2{font-weight:bold}
+                h1{font-size:1.2em !important; line-height:1.2em !important; }
-                    .fen-tt{font-weight:bold;font-size:1.1em}
+                h2{font-size:1em !important; line-height:1em !important; }
-    '''
+                h3{font-size:1em !important; text-transform:uppercase !important; color:#666;}
+                #photo{text-align:center !important; margin:10px 0 -8px;}
+                #lgd{font-size:1em !important; line-height:1em !important;  font-style:italic; color:#333;} '''
 
-    zipurl_format = 'http://medias.lemonde.fr/abonnes/editionelectronique/%Y%m%d/html/%y%m%d.zip'
+    keep_only_tags = [dict(name=['h1','h2','h3','div','txt'])]
-    coverurl_format = '/img/%y%m%d01.jpg'
-    path_format = "%y%m%d"
-    login_url = 'http://www.lemonde.fr/web/journal_electronique/identification/1,56-0,45-0,0.html'
 
-    keep_only_tags = [dict(name=['h1']), dict(name='div', attrs={ 'class': 'photo' }), dict(name='div', attrs={ 'class': 'po-ti2' }), dict(name='div', attrs={ 'class': 'ar-txt' }), dict(name='div', attrs={ 'class': 'po_rtcol' }) ]
+    def __init__(self, options, log, progress_reporter):
-
+        BasicNewsRecipe.__init__(self, options, log, progress_reporter)
-
+        br = BasicNewsRecipe.get_browser(self)
-    remove_tags = [ dict(name='div', attrs={ 'class': 'po-ti' }),dict(name='div', attrs={ 'class': 'po-copy' })]
+        second = time.time() + 24*60*60
-
+        for i in range(7):
-    article_id_pattern = re.compile("[0-9]+\\.html")
+            self.date = time.gmtime(second)
-    article_url_format = 'http://www.lemonde.fr/journalelectronique/donnees/protege/%Y%m%d/html/'
+            try:
+                br.open(time.strftime(self.date_url,self.date))
+                break
+            except HTTPError:
+                second -= 24*60*60
+        self.timefmt = strftime(u" %A %d %B %Y", self.date).replace(u' 0', u' ')
 
     def get_browser(self):
         br = BasicNewsRecipe.get_browser(self)
-        if self.username is not None and self.password is not None:
+        br.open(self.login_url)
-            br.open(self.login_url)
+        br.select_form(nr=0)
-            br.select_form(nr=0)
+        br['login']    = self.username
-            br['login']    = self.username
+        br['password'] = self.password
-            br['password'] = self.password
+        br.submit()
-            br.submit()
         return br
 
-    decalage = 24 * 60 * 60     # today Monde has tomorow date
-
     def get_cover_url(self):
-        url = time.strftime(self.coverurl_format, self.ltime)
+        url = time.strftime(self.couverture_url,self.date)
-        return self.articles_path + url
+        return url
 
     def parse_index(self):
-        browser = self.get_browser()
+        url = time.strftime(self.journal_url,self.date)
-
+        soup = self.index_to_soup(url).sommaire
-        second = time.time()
+        sections = []
-        second += self.decalage
+        for sec in soup.findAll("section"):
-
-        for i in range(7):
-            self.ltime = time.gmtime(second)
-            self.timefmt=time.strftime(" %A %d %B %Y",self.ltime).decode(preferred_encoding)
-            url = time.strftime(self.zipurl_format,self.ltime)
-            try:
-                response = browser.open(url)
-                continue
-            except HTTPError:
-                second -= 24*60*60
-
-        tmp = PersistentTemporaryFile(suffix='.zip')
-        self.report_progress(0.1,_('downloading zip file'))
-        tmp.write(response.read())
-        tmp.close()
-
-        zfile = zipfile.ZipFile(tmp.name, 'r')
-        self.report_progress(0.1,_('extracting zip file'))
-
-        zfile.extractall(self.output_dir)
-        zfile.close()
-
-        path = os.path.join(self.output_dir, time.strftime(self.path_format, self.ltime), "data")
-
-        self.articles_path = path
-
-        files = os.listdir(path)
-
-        nb_index_files = len([ name for name in files if re.match("frame_gauche_[0-9]+.html", name) ])
-
-        flux = []
-
-        article_url = time.strftime(self.article_url_format, self.ltime)
-
-        for i in range(nb_index_files):
-            filename = os.path.join(path, "selection_%d.html" % (i + 1))
-            tmp = open(filename,'r')
-            soup=BeautifulSoup(tmp,convertEntities=BeautifulSoup.HTML_ENTITIES)
-            title=soup.find('span').contents[0]
-            if title=="Une":
-                title="À la une"
-            if title=="Evenement":
-                title="L'événement"
-            if title=="Planete":
-                title="Planète"
-            if title=="Economie - Entreprises":
-                title="Économie"
-            if title=="L'Oeil du Monde":
-                title="L'œil du Monde"
-            if title=="Enquete":
-                title="Enquête"
-            if title=="Editorial - Analyses":
-                title="Analyses"
-            if title=="Le Monde Economie":
-                title="Économie"
-            if title=="Le Monde Culture et idées":
-                title="Idées"
-            if title=="Le Monde Géo et politique":
-                title="Géopolitique"
-            tmp.close()
-
-            filename = os.path.join(path, "frame_gauche_%d.html" % (i + 1))
-            tmp = open(filename,'r')
-            soup = BeautifulSoup(tmp)
             articles = []
-            for link in soup.findAll("a"):
+            if sec['cahier'] != "Le Monde":
-                article_file = link['href']
+                for col in sec.findAll("fnts"):
-                article_id=self.article_id_pattern.search(article_file).group()
+                    col.extract()
-                article = {
+            if sec['cahier']=="Le Monde Magazine":
-                    'title': link.contents[0],
+                continue
-                    'url': article_url + article_id,
+            for art in sec.findAll("art"):
-                    'description': '',
+                if art.txt.string and art.ttr.string:
-                    'content': ''
+                    if art.find(['url']):
-                    }
+                        art.insert(6,'<div id="photo"><img src="'+art.find(['url']).string+'" /></div>')
-                articles.append(article)
+                    if art.find(['lgd']) and art.find(['lgd']).string:
-            tmp.close()
+                        art.insert(7,'<div id="lgd">'+art.find(['lgd']).string+'</div>')
+                    article = "<html><head></head><body>"+unicode(art)+"</body></html>"
+                    article = article.replace('<![CDATA[','').replace(']]>','').replace(' oC ','°C ')
+                    article = article.replace('srttr>','h3>').replace('ssttr>','h2>').replace('ttr>','h1>')
+                    f = PersistentTemporaryFile()
+                    f.write(article)
+                    articles.append({'title':art.ttr.string,'url':"file:///"+f.name})
+            sections.append((sec['nom'], articles))
+        return sections
 
-            flux.append((title, articles))
+    def preprocess_html(self, soup):
+        for lgd in soup.findAll(id="lgd"):
+            lgd.contents[-1].extract()
+        return soup
 
-        return flux
-
-
-
-# Local Variables:
-# mode: python
-# End:

recipes/mlody_technik_pl.recipe

@@ -1,5 +1,5 @@
 # vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:ai
-
+import re
 from calibre.web.feeds.news import BasicNewsRecipe
 class Mlody_technik(BasicNewsRecipe):
     title          = u'Młody technik'
@@ -9,7 +9,19 @@ class Mlody_technik(BasicNewsRecipe):
     language       = 'pl'
     cover_url='http://science-everywhere.pl/wp-content/uploads/2011/10/mt12.jpg'
     no_stylesheets = True
+    preprocess_regexps = [(re.compile(r"<h4>Podobne</h4>", re.IGNORECASE), lambda m: '')]
     oldest_article = 7
     max_articles_per_feed = 100
-    #keep_only_tags=[dict(id='container')]
+    remove_empty_feeds = True
-    feeds          = [(u'Artyku\u0142y', u'http://www.mt.com.pl/feed')]
+    use_embedded_content = False
+    keep_only_tags = [dict(id='content')]
+    remove_tags = [dict(attrs={'class':'st-related-posts'})]
+    remove_tags_after = dict(attrs={'class':'entry-content clearfix'})
+    feeds          = [(u'Wszystko', u'http://www.mt.com.pl/feed'), 
+		(u'MT NEWS 24/7', u'http://www.mt.com.pl/kategoria/mt-newsy-24-7/feed'),
+		(u'Info zoom', u'http://www.mt.com.pl/kategoria/info-zoom/feed'),
+		(u'm.technik', u'http://www.mt.com.pl/kategoria/m-technik/feed'),
+		(u'Szkoła', u'http://www.mt.com.pl/kategoria/szkola-2/feed'),
+		(u'Na Warsztacie', u'http://www.mt.com.pl/kategoria/na-warsztacie/feed'),
+		(u'Z pasji do...', u'http://www.mt.com.pl/kategoria/z-pasji-do/feed'),
+		(u'MT testuje', u'http://www.mt.com.pl/kategoria/mt-testuje/feed')]

recipes/more_intelligent_life.recipe

@@ -0,0 +1,67 @@
+__license__   = 'GPL v3'
+__copyright__ = '2013, Darko Miletic <darko.miletic at gmail.com>'
+'''
+moreintelligentlife.com
+'''
+
+from calibre.web.feeds.news import BasicNewsRecipe
+
+class MoreIntelligentLife(BasicNewsRecipe):
+    title                 = 'More Intelligent Life'
+    __author__            = 'Darko Miletic'
+    description           = "More Intelligent Life (moreintelligentlife.com) is the online version of Intelligent Life, a lifestyle and culture magazine from The Economist. The website offers not only content from the print edition, trickled out over the course of its shelf-life, but also the Editors' Blog, which carries daily posts from the editorial team-quickfire observations and opinions that allow readers to eavesdrop on the conversation in the office."
+    publisher             = 'The Economist Newspaper ltd'
+    category              = 'arts,lifestyle,intelligent life,the economist,ideas,style,culture'
+    oldest_article        = 60
+    max_articles_per_feed = 200
+    no_stylesheets        = True
+    encoding              = 'utf8'
+    use_embedded_content  = False
+    language              = 'en'
+    remove_empty_feeds    = True
+    publication_type      = 'website'
+    extra_css             = """
+                               body{font-family: Arial,"Helvetica neue","Bitstream Vera Sans",sans-serif}
+                               img{margin-bottom: 0.4em; display:block}
+                            """
+
+    conversion_options = {
+                          'comment'   : description
+                        , 'tags'      : category
+                        , 'publisher' : publisher
+                        , 'language'  : language
+                        }
+
+    keep_only_tags    = [dict(attrs={'class':'node'})]
+    remove_tags_after = dict(attrs={'class':'tags'})
+    remove_tags       = [dict(name=['meta','link','iframe','embed','object'])]
+    remove_attributes = ['lang']
+
+    feeds = [(u'Articles', u'http://feeds.feedburner.com/MoreintelligentlifeTotal')]
+
+    def get_cover_url(self):
+        soup = self.index_to_soup('http://moreintelligentlife.com/')
+        for image in soup.findAll('img', src=True):
+           if image['src'].startswith('http://moreintelligentlife.com/files/covers/current_issue_'):
+              return image['src']
+        return None
+
+    def preprocess_html(self, soup):
+        for item in soup.findAll(style=True):
+            del item['style']
+        for item in soup.findAll('a'):
+            limg = item.find('img')
+            if item.string is not None:
+               str = item.string
+               item.replaceWith(str)
+            else:
+               if limg:
+                  item.name = 'div'
+                  item.attrs = []
+               else:
+                   str = self.tag_to_string(item)
+                   item.replaceWith(str)
+        for item in soup.findAll('img'):
+            if not item.has_key('alt'):
+               item['alt'] = 'image'
+        return soup

recipes/nauka_w_polsce.recipe

@@ -0,0 +1,47 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+import re
+class NaukawPolsce(BasicNewsRecipe):
+    title          = u'Nauka w Polsce'
+    __author__        = 'fenuks'
+    description   = u'Serwis Nauka w Polsce ma za zadanie popularyzację polskiej nauki. Można na nim znaleźć wiadomości takie jak:  osiągnięcia polskich naukowców, wydarzenia na polskich uczelniach, osiągnięcia studentów, konkursy dla badaczy, staże i stypendia naukowe, wydarzenia w polskiej nauce, kalendarium wydarzeń w nauce, materiały wideo o nauce.'
+    category       = 'science'
+    language       = 'pl'
+    cover_url = 'http://www.naukawpolsce.pap.pl/Themes/Pap/images/logo-pl.gif'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    no_stylesheets = True
+    remove_empty_feeds = True
+    index = 'http://www.naukawpolsce.pl'
+    keep_only_tags = [dict(name='div', attrs={'class':'margines wiadomosc'})]
+    remove_tags = [dict(name='div', attrs={'class':'tagi'})]
+
+    def find_articles(self, url):
+        articles = []
+        soup=self.index_to_soup(url)
+        for i in soup.findAll(name='div', attrs={'class':'aktualnosci-margines lista-depesz information-content'}):
+            title = i.h1.a.string
+            url = self.index + i.h1.a['href']
+            date = '' #i.span.string
+            articles.append({'title' : title,
+                   'url'   : url,
+                   'date'  : date,
+                   'description' : ''
+                    })
+        return articles
+
+    def parse_index(self):
+         feeds = []
+         feeds.append((u"Historia i kultura", self.find_articles('http://www.naukawpolsce.pl/historia-i-kultura/')))
+         feeds.append((u"Kosmos", self.find_articles('http://www.naukawpolsce.pl/kosmos/')))
+         feeds.append((u"Przyroda", self.find_articles('http://www.naukawpolsce.pl/przyroda/')))
+         feeds.append((u"Społeczeństwo", self.find_articles('http://www.naukawpolsce.pl/spoleczenstwo/')))
+         feeds.append((u"Technologie", self.find_articles('http://www.naukawpolsce.pl/technologie/')))
+         feeds.append((u"Uczelnie", self.find_articles('http://www.naukawpolsce.pl/uczelnie/')))
+         feeds.append((u"Nauki medyczne", self.find_articles('http://www.naukawpolsce.pl/zdrowie/')))
+
+         return feeds
+
+    def preprocess_html(self, soup):
+        for p in soup.findAll(name='p', text=re.compile(' ')):
+            p.extract()
+        return soup

recipes/navegalo.recipe

@@ -0,0 +1,40 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+
+class AdvancedUserRecipe1360354988(BasicNewsRecipe):
+    title          = u'Navegalo.com'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    auto_cleanup = True
+
+from calibre.web.feeds.news import BasicNewsRecipe
+
+class navegalonews(BasicNewsRecipe):
+    __author__     = 'Douglas Delgado'
+    title          = u'Navegalo.com'
+    publisher      = 'Navegalo.com'
+    description    = 'Noticias actualizadas por Navegalo.com. Recipe creado por Douglas Delgado (doudelgado@gmail.com) para su uso con Calibre'
+    category       = 'Spanish, Entertainment'
+    masthead_url   = 'https://encrypted-tbn0.gstatic.com/images?q=tbn:ANd9GcQZhML5lwsdss6FFF7CFR0Sf-Ln052Zmhs1TlIOcAL8JWN8a-dPlA'
+
+    oldest_article = 2
+    delay          = 1
+    max_articles_per_feed = 100
+    auto_cleanup = True
+    encoding       = 'utf-8'
+    language       = 'es_CR'
+    use_embedded_content  = False
+    remove_empty_feeds    = True
+    remove_javascript = True
+    no_stylesheets = True
+
+
+    feeds          = [(u'Nacionales', u'http://www.navegalo.com/es/noticias/noticias/noticias-nacionales?format=feed&type=rss'), (u'Internacionales', u'http://direct.navegalo.com/es/noticias/noticias/noticias-internacionales?format=feed&type=rss'), (u'Deportes', u'http://direct.navegalo.com/es/noticias/noticias/deportes-nacionales?format=feed&type=rss'), (u'Solo futbol', u'http://www.navegalo.com/es/noticias/noticias/solo-futbol?format=feed&type=rss'), (u'Entretenimiento', u'http://www.navegalo.com/es/noticias/noticias/entretenimiento?format=feed&type=rss'), (u'Solo para ellas', u'http://www.navegalo.com/es/noticias/noticias/solo-para-ellas?format=feed&type=rss'), (u'Infiltrados', u'http://direct.navegalo.com/es/noticias/noticias/infiltrados?format=feed&type=rss'), (u'Mano a mano', u'http://direct.navegalo.com/es/noticias/noticias/mano-a-mano?format=feed&type=rss')]
+
+
+
+
+    extra_css = '''
+                    h1{font-family:Arial,Helvetica,sans-serif; font-weight:bold;font-size:30px;}
+                    h2{font-family:Arial,Helvetica,sans-serif; font-weight:normal; font-style:italic; font-size:18px;}
+                '''
+

recipes/new_yorker.recipe

@@ -1,5 +1,5 @@
 __license__   = 'GPL v3'
-__copyright__ = '2008-2011, Darko Miletic <darko.miletic at gmail.com>'
+__copyright__ = '2008-2013, Darko Miletic <darko.miletic at gmail.com>'
 '''
 newyorker.com
 '''
@@ -44,20 +44,18 @@ class NewYorker(BasicNewsRecipe):
                         , 'language'  : language
                         }
 
-    keep_only_tags = [
+    keep_only_tags = [dict(name='div', attrs={'id':'pagebody'})]
-                        dict(name='div', attrs={'class':'headers'})
-                       ,dict(name='div', attrs={'id':['articleheads','items-container','articleRail','articletext','photocredits']})
-                     ]
     remove_tags    = [
                          dict(name=['meta','iframe','base','link','embed','object'])
-                        ,dict(attrs={'class':['utils','socialUtils','articleRailLinks','icons'] })
+                        ,dict(attrs={'class':['utils','socialUtils','articleRailLinks','icons','social-utils-top','entry-keywords','entry-categories','utilsPrintEmail'] })
                         ,dict(attrs={'id':['show-header','show-footer'] })
                      ]
+    remove_tags_after = dict(attrs={'class':'entry-content'}) 
     remove_attributes = ['lang']
     feeds             = [(u'The New Yorker', u'http://www.newyorker.com/services/mrss/feeds/everything.xml')]
 
     def print_version(self, url):
-        return url + '?printable=true'
+        return url + '?printable=true&currentPage=all'
 
     def image_url_processor(self, baseurl, url):
         return url.strip()

recipes/nzherald.recipe

@@ -25,7 +25,7 @@ class NewZealandHerald(BasicNewsRecipe):
                 'http://rss.nzherald.co.nz/rss/xml/nzhrsscid_000000466.xml'),
             ('Life & Style',
                 'http://rss.nzherald.co.nz/rss/xml/nzhrsscid_000000006.xml'),
-            ('Technology'
+            ('Technology',
                 'http://rss.nzherald.co.nz/rss/xml/nzhrsscid_000000005.xml'),
             ('Sport',
                 'http://rss.nzherald.co.nz/rss/xml/nzhrsscid_000000004.xml'),

recipes/osworld_pl.recipe

@@ -0,0 +1,33 @@
+from calibre.web.feeds.news import BasicNewsRecipe
+class OSWorld(BasicNewsRecipe):
+    title          = u'OSWorld.pl'
+    __author__        = 'fenuks'
+    description   = u'OSWorld.pl to serwis internetowy, dzięki któremu poznasz czym naprawdę jest Open Source. Serwis poświęcony jest wolnemu oprogramowaniu jak linux mint, centos czy ubunty. Znajdziecie u nasz artykuły, unity oraz informacje o certyfikatach CACert. OSWorld to mały świat wielkich systemów!'
+    category       = 'OS, IT, open source, Linux'
+    language       = 'pl'
+    cover_url = 'http://osworld.pl/wp-content/uploads/osworld-kwadrat-128x111.png'
+    oldest_article = 7
+    max_articles_per_feed = 100
+    no_stylesheets = True
+    remove_empty_feeds = True
+    use_embedded_content = False
+    keep_only_tags = [dict(id=['dzial', 'posts'])]
+    remove_tags = [dict(attrs={'class':'post-comments'})]
+    remove_tags_after = dict(attrs={'class':'entry clr'})
+    feeds          = [(u'Artyku\u0142y', u'http://osworld.pl/category/artykuly/feed/'), (u'Nowe wersje', u'http://osworld.pl/category/nowe-wersje/feed/')]
+
+    def append_page(self, soup, appendtag):
+        tag = appendtag.find(attrs={'id':'paginacja'})
+        if tag:
+            for nexturl in tag.findAll('a'):
+                soup2 = self.index_to_soup(nexturl['href'])
+                pagetext = soup2.find(attrs={'class':'entry clr'})
+                pos = len(appendtag.contents)
+                appendtag.insert(pos, pagetext)
+            for r in appendtag.findAll(attrs={'id':'paginacja'}):
+                r.extract()
+
+
+    def preprocess_html(self, soup):
+         self.append_page(soup, soup.body)
+         return soup

recipes/pc_lab.recipe

@@ -1,5 +1,4 @@
 #!/usr/bin/env  python
-
 from calibre.web.feeds.recipes import BasicNewsRecipe
 
 class PCLab(BasicNewsRecipe):
@@ -8,12 +7,13 @@ class PCLab(BasicNewsRecipe):
     __author__            = 'ravcio - rlelusz[at]gmail.com'
     description           = u"Articles from PC Lab website"
     language              = 'pl'
-    oldest_article        = 30.0
+    oldest_article        = 30
     max_articles_per_feed = 100
     recursions            = 0
     encoding              = 'iso-8859-2'
     no_stylesheets        = True
     remove_javascript     = True
+    remove_empty_feeds = True
     use_embedded_content  = False
 
     keep_only_tags = [
@@ -21,50 +21,54 @@ class PCLab(BasicNewsRecipe):
                      ]
 
     remove_tags = [
-            dict(name='div', attrs={'class':['chapters']})
+            dict(name='div', attrs={'class':['toc first', 'toc', 'tags', 'recommendedarticles', 'name', 'zumi', 'chapters']})
-            ,dict(name='div', attrs={'id':['script_bxad_slot_display_list_bxad_slot']})
                   ]
 
-    remove_tags_after = [
-            dict(name='div', attrs={'class':['navigation']})
-                ]
-
     #links to RSS feeds
-    feeds = [ ('PCLab', u'http://pclab.pl/xml/artykuly.xml') ]
+    feeds = [
+             (u'Aktualności', 'http://pclab.pl/xml/aktualnosci.xml'),
+             (u'Artykuły', u'http://pclab.pl/xml/artykuly.xml'),
+             (u'Poradniki', 'http://pclab.pl/xml/poradniki.xml')
+             ]
 
     #load second and subsequent page content
     # in: soup - full page with 'next' button
     # out: appendtag - tag to which new page is to be added
     def append_page(self, soup, appendtag):
         # find the 'Next' button
-        pager = soup.find('div', attrs={'class':'next'})
+        pager = soup.find('div', attrs={'class':'navigation'})
-
         if pager:
+            a = pager.find('a')
+            if 'news' in a['href']:
+                pager = None
+            else:
+                pager = pager.find('div', attrs={'class':'next'})
+
+        while pager:
             #search for 'a' element with link to next page (exit if not found)
             a = pager.find('a')
-            if a:
+            nexturl = a['href']
-                nexturl = a['href']
+            soup2 = self.index_to_soup('http://pclab.pl' + nexturl)
+            pager = soup2.find('div', attrs={'class':'next'})
+            pagetext = soup2.find('div', attrs={'class':'substance'})
+            pagetext = pagetext.find('div', attrs={'class':'data'})
 
-                soup2 = self.index_to_soup('http://pclab.pl/' + nexturl)
+            pos = len(appendtag.contents)
-
+            appendtag.insert(pos, pagetext)
-                pagetext_substance = soup2.find('div', attrs={'class':'substance'})
+            pos = len(appendtag.contents)
-                pagetext = pagetext_substance.find('div', attrs={'class':'data'})
-                pagetext.extract()
-
-                pos = len(appendtag.contents)
-                appendtag.insert(pos, pagetext)
-                pos = len(appendtag.contents)
-
-                self.append_page(soup2, appendtag)
 
+        pager = soup.find('div', attrs={'class':'navigation'})
+        if pager:
+            pager.extract()
 
     def preprocess_html(self, soup):
-
         # soup.body contains no title and no navigator, they are in soup
         self.append_page(soup, soup.body)
-
+        for link in soup.findAll('a'):
+            href = link.get('href', None)
+            if href and href.startswith('/'):
+                link['href'] = 'http://pclab.pl' + href
         # finally remove some tags
-        tags = soup.findAll('div',attrs={'class':['tags', 'index', 'script_bxad_slot_display_list_bxad_slot', 'index first', 'zumi', 'navigation']})
+        #for r in soup.findAll('div', attrs={'class':['tags', 'index', 'script_bxad_slot_display_list_bxad_slot', 'index first', 'zumi', 'navigation']})
-        [tag.extract() for tag in tags]
 
         return soup

recipes/pnn.recipe

@@ -0,0 +1,55 @@
+from calibre.web.feeds.recipes import BasicNewsRecipe
+
+'''Calibre recipe to convert the RSS feeds of the PNN to an ebook.'''
+
+class SportsIllustratedRecipe(BasicNewsRecipe) :
+    __author__    = 'n.kucklaender'
+    __copyright__ = 'a.peter'
+    __license__   = 'GPL v3'
+    language      = 'de'
+    description   = 'PNN RSS'
+    version       = 1
+    title         = u'PNN'
+    timefmt       = ' [%d.%m.%Y]'
+
+    oldest_article = 7.0
+    no_stylesheets = True
+    remove_javascript = True
+    use_embedded_content = False
+    publication_type = 'newspaper'
+
+    remove_empty_feeds = True
+    remove_tags = [dict(attrs={'class':['um-weather um-header-weather','um-has-sub um-mainnav','um-box','ts-products','um-meta-nav','um-box um-last','um-footer','um-footer-links','share hidden','um-buttons']}),dict(id=['dinsContainer'])]
+   # remove_tags_before = [dict(name='div', attrs={'class':'um-first'})]
+   # remove_tags_after = [dict(name='div', attrs={'class':'um-metabar'})]
+
+    feeds = [(u'Titelseite', u'http://www.pnn.de/rss.xml'),
+             (u'Dritte Seite', u'http://www.pnn.de/dritte-seite/rss.xml'),
+             (u'Politik', u'http://www.pnn.de/politik/rss.xml'),
+             (u'Meinung', u'http://www.pnn.de/meinung/rss.xml'),
+             (u'Potsdam', u'http://www.pnn.de/potsdam/rss.xml'),
+             (u'Havel-Spree', u'http://www.pnn.de/havel-spree/rss.xml'),
+             (u'Potsdam-Mittelmark', u'http://www.pnn.de/pm/rss.xml'),
+             (u'Berlin-Brandenburg', u'http://www.pnn.de/brandenburg-berlin/rss.xml'),
+             (u'Wirtschaft', u'http://www.pnn.de/wirtschaft/rss.xml'),
+             (u'Sport', u'http://www.pnn.de/sport/rss.xml'),
+             (u'Regionalsport', u'http://www.pnn.de/regionalsport/rss.xml'),
+             (u'Kultur', u'http://www.pnn.de/kultur/rss.xml'),
+             (u'Potsdam-Kultur', u'http://www.pnn.de/potsdam-kultur/rss.xml'),
+             (u'Wissen', u'http://www.pnn.de/wissen/rss.xml'),
+             (u'Medien', u'http://www.pnn.de/medien/rss.xml'),
+             (u'Weltspiegel', u'http://www.pnn.de/weltspiegel/rss.xml'),
+             (u'Wissenschaft', u'http://www.pnn.de/campus/rss.xml'),
+             (u'Mobil', u'http://www.pnn.de/mobil/rss.xml'),
+             (u'Reise', u'http://www.pnn.de/reise/rss.xml'),
+             (u'Ratgeber', u'http://www.pnn.de/ratgeber/rss.xml'),
+             (u'Fragen des Tages', u'http://www.pnn.de/fragen-des-tages/rss.xml'),
+           # (u'Potsdam bin ich', u'http://www.pnn.de/potsdam-bin-ich/rss.xml'),
+             (u'Leserbriefe', u'http://www.pnn.de/leserbriefe/rss.xml')]
+
+    def get_masthead_url(self):
+        return 'http://www.pnn.de/app/base/img/pnn_logo.png'
+
+    def print_version(self, url):
+        return url.replace('.html', ',view,printVersion.html')
+

recipes/spiders_web_pl.recipe

@@ -5,11 +5,14 @@ class SpidersWeb(BasicNewsRecipe):
     oldest_article = 7
     __author__        = 'fenuks'
     description   = u''
-    cover_url      = 'http://www.spidersweb.pl/wp-content/themes/spiderweb/img/Logo.jpg'
+    cover_url      = 'http://www.spidersweb.pl/wp-content/themes/new_sw/images/spidersweb.png'
     category       = 'IT, WEB'
     language       = 'pl'
     no_stylesheers=True
+    remove_javascript = True
+    use_embedded_content = False
     max_articles_per_feed = 100
-    keep_only_tags=[dict(id='Post')]
+    keep_only_tags=[dict(id='start')]
-    remove_tags=[dict(name='div', attrs={'class':['Comments', 'Shows', 'Post-Tags']}), dict(id='Author-Column')]
+    remove_tags_after = dict(attrs={'class':'padding20'})
+    remove_tags=[dict(name='div', attrs={'class':['padding border-bottom', 'padding20', 'padding border-top']})]
     feeds          = [(u'Wpisy', u'http://www.spidersweb.pl/feed')]

recipes/ubuntu_pomoc_org.recipe

@@ -0,0 +1,22 @@
+import re
+from calibre.web.feeds.news import BasicNewsRecipe
+class UbuntuPomoc(BasicNewsRecipe):
+    title          = u'Ubuntu-pomoc.org'
+    __author__        = 'fenuks'
+    description   = u'Strona poświęcona systemowi Ubuntu Linux. Znajdziesz tutaj przydatne i sprawdzone poradniki oraz sposoby rozwiązywania wielu popularnych problemów. Ten blog rozwiąże każdy Twój problem - jeśli nie teraz, to wkrótce! :)'
+    category       = 'Linux, Ubuntu, open source'
+    language       = 'pl'
+    cover_url = 'http://www.ubuntu-pomoc.org/grafika/ubuntupomoc.png'
+    preprocess_regexps = [(re.compile(r'<div class="ciekawostka">.+', re.IGNORECASE|re.DOTALL), lambda m: '')]
+    oldest_article = 7
+    max_articles_per_feed = 100
+    no_stylesheets = True
+    remove_javascript = True
+    remove_empty_feeds = True
+    use_embedded_content = False
+    remove_attrs = ['style']
+    keep_only_tags = [dict(attrs={'class':'post'})]
+    remove_tags_after = dict(attrs={'class':'underEntry'})
+    remove_tags = [dict(attrs={'class':['underPostTitle', 'yarpp-related', 'underEntry', 'social', 'tags', 'commentlist', 'youtube_sc']}), dict(id=['wp_rp_first', 'commentReply'])]
+    feeds          = [(u'Ca\u0142o\u015b\u0107', u'http://feeds.feedburner.com/Ubuntu-Pomoc'),
+                     (u'Gry', u'http://feeds.feedburner.com/GryUbuntu-pomoc')]

recipes/volksrant.recipe

@@ -41,17 +41,9 @@ class AdvancedUserRecipe1249039563(BasicNewsRecipe):
    #######################################################################################################
     temp_files = []
     articles_are_obfuscated = True
-    use_javascript_to_login = True
-
-    def javascript_login(self, br, username, password):
-        'Volksrant wants the user to explicitly allow cookies'
-        if not br.visit('http://www.volkskrant.nl'):
-            raise Exception('Failed to connect to volksrant website')
-        br.click('#pop_cookie_text a[onclick]', wait_for_load=True, timeout=120)
 
     def get_obfuscated_article(self, url):
         br = self.browser.clone_browser()
-        print 'THE CURRENT URL IS: ', url
         br.open(url)
         year = date.today().year
 

recipes/wprost.recipe

@@ -10,89 +10,89 @@ from calibre.web.feeds.news import BasicNewsRecipe
 import re
 
 class Wprost(BasicNewsRecipe):
-        EDITION = 0
+    EDITION = 0
-        FIND_LAST_FULL_ISSUE = True
+    FIND_LAST_FULL_ISSUE = True
-        EXCLUDE_LOCKED = True
+    EXCLUDE_LOCKED = True
-        ICO_BLOCKED = 'http://www.wprost.pl/G/layout2/ico_blocked.png'
+    ICO_BLOCKED = 'http://www.wprost.pl/G/layout2/ico_blocked.png'
+    title = u'Wprost'
+    __author__ = 'matek09'
+    description = 'Weekly magazine'
+    encoding = 'ISO-8859-2'
+    no_stylesheets = True
+    language = 'pl'
+    remove_javascript = True
+    recursions = 0  
+    remove_tags_before = dict(dict(name = 'div', attrs = {'id' : 'print-layer'}))
+    remove_tags_after = dict(dict(name = 'div', attrs = {'id' : 'print-layer'}))
+    '''
+    keep_only_tags =[]
+    keep_only_tags.append(dict(name = 'table', attrs = {'id' : 'title-table'}))
+    keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'div-header'}))
+    keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'div-content'}))
+    keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'def element-autor'}))
+    '''
 
-        title = u'Wprost'
+    preprocess_regexps = [(re.compile(r'style="display: none;"'), lambda match: ''),
-        __author__ = 'matek09'
-        description = 'Weekly magazine'
-        encoding = 'ISO-8859-2'
-        no_stylesheets = True
-        language = 'pl'
-        remove_javascript = True
-	recursions = 0	
-
-        remove_tags_before = dict(dict(name = 'div', attrs = {'id' : 'print-layer'}))
-        remove_tags_after = dict(dict(name = 'div', attrs = {'id' : 'print-layer'}))
-
-        '''keep_only_tags =[]
-        keep_only_tags.append(dict(name = 'table', attrs = {'id' : 'title-table'}))
-        keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'div-header'}))
-        keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'div-content'}))
-        keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'def element-autor'}))'''
-
-        preprocess_regexps = [(re.compile(r'style="display: none;"'), lambda match: ''),
         (re.compile(r'display: block;'), lambda match: ''),
         (re.compile(r'\<td\>\<tr\>\<\/table\>'), lambda match: ''),
         (re.compile(r'\<table .*?\>'), lambda match: ''),
         (re.compile(r'\<tr>'), lambda match: ''),
         (re.compile(r'\<td .*?\>'), lambda match: ''),
-	(re.compile(r'\<div id="footer"\>.*?\</footer\>'), lambda match: '')]
+        (re.compile(r'\<div id="footer"\>.*?\</footer\>'), lambda match: '')]
 
-        remove_tags =[]
+    remove_tags =[]
-        remove_tags.append(dict(name = 'div', attrs = {'class' : 'def element-date'}))
+    remove_tags.append(dict(name = 'div', attrs = {'class' : 'def element-date'}))
-        remove_tags.append(dict(name = 'div', attrs = {'class' : 'def silver'}))
+    remove_tags.append(dict(name = 'div', attrs = {'class' : 'def silver'}))
-        remove_tags.append(dict(name = 'div', attrs = {'id' : 'content-main-column-right'}))
+    remove_tags.append(dict(name = 'div', attrs = {'id' : 'content-main-column-right'}))
 
 
-        extra_css = '''
+    extra_css = '''.div-header {font-size: x-small; font-weight: bold}'''
-                                        .div-header {font-size: x-small; font-weight: bold}
+    #h2 {font-size: x-large; font-weight: bold}
-                                        '''
+    def is_blocked(self, a):
-#h2 {font-size: x-large; font-weight: bold}
+        if a.findNextSibling('img') is None:
-        def is_blocked(self, a):
+            return False
-                if a.findNextSibling('img') is None:
+        else:
-                        return False
+            return True
-                else:
-                        return True
 
 
+    def find_last_issue(self):
+        soup = self.index_to_soup('http://www.wprost.pl/archiwum/')
+        a = 0
+        if self.FIND_LAST_FULL_ISSUE:
+            ico_blocked = soup.findAll('img', attrs={'src' : self.ICO_BLOCKED})
+            a = ico_blocked[-1].findNext('a', attrs={'title' : re.compile(r'Spis *', re.IGNORECASE | re.DOTALL)})
+        else:
+            a = soup.find('a', attrs={'title' : re.compile(r'Spis *', re.IGNORECASE | re.DOTALL)})
+        self.EDITION = a['href'].replace('/tygodnik/?I=', '')
+        self.EDITION_SHORT = a['href'].replace('/tygodnik/?I=15', '')
+        self.cover_url = a.img['src']
 
-        def find_last_issue(self):
+    def parse_index(self):
-                soup = self.index_to_soup('http://www.wprost.pl/archiwum/')
+        self.find_last_issue()
-                a = 0
+        soup = self.index_to_soup('http://www.wprost.pl/tygodnik/?I=' + self.EDITION)
-                if self.FIND_LAST_FULL_ISSUE:
+        feeds = []
-                        ico_blocked = soup.findAll('img', attrs={'src' : self.ICO_BLOCKED})
+        headers = soup.findAll(attrs={'class':'block-header block-header-left mtop20 mbottom20'})
-                        a = ico_blocked[-1].findNext('a', attrs={'title' : re.compile(r'Spis *', re.IGNORECASE | re.DOTALL)})
+        articles_list = soup.findAll(attrs={'class':'standard-box'})
-                else:
+        for i in range(len(headers)):
-                        a = soup.find('a', attrs={'title' : re.compile(r'Spis *', re.IGNORECASE | re.DOTALL)})
+            articles = self.find_articles(articles_list[i])
-                self.EDITION = a['href'].replace('/tygodnik/?I=', '')
+            if len(articles) > 0:
-		self.EDITION_SHORT = a['href'].replace('/tygodnik/?I=15', '')
+                section = headers[i].find('a').string
-		self.cover_url = a.img['src']
+                feeds.append((section, articles))
+        return feeds
 
-        def parse_index(self):
+    def find_articles(self, main_block):
-                self.find_last_issue()
+        articles = []
-                soup = self.index_to_soup('http://www.wprost.pl/tygodnik/?I=' + self.EDITION)
+        for a in main_block.findAll('a'):
-                feeds = []
+            if a.name in "td":
-                for main_block in soup.findAll(attrs={'id': 'content-main-column-element-content'}):
+                break
-                        articles = list(self.find_articles(main_block))
+            if self.EXCLUDE_LOCKED and self.is_blocked(a):
-                        if len(articles) > 0:
+                continue
-                                section = self.tag_to_string(main_block.find('h3'))
+            articles.append({
-                                feeds.append((section, articles))
+                'title' : self.tag_to_string(a),
-                return feeds
+                'url'   : 'http://www.wprost.pl' + a['href'],
-
+                'date'  : '',
-        def find_articles(self, main_block):
+                'description' : ''
-                for a in main_block.findAll('a'):
+            })
-                        if a.name in "td":
+        return articles
-                                break
-                        if self.EXCLUDE_LOCKED & self.is_blocked(a):
-                                continue
-                        yield {
-                                'title' : self.tag_to_string(a),
-                                'url'   : 'http://www.wprost.pl' + a['href'],
-                                'date'  : '',
-                                'description' : ''
-                                }
 
 

recipes/wprost_rss.recipe

@@ -0,0 +1,71 @@
+#!/usr/bin/env  python
+
+__license__   = 'GPL v3'
+__copyright__ = '2010, matek09, matek09@gmail.com'
+__copyright__ = 'Modified 2011,  Mariusz Wolek <mariusz_dot_wolek @ gmail dot com>'
+__copyright__ = 'Modified 2012,  Artur Stachecki <artur.stachecki@gmail.com>'
+
+
+from calibre.web.feeds.news import BasicNewsRecipe
+import re
+
+class Wprost(BasicNewsRecipe):
+    title = u'Wprost (RSS)'
+    __author__ = 'matek09'
+    description = 'Weekly magazine'
+    encoding = 'ISO-8859-2'
+    no_stylesheets = True
+    language = 'pl'
+    remove_javascript = True
+    recursions = 0
+    use_embedded_content = False
+    remove_empty_feeds = True
+    remove_tags_before = dict(dict(name = 'div', attrs = {'id' : 'print-layer'}))
+    remove_tags_after = dict(dict(name = 'div', attrs = {'id' : 'print-layer'}))
+    '''
+    keep_only_tags =[]
+    keep_only_tags.append(dict(name = 'table', attrs = {'id' : 'title-table'}))
+    keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'div-header'}))
+    keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'div-content'}))
+    keep_only_tags.append(dict(name = 'div', attrs = {'class' : 'def element-autor'}))
+    '''
+
+    preprocess_regexps = [(re.compile(r'style="display: none;"'), lambda match: ''),
+        (re.compile(r'display: block;'), lambda match: ''),
+        (re.compile(r'\<td\>\<tr\>\<\/table\>'), lambda match: ''),
+        (re.compile(r'\<table .*?\>'), lambda match: ''),
+        (re.compile(r'\<tr>'), lambda match: ''),
+        (re.compile(r'\<td .*?\>'), lambda match: ''),
+        (re.compile(r'\<div id="footer"\>.*?\</footer\>'), lambda match: '')]
+
+    remove_tags =[]
+    remove_tags.append(dict(name = 'div', attrs = {'class' : 'def element-date'}))
+    remove_tags.append(dict(name = 'div', attrs = {'class' : 'def silver'}))
+    remove_tags.append(dict(name = 'div', attrs = {'id' : 'content-main-column-right'}))
+
+
+    extra_css = '''.div-header {font-size: x-small; font-weight: bold}'''
+    #h2 {font-size: x-large; font-weight: bold}
+
+    feeds = [(u'Tylko u nas', u'http://www.wprost.pl/rss/rss_wprostextra.php'),
+	(u'Wydarzenia', u'http://www.wprost.pl/rss/rss.php'),
+	(u'Komentarze', u'http://www.wprost.pl/rss/rss_komentarze.php'),
+	(u'Wydarzenia: Kraj', u'http://www.wprost.pl/rss/rss_kraj.php'),
+	(u'Komentarze: Kraj', u'http://www.wprost.pl/rss/rss_komentarze_kraj.php'),
+	(u'Wydarzenia: Świat', u'http://www.wprost.pl/rss/rss_swiat.php'),
+	(u'Komentarze: Świat', u'http://www.wprost.pl/rss/rss_komentarze_swiat.php'),
+	(u'Wydarzenia: Gospodarka', u'http://www.wprost.pl/rss/rss_gospodarka.php'),
+	(u'Komentarze: Gospodarka', u'http://www.wprost.pl/rss/rss_komentarze_gospodarka.php'),
+	(u'Wydarzenia: Życie', u'http://www.wprost.pl/rss/rss_zycie.php'),
+	(u'Komentarze: Życie', u'http://www.wprost.pl/rss/rss_komentarze_zycie.php'),
+	(u'Wydarzenia: Sport', u'http://www.wprost.pl/rss/rss_sport.php'),
+	(u'Komentarze: Sport', u'http://www.wprost.pl/rss/rss_komentarze_sport.php'),
+	(u'Przegląd prasy', u'http://www.wprost.pl/rss/rss_prasa.php')
+	]
+
+    def get_cover_url(self):
+        soup = self.index_to_soup('http://www.wprost.pl/tygodnik')
+        cover = soup.find(attrs={'class':'wprost-cover'})
+        if cover:
+            self.cover_url = cover['src']
+        return getattr(self, 'cover_url', self.cover_url)

resources/content_server/index.html

@@ -1,5 +1,5 @@
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
 <?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" version="XHTML 1.1" xml:lang="en">
 	<head>
 		<title>calibre library</title>

resources/default_tweaks.py

@@ -464,12 +464,15 @@ server_listen_on = '0.0.0.0'
 # on at your own risk!
 unified_title_toolbar_on_osx = False
 
-#: Save original file when converting from same format to same format
+#: Save original file when converting/polishing from same format to same format
 # When calibre does a conversion from the same format to the same format, for
 # example, from EPUB to EPUB, the original file is saved, so that in case the
 # conversion is poor, you can tweak the settings and run it again. By setting
 # this to False you can prevent calibre from saving the original file.
+# Similarly, by setting save_original_format_when_polishing to False you can
+# prevent calibre from saving the original file when polishing.
 save_original_format = True
+save_original_format_when_polishing = True
 
 #: Number of recently viewed books to show
 # Right-clicking the View button shows a list of recently viewed books. Control

setup/check.py

@@ -95,9 +95,9 @@ class Check(Command):
                     errors = True
                     self.report_errors(w)
             else:
+                from calibre.utils.serve_coffee import check_coffeescript
                 try:
-                    subprocess.check_call(['coffee', '-c', '-p', f],
+                   check_coffeescript(f)
-                            stdout=open(os.devnull, 'wb'))
                 except:
                     errors = True
             if errors:

src/calibre/constants.py

@@ -4,7 +4,7 @@ __license__   = 'GPL v3'
 __copyright__ = '2008, Kovid Goyal kovid@kovidgoyal.net'
 __docformat__ = 'restructuredtext en'
 __appname__   = u'calibre'
-numeric_version = (0, 9, 18)
+numeric_version = (0, 9, 19)
 __version__   = u'.'.join(map(unicode, numeric_version))
 __author__    = u"Kovid Goyal <kovid@kovidgoyal.net>"
 

src/calibre/customize/builtins.py

@@ -624,12 +624,6 @@ from calibre.library.catalogs.epub_mobi import EPUB_MOBI
 plugins += [CSV_XML, BIBTEX, EPUB_MOBI]
 # }}}
 
-# EPUB Fix plugins {{{
-from calibre.ebooks.epub.fix.unmanifested import Unmanifested
-from calibre.ebooks.epub.fix.epubcheck import Epubcheck
-plugins += [Unmanifested, Epubcheck]
-# }}}
-
 # Profiles {{{
 from calibre.customize.profiles import input_profiles, output_profiles
 plugins += input_profiles + output_profiles
@@ -790,11 +784,11 @@ class ActionConvert(InterfaceActionBase):
     actual_plugin = 'calibre.gui2.actions.convert:ConvertAction'
     description = _('Convert books to various ebook formats')
 
-# class ActionPolish(InterfaceActionBase):
+class ActionPolish(InterfaceActionBase):
-#     name = 'Polish Books'
+    name = 'Polish Books'
-#     actual_plugin = 'calibre.gui2.actions.polish:PolishAction'
+    actual_plugin = 'calibre.gui2.actions.polish:PolishAction'
-#     description = _('Fine tune your ebooks')
+    description = _('Fine tune your ebooks')
-#
+
 class ActionDelete(InterfaceActionBase):
     name = 'Remove Books'
     actual_plugin = 'calibre.gui2.actions.delete:DeleteAction'
@@ -930,7 +924,7 @@ class ActionPluginUpdater(InterfaceActionBase):
 
 plugins += [ActionAdd, ActionFetchAnnotations, ActionGenerateCatalog,
         ActionConvert, ActionDelete, ActionEditMetadata, ActionView,
-        ActionFetchNews, ActionSaveToDisk, ActionQuickview, #ActionPolish,
+        ActionFetchNews, ActionSaveToDisk, ActionQuickview, ActionPolish,
         ActionShowBookDetails,ActionRestart, ActionOpenFolder, ActionConnectShare,
         ActionSendToDevice, ActionHelp, ActionPreferences, ActionSimilarBooks,
         ActionAddToLibrary, ActionEditCollections, ActionChooseLibrary,

src/calibre/customize/ui.py

@@ -17,7 +17,6 @@ from calibre.devices.interface import DevicePlugin
 from calibre.ebooks.metadata import MetaInformation
 from calibre.utils.config import (make_config_dir, Config, ConfigProxy,
                                  plugin_dir, OptionParser)
-from calibre.ebooks.epub.fix import ePubFixer
 from calibre.ebooks.metadata.sources.base import Source
 from calibre.constants import DEBUG
 
@@ -489,15 +488,6 @@ def disabled_device_plugins():
                     yield plugin
 # }}}
 
-# epub fixers {{{
-def epub_fixers():
-    for plugin in _initialized_plugins:
-        if isinstance(plugin, ePubFixer):
-            if not is_disabled(plugin):
-                if platform in plugin.supported_platforms:
-                    yield plugin
-# }}}
-
 # Metadata sources2 {{{
 def metadata_plugins(capabilities):
     capabilities = frozenset(capabilities)

src/calibre/devices/misc.py

@@ -209,8 +209,9 @@ class ALURATEK_COLOR(USBMS):
 
     EBOOK_DIR_MAIN = EBOOK_DIR_CARD_A = 'books'
 
-    VENDOR_NAME = ['USB_2.0', 'EZREADER']
+    VENDOR_NAME = ['USB_2.0', 'EZREADER', 'C4+']
-    WINDOWS_MAIN_MEM = WINDOWS_CARD_A_MEM = ['USB_FLASH_DRIVER', '.']
+    WINDOWS_MAIN_MEM = WINDOWS_CARD_A_MEM = ['USB_FLASH_DRIVER', '.', 'TOUCH']
+    SCAN_FROM_ROOT = True
 
 class TREKSTOR(USBMS):
 
@@ -225,6 +226,7 @@ class TREKSTOR(USBMS):
 
     VENDOR_ID   = [0x1e68]
     PRODUCT_ID  = [0x0041, 0x0042, 0x0052, 0x004e, 0x0056,
+            0x0067, # This is for the Pyrus Mini
             0x003e, # This is for the EBOOK_PLAYER_5M https://bugs.launchpad.net/bugs/792091
             0x5cL, # This is for the 4ink http://www.mobileread.com/forums/showthread.php?t=191318
             ]
@@ -234,7 +236,7 @@ class TREKSTOR(USBMS):
 
     VENDOR_NAME = 'TREKSTOR'
     WINDOWS_MAIN_MEM = WINDOWS_CARD_A_MEM = ['EBOOK_PLAYER_7',
-            'EBOOK_PLAYER_5M', 'EBOOK-READER_3.0', 'EREADER_PYRUS']
+            'EBOOK_PLAYER_5M', 'EBOOK-READER_3.0', 'EREADER_PYRUS', 'PYRUS_MINI']
     SUPPORTS_SUB_DIRS = True
     SUPPORTS_SUB_DIRS_DEFAULT = False
 

src/calibre/devices/smart_device_app/driver.py

@@ -52,9 +52,7 @@ def synchronous(tlockname):
     return _synched
 
 
-class ConnectionListener (Thread):
+class ConnectionListener(Thread):
-
-    all_ip_addresses = dict()
 
     NOT_SERVICED_COUNT = 6
 
@@ -63,7 +61,7 @@ class ConnectionListener (Thread):
         self.daemon = True
         self.driver = driver
         self.keep_running = True
-        all_ip_addresses = dict()
+        self.all_ip_addresses = dict()
 
     def stop(self):
         self.keep_running = False

src/calibre/devices/usbobserver/Makefile

@@ -1,8 +0,0 @@
-usbobserver.so : usbobserver.o
-	gcc -arch i386 -arch ppc -bundle usbobserver.o -o usbobserver.so -framework Python -framework IOKit -framework CoreFoundation
-
-usbobserver.o : usbobserver.c
-	gcc -arch i386 -arch ppc -dynamic -I/Library/Frameworks/Python.framework/Versions/Current/Headers -c usbobserver.c -o usbobserver.o 
-
-clean : 
-	rm -f *.o *.so

src/calibre/ebooks/conversion/cli.py

@@ -67,6 +67,8 @@ def check_command_line_options(parser, args, log):
             ('-h' in args or '--help' in args):
         log.error('Cannot read from', input)
         raise SystemExit(1)
+    if input.endswith('.recipe') and not os.access(input, os.R_OK):
+        input = args[1]
 
     output = args[2]
     if (output.startswith('.') and output[:2] not in {'..', '.'} and '/' not in

src/calibre/ebooks/conversion/plugins/epub_input.py

@@ -80,7 +80,7 @@ class EPUBInput(InputFormatPlugin):
         guide_cover, guide_elem = None, None
         for guide_elem in opf.iterguide():
             if guide_elem.get('type', '').lower() == 'cover':
-                guide_cover = guide_elem.get('href', '')
+                guide_cover = guide_elem.get('href', '').partition('#')[0]
                 break
         if not guide_cover:
             return
@@ -103,6 +103,12 @@ class EPUBInput(InputFormatPlugin):
         if not self.for_viewer:
             spine[0].getparent().remove(spine[0])
             removed = guide_cover
+        else:
+            # Ensure the cover is displayed as the first item in the book, some
+            # epub files have it set with linear='no' which causes the cover to
+            # display in the end
+            spine[0].attrib.pop('linear', None)
+            opf.spine[0].is_linear = True
         guide_elem.set('href', 'calibre_raster_cover.jpg')
         from calibre.ebooks.oeb.base import OPF
         t = etree.SubElement(elem[0].getparent(), OPF('item'),

src/calibre/ebooks/conversion/plugins/oeb_output.py

@@ -82,8 +82,8 @@ class OEBOutput(OutputFormatPlugin):
                     self.log.warn('The cover image has an id != "cover". Renaming'
                             ' to work around bug in Nook Color')
 
-                    import uuid
+                    from calibre.ebooks.oeb.base import uuid_id
-                    newid = str(uuid.uuid4())
+                    newid = uuid_id()
 
                     for item in manifest_items_with_id('cover'):
                         item.set('id', newid)

src/calibre/ebooks/conversion/plugins/recipe_input.py

@@ -68,10 +68,15 @@ class RecipeInput(InputFormatPlugin):
                 recipe = compile_recipe(self.recipe_source)
                 log('Using custom recipe')
             else:
-                from calibre.web.feeds.recipes.collection import \
+                from calibre.web.feeds.recipes.collection import (
-                        get_builtin_recipe_by_title
+                        get_builtin_recipe_by_title, get_builtin_recipe_titles)
                 title = getattr(opts, 'original_recipe_input_arg', recipe_or_file)
                 title = os.path.basename(title).rpartition('.')[0]
+                titles = frozenset(get_builtin_recipe_titles())
+                if title not in titles:
+                    title = getattr(opts, 'original_recipe_input_arg', recipe_or_file)
+                    title = title.rpartition('.')[0]
+
                 raw = get_builtin_recipe_by_title(title, log=log,
                         download_recipe=not opts.dont_download_recipe)
                 builtin = False

src/calibre/ebooks/conversion/preprocess.py

@@ -62,6 +62,26 @@ def wrap_lines(match):
     else:
         return ital+' '
 
+def smarten_punctuation(html, log):
+    from calibre.utils.smartypants import smartyPants
+    from calibre.ebooks.chardet import substitute_entites
+    from calibre.ebooks.conversion.utils import HeuristicProcessor
+    preprocessor = HeuristicProcessor(log=log)
+    from uuid import uuid4
+    start = 'calibre-smartypants-'+str(uuid4())
+    stop = 'calibre-smartypants-'+str(uuid4())
+    html = html.replace('<!--', start)
+    html = html.replace('-->', stop)
+    html = preprocessor.fix_nbsp_indents(html)
+    html = smartyPants(html)
+    html = html.replace(start, '<!--')
+    html = html.replace(stop, '-->')
+    # convert ellipsis to entities to prevent wrapping
+    html = re.sub(r'(?u)(?<=\w)\s?(\.\s?){2}\.', '&hellip;', html)
+    # convert double dashes to em-dash
+    html = re.sub(r'\s--\s', u'\u2014', html)
+    return substitute_entites(html)
+
 class DocAnalysis(object):
     '''
     Provides various text analysis functions to determine how the document is structured.
@@ -638,7 +658,7 @@ class HTMLPreProcessor(object):
             html = preprocessor(html)
 
         if getattr(self.extra_opts, 'smarten_punctuation', False):
-            html = self.smarten_punctuation(html)
+            html = smarten_punctuation(html, self.log)
 
         try:
             unsupported_unicode_chars = self.extra_opts.output_profile.unsupported_unicode_chars
@@ -653,23 +673,4 @@ class HTMLPreProcessor(object):
 
         return html
 
-    def smarten_punctuation(self, html):
-        from calibre.utils.smartypants import smartyPants
-        from calibre.ebooks.chardet import substitute_entites
-        from calibre.ebooks.conversion.utils import HeuristicProcessor
-        preprocessor = HeuristicProcessor(self.extra_opts, self.log)
-        from uuid import uuid4
-        start = 'calibre-smartypants-'+str(uuid4())
-        stop = 'calibre-smartypants-'+str(uuid4())
-        html = html.replace('<!--', start)
-        html = html.replace('-->', stop)
-        html = preprocessor.fix_nbsp_indents(html)
-        html = smartyPants(html)
-        html = html.replace(start, '<!--')
-        html = html.replace(stop, '-->')
-        # convert ellipsis to entities to prevent wrapping
-        html = re.sub(r'(?u)(?<=\w)\s?(\.\s?){2}\.', '&hellip;', html)
-        # convert double dashes to em-dash
-        html = re.sub(r'\s--\s', u'\u2014', html)
-        return substitute_entites(html)
 

src/calibre/ebooks/epub/fix/__init__.py

@@ -1,67 +0,0 @@
-#!/usr/bin/env python
-# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:ai
-
-__license__   = 'GPL v3'
-__copyright__ = '2010, Kovid Goyal <kovid@kovidgoyal.net>'
-__docformat__ = 'restructuredtext en'
-
-
-from calibre.customize import Plugin
-
-class InvalidEpub(ValueError):
-    pass
-
-class ParseError(ValueError):
-
-    def __init__(self, name, desc):
-        self.name = name
-        self.desc = desc
-        ValueError.__init__(self,
-            _('Failed to parse: %(name)s with error: %(err)s')%dict(
-                name=name, err=desc))
-
-class ePubFixer(Plugin):
-
-    supported_platforms = ['windows', 'osx', 'linux']
-    author = 'Kovid Goyal'
-    type = _('ePub Fixer')
-    can_be_disabled = True
-
-    # API that subclasses must implement {{{
-    @property
-    def short_description(self):
-        raise NotImplementedError
-
-    @property
-    def long_description(self):
-        raise NotImplementedError
-
-    @property
-    def fix_name(self):
-        raise NotImplementedError
-
-    @property
-    def options(self):
-        '''
-        Return a list of 4-tuples
-        (option_name, type, default, help_text)
-        type is one of 'bool', 'int', 'string'
-        '''
-        return []
-
-    def run(self, container, opts, log, fix=False):
-        raise NotImplementedError
-    # }}}
-
-    def add_options_to_parser(self, parser):
-        parser.add_option('--' + self.fix_name.replace('_', '-'),
-                help=self.long_description, action='store_true', default=False)
-        for option in self.options:
-            action = 'store'
-            if option[1] == 'bool':
-                action = 'store_true'
-            kwargs = {'action': action, 'default':option[2], 'help':option[3]}
-            if option[1] != 'bool':
-                kwargs['type'] = option[1]
-            parser.add_option('--'+option[0].replace('_', '-'), **kwargs)
-

src/calibre/ebooks/epub/fix/container.py

@@ -1,220 +0,0 @@
-#!/usr/bin/env python
-# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:ai
-
-__license__   = 'GPL v3'
-__copyright__ = '2010, Kovid Goyal <kovid@kovidgoyal.net>'
-__docformat__ = 'restructuredtext en'
-
-import os, posixpath, urllib, sys, re
-
-from lxml import etree
-from lxml.etree import XMLSyntaxError
-
-from calibre.ebooks.epub.fix import InvalidEpub, ParseError
-from calibre import guess_type, prepare_string_for_xml
-from calibre.ebooks.chardet import xml_to_unicode
-from calibre.constants import iswindows
-from calibre.utils.zipfile import ZipFile, ZIP_STORED
-
-exists, join = os.path.exists, os.path.join
-
-OCF_NS = 'urn:oasis:names:tc:opendocument:xmlns:container'
-OPF_NS = 'http://www.idpf.org/2007/opf'
-
-class Container(object):
-
-    META_INF = {
-            'container.xml' : True,
-            'manifest.xml' : False,
-            'encryption.xml' : False,
-            'metadata.xml' : False,
-            'signatures.xml' : False,
-            'rights.xml' : False,
-    }
-
-    def __init__(self, path, log):
-        self.root = os.path.abspath(path)
-        self.log = log
-        self.dirtied = set([])
-        self.cache = {}
-        self.mime_map = {}
-
-        if exists(join(self.root, 'mimetype')):
-            os.remove(join(self.root, 'mimetype'))
-
-        container_path = join(self.root, 'META-INF', 'container.xml')
-        if not exists(container_path):
-            raise InvalidEpub('No META-INF/container.xml in epub')
-        self.container = etree.fromstring(open(container_path, 'rb').read())
-        opf_files = self.container.xpath((
-            r'child::ocf:rootfiles/ocf:rootfile'
-            '[@media-type="%s" and @full-path]'%guess_type('a.opf')[0]
-            ), namespaces={'ocf':OCF_NS}
-        )
-        if not opf_files:
-            raise InvalidEpub('META-INF/container.xml contains no link to OPF file')
-        opf_path = os.path.join(self.root,
-                *opf_files[0].get('full-path').split('/'))
-        if not exists(opf_path):
-            raise InvalidEpub('OPF file does not exist at location pointed to'
-                    ' by META-INF/container.xml')
-
-        # Map of relative paths with / separators to absolute
-        # paths on filesystem with os separators
-        self.name_map = {}
-        for dirpath, dirnames, filenames in os.walk(self.root):
-            for f in filenames:
-                path = join(dirpath, f)
-                name = os.path.relpath(path, self.root).replace(os.sep, '/')
-                self.name_map[name] = path
-                if path == opf_path:
-                    self.opf_name = name
-                    self.mime_map[name] = guess_type('a.opf')[0]
-
-        for item in self.opf.xpath(
-                '//opf:manifest/opf:item[@href and @media-type]',
-                namespaces={'opf':OPF_NS}):
-            href = item.get('href')
-            self.mime_map[self.href_to_name(href,
-                posixpath.dirname(self.opf_name))] = item.get('media-type')
-
-    def manifest_worthy_names(self):
-        for name in self.name_map:
-            if name.endswith('.opf'): continue
-            if name.startswith('META-INF') and \
-                    posixpath.basename(name) in self.META_INF: continue
-            yield name
-
-    def delete_name(self, name):
-        self.mime_map.pop(name, None)
-        path = self.name_map[name]
-        os.remove(path)
-        self.name_map.pop(name)
-
-    def manifest_item_for_name(self, name):
-        href = self.name_to_href(name,
-            posixpath.dirname(self.opf_name))
-        q = prepare_string_for_xml(href, attribute=True)
-        existing = self.opf.xpath('//opf:manifest/opf:item[@href="%s"]'%q,
-                namespaces={'opf':OPF_NS})
-        if not existing:
-            return None
-        return existing[0]
-
-    def add_name_to_manifest(self, name, mt=None):
-        item = self.manifest_item_for_name(name)
-        if item is not None:
-            return
-        manifest = self.opf.xpath('//opf:manifest', namespaces={'opf':OPF_NS})[0]
-        item = manifest.makeelement('{%s}item'%OPF_NS, nsmap={'opf':OPF_NS},
-                href=self.name_to_href(name, posixpath.dirname(self.opf_name)),
-                id=self.generate_manifest_id())
-        if not mt:
-            mt = guess_type(posixpath.basename(name))[0]
-        if not mt:
-            mt = 'application/octest-stream'
-        item.set('media-type', mt)
-        manifest.append(item)
-        self.fix_tail(item)
-
-    def fix_tail(self, item):
-        '''
-        Designed only to work with self closing elements after item has
-        just been inserted/appended
-        '''
-        parent = item.getparent()
-        idx = parent.index(item)
-        if idx == 0:
-            item.tail = parent.text
-        else:
-            item.tail = parent[idx-1].tail
-            if idx == len(parent)-1:
-                parent[idx-1].tail = parent.text
-
-    def generate_manifest_id(self):
-        items = self.opf.xpath('//opf:manifest/opf:item[@id]',
-                namespaces={'opf':OPF_NS})
-        ids = set([x.get('id') for x in items])
-        for x in xrange(sys.maxint):
-            c = 'id%d'%x
-            if c not in ids:
-                return c
-
-    @property
-    def opf(self):
-        return self.get(self.opf_name)
-
-    def href_to_name(self, href, base=''):
-        href = urllib.unquote(href.partition('#')[0])
-        name = href
-        if base:
-            name = posixpath.join(base, href)
-        return name
-
-    def name_to_href(self, name, base):
-        if not base:
-            return name
-        return posixpath.relpath(name, base)
-
-    def get_raw(self, name):
-        path = self.name_map[name]
-        return open(path, 'rb').read()
-
-    def get(self, name):
-        if name in self.cache:
-            return self.cache[name]
-        raw = self.get_raw(name)
-        if name in self.mime_map:
-            try:
-                raw = self._parse(raw, self.mime_map[name])
-            except XMLSyntaxError as err:
-                raise ParseError(name, unicode(err))
-        self.cache[name] = raw
-        return raw
-
-    def set(self, name, val):
-        self.cache[name] = val
-        self.dirtied.add(name)
-
-    def _parse(self, raw, mimetype):
-        mt = mimetype.lower()
-        if mt.endswith('+xml'):
-            parser = etree.XMLParser(no_network=True, huge_tree=not iswindows)
-            raw = xml_to_unicode(raw,
-                strip_encoding_pats=True, assume_utf8=True,
-                resolve_entities=True)[0].strip()
-            idx = raw.find('<html')
-            if idx == -1:
-                idx = raw.find('<HTML')
-            if idx > -1:
-                pre = raw[:idx]
-                raw = raw[idx:]
-                if '<!DOCTYPE' in pre:
-                    user_entities = {}
-                    for match in re.finditer(r'<!ENTITY\s+(\S+)\s+([^>]+)', pre):
-                        val = match.group(2)
-                        if val.startswith('"') and val.endswith('"'):
-                            val = val[1:-1]
-                        user_entities[match.group(1)] = val
-                    if user_entities:
-                        pat = re.compile(r'&(%s);'%('|'.join(user_entities.keys())))
-                        raw = pat.sub(lambda m:user_entities[m.group(1)], raw)
-            return etree.fromstring(raw, parser=parser)
-        return raw
-
-    def write(self, path):
-        for name in self.dirtied:
-            data = self.cache[name]
-            raw = data
-            if hasattr(data, 'xpath'):
-                raw = etree.tostring(data, encoding='utf-8',
-                        xml_declaration=True)
-            with open(self.name_map[name], 'wb') as f:
-                f.write(raw)
-        self.dirtied.clear()
-        zf = ZipFile(path, 'w')
-        zf.writestr('mimetype', bytes(guess_type('a.epub')[0]),
-                compression=ZIP_STORED)
-        zf.add_dir(self.root)
-        zf.close()
-

src/calibre/ebooks/epub/fix/epubcheck.py

@@ -1,91 +0,0 @@
-#!/usr/bin/env python
-# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:ai
-
-__license__   = 'GPL v3'
-__copyright__ = '2010, Kovid Goyal <kovid@kovidgoyal.net>'
-__docformat__ = 'restructuredtext en'
-
-from calibre.ebooks.epub.fix import ePubFixer, InvalidEpub
-
-
-class Epubcheck(ePubFixer):
-
-    name = 'Workaround epubcheck bugs'
-
-    @property
-    def short_description(self):
-        return _('Workaround epubcheck bugs')
-
-    @property
-    def long_description(self):
-        return _('Workarounds for bugs in the latest release of epubcheck. '
-                'epubcheck reports many things as errors that are not '
-                'actually errors. epub-fix will try to detect these and replace '
-                'them with constructs that epubcheck likes. This may cause '
-                'significant changes to your epub, complain to the epubcheck '
-                'project.')
-
-    @property
-    def description(self):
-        return self.long_description
-
-    @property
-    def fix_name(self):
-        return 'epubcheck'
-
-    def fix_pubdates(self):
-        from calibre.utils.date import parse_date, strptime
-
-        dirtied = False
-        opf = self.container.opf
-        for dcdate in opf.xpath('//dc:date',
-                namespaces={'dc':'http://purl.org/dc/elements/1.1/'}):
-            raw = dcdate.text
-            if not raw: raw = ''
-            default = strptime('2000-1-1', '%Y-%m-%d', as_utc=True)
-            try:
-                ts = parse_date(raw, assume_utc=False, as_utc=True,
-                        default=default)
-            except:
-                raise InvalidEpub('Invalid date set in OPF', raw)
-            try:
-                sval = ts.strftime('%Y-%m-%d')
-            except:
-                from calibre import strftime
-                sval = strftime('%Y-%m-%d', ts.timetuple())
-            if sval != raw:
-                self.log.error(
-                    'OPF contains date', raw, 'that epubcheck does not like')
-                if self.fix:
-                    dcdate.text = sval
-                    self.log('\tReplaced', raw, 'with', sval)
-                    dirtied = True
-        if dirtied:
-            self.container.set(self.container.opf_name, opf)
-
-    def fix_preserve_aspect_ratio(self):
-        for name in self.container.name_map:
-            mt = self.container.mime_map.get(name, '')
-            if mt.lower() == 'application/xhtml+xml':
-                root = self.container.get(name)
-                dirtied = False
-                for svg in root.xpath('//svg:svg[@preserveAspectRatio="none"]',
-                        namespaces={'svg':'http://www.w3.org/2000/svg'}):
-                    self.log.error('Found <svg> element with'
-                            ' preserveAspectRatio="none" which epubcheck '
-                            'cannot handle')
-                    if self.fix:
-                        svg.set('preserveAspectRatio', 'xMidYMid meet')
-                        dirtied = True
-                        self.log('\tReplaced none with xMidYMid meet')
-                if dirtied:
-                    self.container.set(name, root)
-
-
-    def run(self, container, opts, log, fix=False):
-        self.container = container
-        self.opts = opts
-        self.log = log
-        self.fix = fix
-        self.fix_pubdates()
-        self.fix_preserve_aspect_ratio()

src/calibre/ebooks/epub/fix/main.py

@@ -1,62 +0,0 @@
-#!/usr/bin/env python
-# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:ai
-
-__license__   = 'GPL v3'
-__copyright__ = '2010, Kovid Goyal <kovid@kovidgoyal.net>'
-__docformat__ = 'restructuredtext en'
-
-import sys, os
-
-from calibre.utils.config import OptionParser
-from calibre.ptempfile import TemporaryDirectory
-from calibre import CurrentDir
-from calibre.utils.zipfile import ZipFile
-from calibre.utils.logging import default_log
-from calibre.customize.ui import epub_fixers
-from calibre.ebooks.epub.fix.container import Container
-from calibre.ebooks.epub.fix import ParseError
-
-
-def option_parser():
-    parser = OptionParser(usage=_(
-        '%prog [options] file.epub\n\n'
-        'Fix common problems in EPUB files that can cause them '
-        'to be rejected by poorly designed publishing services.\n\n'
-        'By default, no fixing is done and messages are printed out '
-        'for each error detected. Use the options to control which errors '
-        'are automatically fixed.'))
-    for fixer in epub_fixers():
-        fixer.add_options_to_parser(parser)
-
-    return parser
-
-
-def run(epub, opts, log):
-    with TemporaryDirectory('_epub-fix') as tdir:
-        with CurrentDir(tdir):
-            zf = ZipFile(epub)
-            zf.extractall()
-            zf.close()
-            container = Container(tdir, log)
-            for fixer in epub_fixers():
-                fix = getattr(opts, fixer.fix_name, False)
-                fixer.run(container, opts, log, fix=fix)
-            container.write(epub)
-
-def main(args=sys.argv):
-    parser = option_parser()
-    opts, args = parser.parse_args(args)
-    if len(args) != 2:
-        parser.print_help()
-        print
-        default_log.error(_('You must specify an epub file'))
-        return
-    epub = os.path.abspath(args[1])
-    try:
-        run(epub, opts, default_log)
-    except ParseError as err:
-        default_log.error(unicode(err))
-        raise SystemExit(1)
-
-if __name__ == '__main__':
-    main()

src/calibre/ebooks/epub/fix/unmanifested.py

@@ -1,53 +0,0 @@
-#!/usr/bin/env python
-# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:ai
-
-__license__   = 'GPL v3'
-__copyright__ = '2010, Kovid Goyal <kovid@kovidgoyal.net>'
-__docformat__ = 'restructuredtext en'
-
-
-from calibre.ebooks.epub.fix import ePubFixer
-
-class Unmanifested(ePubFixer):
-
-    name = 'Fix unmanifested files'
-
-    @property
-    def short_description(self):
-        return _('Fix unmanifested files')
-
-    @property
-    def long_description(self):
-        return _('Fix unmanifested files. epub-fix can either add them to '
-        'the manifest or delete them as specified by the '
-        'delete unmanifested option.')
-
-    @property
-    def description(self):
-        return self.long_description
-
-    @property
-    def fix_name(self):
-        return 'unmanifested'
-
-    @property
-    def options(self):
-        return [('delete_unmanifested', 'bool', False,
-            _('Delete unmanifested files instead of adding them to the manifest'))]
-
-    def run(self, container, opts, log, fix=False):
-        dirtied = False
-        for name in list(container.manifest_worthy_names()):
-            item = container.manifest_item_for_name(name)
-            if item is None:
-                log.error(name, 'not in manifest')
-                if fix:
-                    if opts.delete_unmanifested:
-                        container.delete_name(name)
-                        log('\tDeleted')
-                    else:
-                        container.add_name_to_manifest(name)
-                        log('\tAdded to manifest')
-                        dirtied = True
-        if dirtied:
-            container.set(container.opf_name, container.opf)

src/calibre/ebooks/metadata/epub.py

@@ -249,6 +249,30 @@ def _write_new_cover(new_cdata, cpath):
     save_cover_data_to(new_cdata, new_cover.name)
     return new_cover
 
+def update_metadata(opf, mi, apply_null=False, update_timestamp=False):
+    for x in ('guide', 'toc', 'manifest', 'spine'):
+        setattr(mi, x, None)
+    if mi.languages:
+        langs = []
+        for lc in mi.languages:
+            lc2 = lang_as_iso639_1(lc)
+            if lc2: lc = lc2
+            langs.append(lc)
+        mi.languages = langs
+
+    opf.smart_update(mi)
+    if getattr(mi, 'uuid', None):
+        opf.application_id = mi.uuid
+    if apply_null:
+        if not getattr(mi, 'series', None):
+            opf.series = None
+        if not getattr(mi, 'tags', []):
+            opf.tags = []
+        if not getattr(mi, 'isbn', None):
+            opf.isbn = None
+    if update_timestamp and mi.timestamp is not None:
+        opf.timestamp = mi.timestamp
+
 def set_metadata(stream, mi, apply_null=False, update_timestamp=False):
     stream.seek(0)
     reader = get_zip_reader(stream, root=os.getcwdu())
@@ -279,29 +303,8 @@ def set_metadata(stream, mi, apply_null=False, update_timestamp=False):
             import traceback
             traceback.print_exc()
 
-    for x in ('guide', 'toc', 'manifest', 'spine'):
+    update_metadata(reader.opf, mi, apply_null=apply_null,
-        setattr(mi, x, None)
+                    update_timestamp=update_timestamp)
-    if mi.languages:
-        langs = []
-        for lc in mi.languages:
-            lc2 = lang_as_iso639_1(lc)
-            if lc2: lc = lc2
-            langs.append(lc)
-        mi.languages = langs
-
-
-    reader.opf.smart_update(mi)
-    if getattr(mi, 'uuid', None):
-        reader.opf.application_id = mi.uuid
-    if apply_null:
-        if not getattr(mi, 'series', None):
-            reader.opf.series = None
-        if not getattr(mi, 'tags', []):
-            reader.opf.tags = []
-        if not getattr(mi, 'isbn', None):
-            reader.opf.isbn = None
-    if update_timestamp and mi.timestamp is not None:
-        reader.opf.timestamp = mi.timestamp
 
     newopf = StringIO(reader.opf.render())
     if isinstance(reader.archive, LocalZipFile):

src/calibre/ebooks/oeb/display/webview.py

@@ -31,7 +31,8 @@ def self_closing_sub(match):
     return '<%s%s></%s>'%(match.group(1), match.group(2), match.group(1))
 
 def load_html(path, view, codec='utf-8', mime_type=None,
-        pre_load_callback=lambda x:None, path_is_html=False):
+        pre_load_callback=lambda x:None, path_is_html=False,
+              force_as_html=False):
     from PyQt4.Qt import QUrl, QByteArray
     if mime_type is None:
         mime_type = guess_type(path)[0]
@@ -44,18 +45,20 @@ def load_html(path, view, codec='utf-8', mime_type=None,
             html = f.read().decode(codec, 'replace')
 
     html = EntityDeclarationProcessor(html).processed_html
-    has_svg = re.search(r'<[:a-zA-Z]*svg', html) is not None
     self_closing_pat = re.compile(r'<\s*([A-Za-z1-6]+)([^>]*)/\s*>')
     html = self_closing_pat.sub(self_closing_sub, html)
 
     loading_url = QUrl.fromLocalFile(path)
     pre_load_callback(loading_url)
 
-    if has_svg:
+    if force_as_html or re.search(r'<[:a-zA-Z]*svg', html) is None:
+        view.setHtml(html, loading_url)
+    else:
         view.setContent(QByteArray(html.encode(codec)), mime_type,
                 loading_url)
-    else:
+        mf = view.page().mainFrame()
-        view.setHtml(html, loading_url)
+        elem = mf.findFirstElement('parsererror')
-
+        if not elem.isNull():
-
+            return False
+    return True
 

src/calibre/ebooks/oeb/polish/container.py

@@ -7,12 +7,14 @@ __license__   = 'GPL v3'
 __copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
 __docformat__ = 'restructuredtext en'
 
-import os, logging, sys, hashlib, uuid
+import os, logging, sys, hashlib, uuid, re
-from urllib import unquote as urlunquote
+from io import BytesIO
+from urllib import unquote as urlunquote, quote as urlquote
+from urlparse import urlparse
 
 from lxml import etree
 
-from calibre import guess_type, CurrentDir
+from calibre import guess_type as _guess_type, CurrentDir
 from calibre.customize.ui import (plugin_for_input_format,
         plugin_for_output_format)
 from calibre.ebooks.chardet import xml_to_unicode
@@ -22,8 +24,8 @@ from calibre.ebooks.conversion.preprocess import HTMLPreProcessor, CSSPreProcess
 from calibre.ebooks.mobi import MobiError
 from calibre.ebooks.mobi.reader.headers import MetadataHeader
 from calibre.ebooks.mobi.tweak import set_cover
-from calibre.ebooks.oeb.base import (serialize, OEB_DOCS, _css_logger,
+from calibre.ebooks.oeb.base import (
-                                     OEB_STYLES, OPF2_NS)
+    serialize, OEB_DOCS, _css_logger, OEB_STYLES, OPF2_NS, DC11_NS, OPF)
 from calibre.ebooks.oeb.polish.errors import InvalidBook, DRMError
 from calibre.ebooks.oeb.parse_utils import NotHTML, parse_html, RECOVER_PARSER
 from calibre.ptempfile import PersistentTemporaryDirectory, PersistentTemporaryFile
@@ -33,10 +35,32 @@ from calibre.utils.zipfile import ZipFile
 
 exists, join, relpath = os.path.exists, os.path.join, os.path.relpath
 
-OEB_FONTS = {guess_type('a.ttf')[0], guess_type('b.ttf')[0]}
+def guess_type(x):
+    return _guess_type(x)[0] or 'application/octet-stream'
+
+OEB_FONTS = {guess_type('a.ttf'), guess_type('b.ttf')}
+OPF_NAMESPACES = {'opf':OPF2_NS, 'dc':DC11_NS}
 
 class Container(object):
 
+    '''
+    A container represents an Open EBook as a directory full of files and an
+    opf file. There are two important concepts:
+
+        * The root directory. This is the base of the ebook. All the ebooks
+          files are inside this directory or in its sub-directories.
+
+        * Names: These are paths to the books' files relative to the root
+          directory. They always contain POSIX separators and are unquoted. They
+          can be thought of as canonical identifiers for files in the book.
+          Most methods on the container object work with names.
+
+    When converting between hrefs and names use the methods provided by this
+    class, they assume all hrefs are quoted.
+    '''
+
+    book_type = 'oeb'
+
     def __init__(self, rootpath, opfpath, log):
         self.root = os.path.abspath(rootpath)
         self.log = log
@@ -54,44 +78,67 @@ class Container(object):
         for dirpath, _dirnames, filenames in os.walk(self.root):
             for f in filenames:
                 path = join(dirpath, f)
-                name = relpath(path, self.root).replace(os.sep, '/')
+                name = self.abspath_to_name(path)
                 self.name_path_map[name] = path
-                self.mime_map[name] = guess_type(path)[0]
+                self.mime_map[name] = guess_type(path)
                 # Special case if we have stumbled onto the opf
                 if path == opfpath:
                     self.opf_name = name
                     self.opf_dir = os.path.dirname(path)
-                    self.mime_map[name] = guess_type('a.opf')[0]
+                    self.mime_map[name] = guess_type('a.opf')
+
+        if not hasattr(self, 'opf_name'):
+            raise InvalidBook('Book has no OPF file')
 
         # Update mime map with data from the OPF
-        for item in self.opf.xpath(
+        for item in self.opf_xpath('//opf:manifest/opf:item[@href and @media-type]'):
-                '//opf:manifest/opf:item[@href and @media-type]',
-                namespaces={'opf':OPF2_NS}):
             href = item.get('href')
-            self.mime_map[self.href_to_name(href)] = item.get('media-type')
+            self.mime_map[self.href_to_name(href, self.opf_name)] = item.get('media-type')
 
     def abspath_to_name(self, fullpath):
         return self.relpath(os.path.abspath(fullpath)).replace(os.sep, '/')
 
+    def name_to_abspath(self, name):
+        return os.path.abspath(join(self.root, *name.split('/')))
+
+    def exists(self, name):
+        return os.path.exists(self.name_to_abspath(name))
+
     def href_to_name(self, href, base=None):
         '''
-        Convert an href (relative to base) to a name (i.e. a path
+        Convert an href (relative to base) to a name. base must be a name or
-        relative to self.root with POSIX separators).
+        None, in which case self.root is used.
-
-        base must be an absolute path with OS separators or None, in which case
-        the href is interpreted relative to the dir containing the OPF.
         '''
         if base is None:
-            base = self.opf_dir
+            base = self.root
-        href = urlunquote(href.partition('#')[0])
+        else:
+            base = os.path.dirname(self.name_to_abspath(base))
+        purl = urlparse(href)
+        if purl.scheme or not purl.path or purl.path.startswith('/'):
+            return None
+        href = urlunquote(purl.path)
         fullpath = os.path.join(base, *href.split('/'))
         return self.abspath_to_name(fullpath)
 
+    def name_to_href(self, name, base=None):
+        '''Convert a name to a href relative to base, which must be a name or
+        None in which case self.root is used as the base'''
+        fullpath = self.name_to_abspath(name)
+        basepath = self.root if base is None else os.path.dirname(self.name_to_abspath(base))
+        path = relpath(fullpath, basepath).replace(os.sep, '/')
+        return urlquote(path)
+
+    def opf_xpath(self, expr):
+        return self.opf.xpath(expr, namespaces=OPF_NAMESPACES)
+
     def has_name(self, name):
         return name in self.name_path_map
 
-    def relpath(self, path):
+    def relpath(self, path, base=None):
-        return relpath(path, self.root)
+        '''Convert an absolute path (with os separators) to a path relative to
+        base (defaults to self.root). The relative path is *not* a name. Use
+        abspath_to_name() for that.'''
+        return relpath(path, base or self.root)
 
     def decode(self, data):
         """Automatically decode :param:`data` into a `unicode` object."""
@@ -162,7 +209,7 @@ class Container(object):
     def parsed(self, name):
         ans = self.parsed_cache.get(name, None)
         if ans is None:
-            mime = self.mime_map.get(name, guess_type(name)[0])
+            mime = self.mime_map.get(name, guess_type(name))
             ans = self.parse(self.name_path_map[name], mime)
             self.parsed_cache[name] = ans
         return ans
@@ -171,15 +218,29 @@ class Container(object):
     def opf(self):
         return self.parsed(self.opf_name)
 
+    @property
+    def mi(self):
+        from calibre.ebooks.metadata.opf2 import OPF as O
+        mi = self.serialize_item(self.opf_name)
+        return O(BytesIO(mi), basedir=self.opf_dir, unquote_urls=False,
+                populate_spine=False).to_book_metadata()
+
+    @property
+    def manifest_id_map(self):
+        return {item.get('id'):self.href_to_name(item.get('href'), self.opf_name)
+            for item in self.opf_xpath('//opf:manifest/opf:item[@href and @id]')}
+
+    @property
+    def guide_type_map(self):
+        return {item.get('type', ''):self.href_to_name(item.get('href'), self.opf_name)
+            for item in self.opf_xpath('//opf:guide/opf:reference[@href and @type]')}
+
     @property
     def spine_items(self):
-        manifest_id_map = {item.get('id'):self.href_to_name(item.get('href'))
+        manifest_id_map = self.manifest_id_map
-            for item in self.opf.xpath('//opf:manifest/opf:item[@href and @id]',
-                namespaces={'opf':OPF2_NS})}
 
         linear, non_linear = [], []
-        for item in self.opf.xpath('//opf:spine/opf:itemref[@idref]',
+        for item in self.opf_xpath('//opf:spine/opf:itemref[@idref]'):
-                                   namespaces={'opf':OPF2_NS}):
             idref = item.get('idref')
             name = manifest_id_map.get(idref, None)
             path = self.name_path_map.get(name, None)
@@ -198,30 +259,27 @@ class Container(object):
         any internal caches.
         '''
         removed = set()
-        for elem in self.opf.xpath('//opf:manifest/opf:item[@href]',
+        for elem in self.opf_xpath('//opf:manifest/opf:item[@href]'):
-                                   namespaces={'opf':OPF2_NS}):
+            if self.href_to_name(elem.get('href'), self.opf_name) == name:
-            if self.href_to_name(elem.get('href')) == name:
                 id_ = elem.get('id', None)
                 if id_ is not None:
                     removed.add(id_)
-                elem.getparent().remove(elem)
+                self.remove_from_xml(elem)
                 self.dirty(self.opf_name)
         if removed:
-            for item in self.opf.xpath('//opf:spine/opf:itemref[@idref]',
+            for item in self.opf_xpath('//opf:spine/opf:itemref[@idref]'):
-                                    namespaces={'opf':OPF2_NS}):
                 idref = item.get('idref')
                 if idref in removed:
-                    item.getparent().remove(item)
+                    self.remove_from_xml(item)
                     self.dirty(self.opf_name)
 
-        for item in self.opf.xpath('//opf:guide/opf:reference[@href]',
+        for item in self.opf_xpath('//opf:guide/opf:reference[@href]'):
-                                    namespaces={'opf':OPF2_NS}):
+            if self.href_to_name(item.get('href'), self.opf_name) == name:
-            if self.href_to_name(item.get('href')) == name:
+                self.remove_from_xml(item)
-                item.getparent().remove(item)
                 self.dirty(self.opf_name)
 
-        path = self.name_path_map.pop(name)
+        path = self.name_path_map.pop(name, None)
-        if os.path.exists(path):
+        if path and os.path.exists(path):
             os.remove(path)
         self.mime_map.pop(name, None)
         self.parsed_cache.pop(name, None)
@@ -230,13 +288,142 @@ class Container(object):
     def dirty(self, name):
         self.dirtied.add(name)
 
+    def remove_from_xml(self, item):
+        'Removes item from parent, fixing indentation (works only with self closing items)'
+        parent = item.getparent()
+        idx = parent.index(item)
+        if idx == 0:
+            # We are removing the first item - only care about adjusting
+            # the tail if this was the only child
+            if len(parent) == 1:
+                parent.text = item.tail
+        else:
+            # Make sure the preceding item has this tail
+            parent[idx-1].tail = item.tail
+        parent.remove(item)
+        return item
+
+    def insert_into_xml(self, parent, item, index=None):
+        '''Insert item into parent (or append if index is None), fixing
+        indentation. Only works with self closing items.'''
+        if index is None:
+            parent.append(item)
+        else:
+            parent.insert(index, item)
+        idx = parent.index(item)
+        if idx == 0:
+            item.tail = parent.text
+            # If this is the only child of this parent element, we need a
+            # little extra work as we have gone from a self-closing <foo />
+            # element to <foo><item /></foo>
+            if len(parent) == 1:
+                sibling = parent.getprevious()
+                if sibling is None:
+                    # Give up!
+                    return
+                parent.text = sibling.text
+                item.tail = sibling.tail
+        else:
+            item.tail = parent[idx-1].tail
+            if idx == len(parent)-1:
+                parent[idx-1].tail = parent.text
+
+    def opf_get_or_create(self, name):
+        ans = self.opf_xpath('//opf:'+name)
+        if ans:
+            return ans[0]
+        self.dirty(self.opf_name)
+        package = self.opf_xpath('//opf:package')[0]
+        item = package.makeelement(OPF(name))
+        item.tail = '\n'
+        package.append(item)
+        return item
+
+    def generate_item(self, name, id_prefix=None, media_type=None):
+        '''Add an item to the manifest with href derived from the given
+        name. Ensures uniqueness of href and id automatically. Returns
+        generated item.'''
+        id_prefix = id_prefix or 'id'
+        media_type = media_type or guess_type(name)
+        href = self.name_to_href(name, self.opf_name)
+        base, ext = href.rpartition('.')[0::2]
+        all_ids = {x.get('id') for x in self.opf_xpath('//*[@id]')}
+        c = 0
+        item_id = id_prefix
+        while item_id in all_ids:
+            c += 1
+            item_id = id_prefix + '%d'%c
+        all_names = {x.get('href') for x in self.opf_xpath(
+                '//opf:manifest/opf:item[@href]')}
+
+        def exists(h):
+            return self.exists(self.href_to_name(h, self.opf_name))
+
+        c = 0
+        while href in all_names or exists(href):
+            c += 1
+            href = '%s_%d.%s'%(base, c, ext)
+        manifest = self.opf_xpath('//opf:manifest')[0]
+        item = manifest.makeelement(OPF('item'),
+                                    id=item_id, href=href)
+        item.set('media-type', media_type)
+        self.insert_into_xml(manifest, item)
+        self.dirty(self.opf_name)
+        name = self.href_to_name(href, self.opf_name)
+        self.name_path_map[name] = self.name_to_abspath(name)
+        self.mime_map[name] = media_type
+        return item
+
+    def format_opf(self):
+        mdata = self.opf_xpath('//opf:metadata')[0]
+        mdata.text = '\n    '
+        remove = set()
+        for child in mdata:
+            child.tail = '\n    '
+            if (child.get('name', '').startswith('calibre:') and
+                child.get('content', '').strip() in {'{}', ''}):
+                remove.add(child)
+        for child in remove: mdata.remove(child)
+        if len(mdata) > 0:
+            mdata[-1].tail = '\n  '
+
+    def serialize_item(self, name):
+        data = self.parsed(name)
+        if name == self.opf_name:
+            self.format_opf()
+        data = serialize(data, self.mime_map[name])
+        if name == self.opf_name:
+            # Needed as I can't get lxml to output opf:role and
+            # not output <opf:metadata> as well
+            data = re.sub(br'(<[/]{0,1})opf:', r'\1', data)
+        return data
+
+    def commit_item(self, name):
+        if name not in self.parsed_cache:
+            return
+        data = self.serialize_item(name)
+        self.dirtied.remove(name)
+        self.parsed_cache.pop(name)
+        with open(self.name_path_map[name], 'wb') as f:
+            f.write(data)
+
+    def open(self, name, mode='rb'):
+        ''' Open the file pointed to by name for direct read/write. Note that
+        this will commit the file if it is dirtied and remove it from the parse
+        cache. You must finish with this file before accessing the parsed
+        version of it again, or bad things will happen. '''
+        if name in self.dirtied:
+            self.commit_item(name)
+        self.parsed_cache.pop(name, False)
+        path = self.name_to_abspath(name)
+        base = os.path.dirname(path)
+        if not os.path.exists(base):
+            os.makedirs(base)
+        return open(path, mode)
+
     def commit(self, outpath=None):
         for name in tuple(self.dirtied):
-            self.dirtied.remove(name)
+            self.commit_item(name)
-            data = self.parsed_cache.pop(name)
-            data = serialize(data, self.mime_map[name])
-            with open(self.name_path_map[name], 'wb') as f:
-                f.write(data)
 
     def compare_to(self, other):
         if set(self.name_path_map) != set(other.name_path_map):
@@ -257,6 +444,8 @@ OCF_NS = 'urn:oasis:names:tc:opendocument:xmlns:container'
 
 class EpubContainer(Container):
 
+    book_type = 'epub'
+
     META_INF = {
             'container.xml' : True,
             'manifest.xml' : False,
@@ -290,7 +479,7 @@ class EpubContainer(Container):
         self.container = etree.fromstring(open(container_path, 'rb').read())
         opf_files = self.container.xpath((
             r'child::ocf:rootfiles/ocf:rootfile'
-            '[@media-type="%s" and @full-path]'%guess_type('a.opf')[0]
+            '[@media-type="%s" and @full-path]'%guess_type('a.opf')
             ), namespaces={'ocf':OCF_NS}
         )
         if not opf_files:
@@ -314,7 +503,7 @@ class EpubContainer(Container):
             if alg not in {ADOBE_OBFUSCATION, IDPF_OBFUSCATION}:
                 raise DRMError()
             cr = em.getparent().xpath('descendant::*[local-name()="CipherReference" and @URI]')[0]
-            name = self.href_to_name(cr.get('URI'), self.root)
+            name = self.href_to_name(cr.get('URI'))
             path = self.name_path_map.get(name, None)
             if path is not None:
                 fonts[name] = alg
@@ -327,14 +516,14 @@ class EpubContainer(Container):
                 package_id = val
                 break
         if package_id is not None:
-            for elem in self.opf.xpath('//*[@id=%r]'%package_id):
+            for elem in self.opf_xpath('//*[@id=%r]'%package_id):
                 if elem.text:
                     unique_identifier = elem.text.rpartition(':')[-1]
                     break
         if unique_identifier is not None:
             idpf_key = hashlib.sha1(unique_identifier).digest()
         key = None
-        for item in self.opf.xpath('//*[local-name()="metadata"]/*'
+        for item in self.opf_xpath('//*[local-name()="metadata"]/*'
                                    '[local-name()="identifier"]'):
             scheme = None
             for xkey in item.attrib.keys():
@@ -369,7 +558,7 @@ class EpubContainer(Container):
             outpath = self.pathtoepub
         from calibre.ebooks.tweak import zip_rebuilder
         with open(join(self.root, 'mimetype'), 'wb') as f:
-            f.write(guess_type('a.epub')[0])
+            f.write(guess_type('a.epub'))
         zip_rebuilder(self.root, outpath)
 
 # }}}
@@ -397,6 +586,8 @@ def do_explode(path, dest):
 
 class AZW3Container(Container):
 
+    book_type = 'azw3'
+
     def __init__(self, pathtoazw3, log):
         self.pathtoazw3 = pathtoazw3
         tdir = self.root = PersistentTemporaryDirectory('_azw3_container')

src/calibre/ebooks/oeb/polish/cover.py

@@ -0,0 +1,264 @@
+#!/usr/bin/env python
+# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:fdm=marker:ai
+from __future__ import (unicode_literals, division, absolute_import,
+                        print_function)
+
+__license__   = 'GPL v3'
+__copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
+__docformat__ = 'restructuredtext en'
+
+import shutil, re, os
+
+from calibre.ebooks.oeb.base import OPF, OEB_DOCS, XPath, XLINK, xml2text
+from calibre.ebooks.oeb.polish.replace import replace_links
+from calibre.utils.magick.draw import identify
+
+def set_azw3_cover(container, cover_path, report):
+    name = None
+    found = True
+    for gi in container.opf_xpath('//opf:guide/opf:reference[@href and contains(@type, "cover")]'):
+        href = gi.get('href')
+        name = container.href_to_name(href, container.opf_name)
+        container.remove_from_xml(gi)
+    if name is None or not container.has_name(name):
+        item = container.generate_item(name='cover.jpeg', id_prefix='cover')
+        name = container.href_to_name(item.get('href'), container.opf_name)
+        found = False
+    href = container.name_to_href(name, container.opf_name)
+    guide = container.opf_xpath('//opf:guide')[0]
+    container.insert_into_xml(guide, guide.makeelement(
+        OPF('reference'), href=href, type='cover'))
+    with open(cover_path, 'rb') as src, container.open(name, 'wb') as dest:
+        shutil.copyfileobj(src, dest)
+    container.dirty(container.opf_name)
+    report('Cover updated' if found else 'Cover inserted')
+
+def set_cover(container, cover_path, report):
+    if container.book_type == 'azw3':
+        set_azw3_cover(container, cover_path, report)
+    else:
+        set_epub_cover(container, cover_path, report)
+
+###############################################################################
+# The delightful EPUB cover processing
+
+def is_raster_image(media_type):
+    return media_type and media_type.lower() in {
+        'image/png', 'image/jpeg', 'image/jpg', 'image/gif'}
+
+COVER_TYPES = {            'coverimagestandard', 'other.ms-coverimage-standard',
+            'other.ms-titleimage-standard', 'other.ms-titleimage',
+            'other.ms-coverimage', 'other.ms-thumbimage-standard',
+            'other.ms-thumbimage', 'thumbimagestandard', 'cover'}
+
+def find_cover_image(container):
+    'Find a raster image marked as a cover in the OPF'
+    manifest_id_map = container.manifest_id_map
+    mm = container.mime_map
+    for meta in container.opf_xpath('//opf:meta[@name="cover" and @content]'):
+        item_id = meta.get('content')
+        name = manifest_id_map.get(item_id, None)
+        media_type = mm.get(name, None)
+        if is_raster_image(media_type):
+            return name
+
+    # First look for a guide item with type == 'cover'
+    guide_type_map = container.guide_type_map
+    for ref_type, name in guide_type_map.iteritems():
+        if ref_type.lower() == 'cover' and is_raster_image(mm.get(name, None)):
+            return name
+
+    # Find the largest image from all possible guide cover items
+    largest_cover = (None, 0)
+    for ref_type, name in guide_type_map.iteritems():
+        if ref_type.lower() in COVER_TYPES and is_raster_image(mm.get(name, None)):
+            path = container.name_path_map.get(name, None)
+            if path:
+                sz = os.path.getsize(path)
+                if sz > largest_cover[1]:
+                    largest_cover = (name, sz)
+
+    if largest_cover[0]:
+        return largest_cover[0]
+
+def find_cover_page(container):
+    'Find a document marked as a cover in the OPF'
+    mm = container.mime_map
+    guide_type_map = container.guide_type_map
+    for ref_type, name in guide_type_map.iteritems():
+        if ref_type.lower() == 'cover' and mm.get(name, '').lower() in OEB_DOCS:
+            return name
+
+def find_cover_image_in_page(container, cover_page):
+    root = container.parsed(cover_page)
+    body = XPath('//h:body')(root)
+    if len(body) != 1: return
+    body = body[0]
+    images = []
+    for img in XPath('descendant::h:img[@src]|descendant::svg:svg/descendant::svg:image')(body):
+        href = img.get('src') or img.get(XLINK('href'))
+        if href:
+            name = container.href_to_name(href, base=cover_page)
+            images.append(name)
+    text = re.sub(r'\s+', '', xml2text(body))
+    if text or len(images) > 1:
+        # Document has more content than a single image
+        return
+    if images:
+        return images[0]
+
+def clean_opf(container):
+    'Remove all references to covers from the OPF'
+    manifest_id_map = container.manifest_id_map
+    for meta in container.opf_xpath('//opf:meta[@name="cover" and @content]'):
+        name = manifest_id_map.get(meta.get('content', None), None)
+        container.remove_from_xml(meta)
+        if name and name in container.name_path_map:
+            yield name
+
+    gtm = container.guide_type_map
+    for ref in container.opf_xpath('//opf:guide/opf:reference[@type]'):
+        typ = ref.get('type', '')
+        if typ.lower() in COVER_TYPES:
+            container.remove_from_xml(ref)
+            name = gtm.get(typ, None)
+            if name and name in container.name_path_map:
+                yield name
+
+    container.dirty(container.opf_name)
+
+def create_epub_cover(container, cover_path):
+    from calibre.ebooks.conversion.config import load_defaults
+    from calibre.ebooks.oeb.transforms.cover import CoverManager
+
+    ext = cover_path.rpartition('.')[-1].lower()
+    raster_cover_item = container.generate_item('cover.'+ext, id_prefix='cover')
+    raster_cover = container.href_to_name(raster_cover_item.get('href'),
+                                          container.opf_name)
+    with open(cover_path, 'rb') as src, container.open(raster_cover, 'wb') as dest:
+        shutil.copyfileobj(src, dest)
+    opts = load_defaults('epub_output')
+    keep_aspect = opts.get('preserve_cover_aspect_ratio', False)
+    no_svg = opts.get('no_svg_cover', False)
+    if no_svg:
+        style = 'style="height: 100%%"'
+        templ = CoverManager.NONSVG_TEMPLATE.replace('__style__', style)
+    else:
+        width, height = 600, 800
+        try:
+            width, height = identify(cover_path)[:2]
+        except:
+            container.log.exception("Failed to get width and height of cover")
+        ar = 'xMidYMid meet' if keep_aspect else 'none'
+        templ = CoverManager.SVG_TEMPLATE.replace('__ar__', ar)
+        templ = templ.replace('__viewbox__', '0 0 %d %d'%(width, height))
+        templ = templ.replace('__width__',  str(width))
+        templ = templ.replace('__height__', str(height))
+    titlepage_item = container.generate_item('titlepage.xhtml',
+                                             id_prefix='titlepage')
+    titlepage = container.href_to_name(titlepage_item.get('href'),
+                                          container.opf_name)
+    raw = templ%container.name_to_href(raster_cover).encode('utf-8')
+    with container.open(titlepage, 'wb') as f:
+        f.write(raw)
+
+    # We have to make sure the raster cover item has id="cover" for the moron
+    # that wrote the Nook firmware
+    if raster_cover_item.get('id') != 'cover':
+        from calibre.ebooks.oeb.base import uuid_id
+        newid = uuid_id()
+        for item in container.opf_xpath('//*[@id="cover"]'):
+            item.set('id', newid)
+        for item in container.opf_xpath('//*[@idref="cover"]'):
+            item.set('idref', newid)
+        raster_cover_item.set('id', 'cover')
+
+    spine = container.opf_xpath('//opf:spine')[0]
+    ref = spine.makeelement(OPF('itemref'), idref=titlepage_item.get('id'))
+    container.insert_into_xml(spine, ref, index=0)
+    guide = container.opf_get_or_create('guide')
+    container.insert_into_xml(guide, guide.makeelement(
+        OPF('reference'), type='cover', title=_('Cover'),
+        href=container.name_to_href(titlepage)))
+    metadata = container.opf_get_or_create('metadata')
+    meta = metadata.makeelement(OPF('meta'), name='cover')
+    meta.set('content', raster_cover_item.get('id'))
+    container.insert_into_xml(metadata, meta)
+
+    return raster_cover, titlepage
+
+def remove_cover_image_in_page(container, page, cover_images):
+    for img in container.parsed(page).xpath('//*[local-name()="img" and @src]'):
+        href = img.get('src')
+        name = container.href_to_name(href, page)
+        if name in cover_images:
+            img.getparent.remove(img)
+        break
+
+def set_epub_cover(container, cover_path, report):
+    cover_image = find_cover_image(container)
+    cover_page = find_cover_page(container)
+    wrapped_image = extra_cover_page = None
+    updated = False
+    log = container.log
+
+    possible_removals = set(clean_opf(container))
+    possible_removals
+    # TODO: Handle possible_removals and also iterate over links in the removed
+    # pages and handle possibly removing stylesheets referred to by them.
+
+    spine_items = tuple(container.spine_items)
+    if cover_page is None:
+        # Check if the first item in the spine is a simple cover wrapper
+        candidate = container.abspath_to_name(spine_items[0])
+        if find_cover_image_in_page(container, candidate) is not None:
+            cover_page = candidate
+
+    if cover_page is not None:
+        log('Found existing cover page')
+        wrapped_image = find_cover_image_in_page(container, cover_page)
+
+        if len(spine_items) > 1:
+            # Look for an extra cover page
+            c = container.abspath_to_name(spine_items[1])
+            if c != cover_page:
+                candidate = find_cover_image_in_page(container, c)
+                if candidate and candidate in {wrapped_image, cover_image}:
+                    log('Found an extra cover page that is a simple wrapper, removing it')
+                    # This page has only a single image and that image is the
+                    # cover image, remove it.
+                    container.remove_item(c)
+                    extra_cover_page = c
+                    spine_items = spine_items[:1] + spine_items[2:]
+                elif candidate is None:
+                    # Remove the cover image if it is the first image in this
+                    # page
+                    remove_cover_image_in_page(container, c, {wrapped_image,
+                                                          cover_image})
+
+        if wrapped_image is not None:
+            # The cover page is a simple wrapper around a single cover image,
+            # we can remove it safely.
+            log('Existing cover page is a simple wrapper, removing it')
+            container.remove_item(cover_page)
+            container.remove_item(wrapped_image)
+            updated = True
+
+    if cover_image and cover_image != wrapped_image:
+        # Remove the old cover image
+        container.remove_item(cover_image)
+
+    # Insert the new cover
+    raster_cover, titlepage = create_epub_cover(container, cover_path)
+
+    report('Cover updated' if updated else 'Cover inserted')
+
+    # Replace links to the old cover image/cover page
+    link_sub = {s:d for s, d in {
+        cover_page:titlepage, wrapped_image:raster_cover,
+        cover_image:raster_cover, extra_cover_page:titlepage}.iteritems()
+        if s is not None}
+    if link_sub:
+        replace_links(container, link_sub, frag_map=lambda x, y:None)
+
+

src/calibre/ebooks/oeb/polish/jacket.py

@@ -0,0 +1,75 @@
+#!/usr/bin/env python
+# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:fdm=marker:ai
+from __future__ import (unicode_literals, division, absolute_import,
+                        print_function)
+
+__license__   = 'GPL v3'
+__copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
+__docformat__ = 'restructuredtext en'
+
+from calibre.customize.ui import output_profiles
+from calibre.ebooks.conversion.config import load_defaults
+from calibre.ebooks.oeb.base import XPath, OPF
+from calibre.ebooks.oeb.polish.cover import find_cover_page
+from calibre.ebooks.oeb.transforms.jacket import render_jacket as render
+
+def render_jacket(mi):
+    ps = load_defaults('page_setup')
+    op = ps.get('output_profile', 'default')
+    opmap = {x.short_name:x for x in output_profiles()}
+    output_profile = opmap.get(op, opmap['default'])
+    return render(mi, output_profile)
+
+def is_legacy_jacket(root):
+    return len(root.xpath(
+        '//*[starts-with(@class,"calibrerescale") and (local-name()="h1" or local-name()="h2")]')) > 0
+
+def is_current_jacket(root):
+    return len(XPath(
+        '//h:meta[@name="calibre-content" and @content="jacket"]')(root)) > 0
+
+def find_existing_jacket(container):
+    for item in container.spine_items:
+        name = container.abspath_to_name(item)
+        if container.book_type == 'azw3':
+            root = container.parsed(name)
+            if is_current_jacket(root):
+                return name
+        else:
+            if name.rpartition('/')[-1].startswith('jacket') and name.endswith('.xhtml'):
+                root = container.parsed(name)
+                if is_current_jacket(root) or is_legacy_jacket(root):
+                    return name
+
+def replace_jacket(container, name):
+    root = render_jacket(container.mi)
+    container.parsed_cache[name] = root
+    container.dirty(name)
+
+def remove_jacket(container):
+    name = find_existing_jacket(container)
+    if name is not None:
+        container.remove_item(name)
+        return True
+    return False
+
+def add_or_replace_jacket(container):
+    name = find_existing_jacket(container)
+    found = True
+    if name is None:
+        jacket_item = container.generate_item('jacket.xhtml', id_prefix='jacket')
+        name = container.href_to_name(jacket_item.get('href'), container.opf_name)
+        found = False
+    replace_jacket(container, name)
+    if not found:
+        # Insert new jacket into spine
+        index = 0
+        sp = container.abspath_to_name(container.spine_items.next())
+        if sp == find_cover_page(container):
+            index = 1
+        itemref = container.opf.makeelement(OPF('itemref'),
+                                            idref=jacket_item.get('id'))
+        container.insert_into_xml(container.opf_xpath('//opf:spine')[0], itemref,
+                              index=index)
+    return found
+

src/calibre/ebooks/oeb/polish/main.py

@@ -7,19 +7,26 @@ __license__   = 'GPL v3'
 __copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
 __docformat__ = 'restructuredtext en'
 
-import re
+import re, sys, os, time
 from collections import namedtuple
 from functools import partial
 
 from calibre.ebooks.oeb.polish.container import get_container
 from calibre.ebooks.oeb.polish.stats import StatsCollector
 from calibre.ebooks.oeb.polish.subset import subset_all_fonts
+from calibre.ebooks.oeb.polish.cover import set_cover
+from calibre.ebooks.oeb.polish.replace import smarten_punctuation
+from calibre.ebooks.oeb.polish.jacket import (
+    replace_jacket, add_or_replace_jacket, find_existing_jacket, remove_jacket)
 from calibre.utils.logging import Log
 
 ALL_OPTS = {
     'subset': False,
     'opf': None,
     'cover': None,
+    'jacket': False,
+    'remove_jacket':False,
+    'smarten_punctuation':False,
 }
 
 SUPPORTED = {'EPUB', 'AZW3'}
@@ -37,8 +44,8 @@ changes needed for the desired effect.</p>
 
 <p>You should use this tool as the last step in your ebook creation process.</p>
 
-<p>Note that polishing only works on files in the <b>%s</b> formats.</p>
+<p>Note that polishing only works on files in the %s formats.</p>
-''')%_(' or ').join(SUPPORTED),
+''')%_(' or ').join('<b>%s</b>'%x for x in SUPPORTED),
 
 'subset': _('''\
 <p>Subsetting fonts means reducing an embedded font to contain
@@ -58,6 +65,22 @@ characters or completely removed.</p>
 date you decide to add more text to your books, the newly added
 text might not be covered by the subset font.</p>
 '''),
+
+'jacket': _('''\
+<p>Insert a "book jacket" page at the start of the book that contains
+all the book metadata such as title, tags, authors, series, comments,
+etc.</p>'''),
+
+'remove_jacket': _('''\
+<p>Remove a previous inserted book jacket page.</p>
+'''),
+
+'smarten_punctuation': _('''\
+<p>Convert plain text, dashes, ellipsis, multiple hyphens, etc. into their
+typographically correct equivalents.</p>
+<p>Note that the algorithm can sometimes generate incorrect results, especially
+when single quotes at the start of contractions are involved.</p>
+'''),
 }
 
 def hfix(name, raw):
@@ -71,49 +94,125 @@ def hfix(name, raw):
 CLI_HELP = {x:hfix(x, re.sub('<.*?>', '', y)) for x, y in HELP.iteritems()}
 # }}}
 
+def update_metadata(ebook, new_opf):
+    from calibre.ebooks.metadata.opf2 import OPF
+    from calibre.ebooks.metadata.epub import update_metadata
+    opfpath = ebook.name_to_abspath(ebook.opf_name)
+    with ebook.open(ebook.opf_name, 'r+b') as stream, open(new_opf, 'rb') as ns:
+        opf = OPF(stream, basedir=os.path.dirname(opfpath), populate_spine=False,
+                  unquote_urls=False)
+        mi = OPF(ns, unquote_urls=False,
+                      populate_spine=False).to_book_metadata()
+        mi.cover, mi.cover_data = None, (None, None)
+
+        update_metadata(opf, mi, apply_null=True, update_timestamp=True)
+        stream.seek(0)
+        stream.truncate()
+        stream.write(opf.render())
+
 def polish(file_map, opts, log, report):
+    rt = lambda x: report('\n### ' + x)
+    st = time.time()
     for inbook, outbook in file_map.iteritems():
-        report('Polishing: %s'%(inbook.rpartition('.')[-1].upper()))
+        report(_('## Polishing: %s')%(inbook.rpartition('.')[-1].upper()))
         ebook = get_container(inbook, log)
+        jacket = None
 
         if opts.subset:
             stats = StatsCollector(ebook)
 
+        if opts.opf:
+            rt(_('Updating metadata'))
+            update_metadata(ebook, opts.opf)
+            jacket = find_existing_jacket(ebook)
+            if jacket is not None:
+                replace_jacket(ebook, jacket)
+                report(_('Updated metadata jacket'))
+            report(_('Metadata updated\n'))
+
+        if opts.cover:
+            rt(_('Setting cover'))
+            set_cover(ebook, opts.cover, report)
+            report('')
+
+        if opts.jacket:
+            rt(_('Inserting metadata jacket'))
+            if jacket is None:
+                if add_or_replace_jacket(ebook):
+                    report(_('Existing metadata jacket replaced'))
+                else:
+                    report(_('Metadata jacket inserted'))
+            else:
+                report(_('Existing metadata jacket replaced'))
+            report('')
+
+        if opts.remove_jacket:
+            rt(_('Removing metadata jacket'))
+            if remove_jacket(ebook):
+                report(_('Metadata jacket removed'))
+            else:
+                report(_('No metadata jacket found'))
+            report('')
+
+        if opts.smarten_punctuation:
+            rt(_('Smartening punctuation'))
+            smarten_punctuation(ebook, report)
+            report('')
+
         if opts.subset:
-            report('\n### Subsetting embedded fonts')
+            rt(_('Subsetting embedded fonts'))
             subset_all_fonts(ebook, stats.font_stats, report)
             report('')
 
         ebook.commit(outbook)
+        report('-'*70)
+    report(_('Polishing took: %.1f seconds')%(time.time()-st))
+
+REPORT = '{0} REPORT {0}'.format('-'*30)
 
 def gui_polish(data):
     files = data.pop('files')
+    if not data.pop('metadata'):
+        data.pop('opf')
+        data.pop('cover')
     file_map = {x:x for x in files}
     opts = ALL_OPTS.copy()
     opts.update(data)
-    O = namedtuple('Options', ' '.join(data.iterkeys()))
+    O = namedtuple('Options', ' '.join(ALL_OPTS.iterkeys()))
     opts = O(**opts)
     log = Log(level=Log.DEBUG)
     report = []
     polish(file_map, opts, log, report.append)
-    log('\n', '-'*30, ' REPORT ', '-'*30)
+    log('')
+    log(REPORT)
     for msg in report:
         log(msg)
+    return '\n\n'.join(report)
 
 def option_parser():
     from calibre.utils.config import OptionParser
     USAGE = '%prog [options] input_file [output_file]\n\n' + re.sub(
         r'<.*?>', '', CLI_HELP['about'])
     parser = OptionParser(usage=USAGE)
-    o = partial(parser.add_option, default=False, action='store_true')
+    a = parser.add_option
+    o = partial(a, default=False, action='store_true')
     o('--subset-fonts', '-f', dest='subset', help=CLI_HELP['subset'])
+    a('--cover', '-c', help=_(
+        'Path to a cover image. Changes the cover specified in the ebook. '
+        'If no cover is present, or the cover is not properly identified, inserts a new cover.'))
+    a('--opf', '-o', help=_(
+        'Path to an OPF file. The metadata in the book is updated from the OPF file.'))
+    o('--jacket', '-j', help=CLI_HELP['jacket'])
+    o('--remove-jacket', help=CLI_HELP['remove_jacket'])
+    o('--smarten-punctuation', '-p', help=CLI_HELP['smarten_punctuation'])
+
     o('--verbose', help=_('Produce more verbose output, useful for debugging.'))
 
     return parser
 
-def cli_polish():
+def main(args=None):
     parser = option_parser()
-    opts, args = parser.parse_args()
+    opts, args = parser.parse_args(args or sys.argv[1:])
     log = Log(level=Log.DEBUG if opts.verbose else Log.INFO)
     if not args:
         parser.print_help()
@@ -137,24 +236,19 @@ def cli_polish():
     O = namedtuple('Options', ' '.join(popts.iterkeys()))
     popts = O(**popts)
     report = []
-    something = False
+    if not tuple(filter(None, (getattr(popts, name) for name in ALL_OPTS))):
-    for name in ALL_OPTS:
-        if name not in {'opf', 'cover'}:
-            if getattr(popts, name):
-                something = True
-
-    if not something:
         parser.print_help()
         log.error(_('You must specify at least one action to perform'))
         raise SystemExit(1)
 
     polish({inbook:outbook}, popts, log, report.append)
-    log('\n', '-'*30, ' REPORT ', '-'*30)
+    log('')
+    log(REPORT)
     for msg in report:
         log(msg)
 
     log('Output written to:', outbook)
 
 if __name__ == '__main__':
-    cli_polish()
+    main()
 

src/calibre/ebooks/oeb/polish/replace.py

@@ -0,0 +1,85 @@
+#!/usr/bin/env python
+# vim:fileencoding=UTF-8:ts=4:sw=4:sta:et:sts=4:fdm=marker:ai
+from __future__ import (unicode_literals, division, absolute_import,
+                        print_function)
+
+__license__   = 'GPL v3'
+__copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
+__docformat__ = 'restructuredtext en'
+
+import codecs
+from urlparse import urlparse
+
+from cssutils import replaceUrls
+
+from calibre.ebooks.chardet import strip_encoding_declarations
+from calibre.ebooks.oeb.polish.container import guess_type
+from calibre.ebooks.oeb.base import (OEB_DOCS, OEB_STYLES, rewrite_links)
+
+class LinkReplacer(object):
+
+    def __init__(self, base, container, link_map, frag_map):
+        self.base = base
+        self.frag_map = frag_map
+        self.link_map = link_map
+        self.container = container
+        self.replaced = False
+
+    def __call__(self, url):
+        name = self.container.href_to_name(url, self.base)
+        if not name:
+            return url
+        nname = self.link_map.get(name, None)
+        if not nname:
+            return url
+        purl = urlparse(url)
+        href = self.container.name_to_href(nname, self.base)
+        if purl.fragment:
+            nfrag = self.frag_map(name, purl.fragment)
+            if nfrag:
+                href += '#%s'%nfrag
+        if href != url:
+            self.replaced = True
+        return href
+
+def replace_links(container, link_map, frag_map=lambda name, frag:frag):
+    ncx_type = guess_type('toc.ncx')
+    for name, media_type in container.mime_map.iteritems():
+        repl = LinkReplacer(name, container, link_map, frag_map)
+        if media_type.lower() in OEB_DOCS:
+            rewrite_links(container.parsed(name), repl)
+        elif media_type.lower() in OEB_STYLES:
+            replaceUrls(container.parsed(name), repl)
+        elif media_type.lower() == ncx_type:
+            for elem in container.parsed(name).xpath('//*[@src]'):
+                src = elem.get('src')
+                nsrc = repl(src)
+                if src != nsrc:
+                    elem.set('src', nsrc)
+
+        if repl.replaced:
+            container.dirty(name)
+
+def smarten_punctuation(container, report):
+    from calibre.ebooks.conversion.preprocess import smarten_punctuation
+    for path in container.spine_items:
+        name = container.abspath_to_name(path)
+        changed = False
+        with container.open(name, 'r+b') as f:
+            html = container.decode(f.read())
+            newhtml = smarten_punctuation(html, container.log)
+            if newhtml != html:
+                changed = True
+                report(_('Smartened punctuation in: %s')%name)
+                newhtml = strip_encoding_declarations(newhtml)
+                f.seek(0)
+                f.truncate()
+                f.write(codecs.BOM_UTF8 + newhtml.encode('utf-8'))
+        if changed:
+            # Add an encoding declaration (it will be added automatically when
+            # serialized)
+            root = container.parsed(name)
+            for m in root.xpath('descendant::*[local-name()="meta" and @http-equiv]'):
+                m.getparent().remove(m)
+            container.dirty(name)
+

src/calibre/ebooks/oeb/polish/stats.py

@@ -61,9 +61,8 @@ def get_matching_rules(rules, font):
     # Filter on font stretch
     width = widths[font.get('font-stretch', 'normal')]
 
-    min_dist = min(abs(width-f['width']) for f in matches)
+    min_dist = min(abs(width-y['width']) for y in matches)
-    nearest = [f for f in matches if abs(width-f['width']) ==
+    nearest = [x for x in matches if abs(width-x['width']) == min_dist]
-        min_dist]
     if width <= 4:
         lmatches = [f for f in nearest if f['width'] <= width]
     else:
@@ -108,6 +107,8 @@ class Page(QWebPage): # {{{
         self.js = None
         self.evaljs = self.mainFrame().evaluateJavaScript
         self.bridge_value = None
+        nam = self.networkAccessManager()
+        nam.setNetworkAccessible(nam.NotAccessible)
 
     def javaScriptConsoleMessage(self, msg, lineno, msgid):
         self.log(u'JS:', unicode(msg))
@@ -199,6 +200,22 @@ class StatsCollector(object):
 
         self.render_book()
 
+    def href_to_name(self, href, warn_name):
+        if not href.startswith('file://'):
+            self.log.warn('Non-local URI in', warn_name, ':', href, 'ignoring')
+            return None
+        src = href[len('file://'):]
+        if iswindows and len(src) > 2 and (src[0], src[2]) == ('/', ':'):
+            src = src[1:]
+        src = src.replace('/', os.sep)
+        src = unquote(src)
+        name = self.container.abspath_to_name(src)
+        if not self.container.has_name(name):
+            self.log.warn('Missing resource', href, 'in', warn_name,
+                          'ignoring')
+            return None
+        return name
+
     def collect_font_stats(self):
         self.page.evaljs('window.font_stats.get_font_face_rules()')
         font_face_rules = self.page.bridge_value
@@ -220,19 +237,7 @@ class StatsCollector(object):
             if not src: continue
             style = parseStyle('background-image:%s'%src, validate=False)
             src = style.getProperty('background-image').propertyValue[0].uri
-            if not src.startswith('file://'):
+            name = self.href_to_name(src, '@font-face rule')
-                self.log.warn('Unknown URI in @font-face: %r'%src)
-                continue
-            src = src[len('file://'):]
-            if iswindows and src.startswith('/'):
-                src = src[1:]
-            src = src.replace('/', os.sep)
-            src = unquote(src)
-            name = self.container.abspath_to_name(src)
-            if not self.container.has_name(name):
-                self.log.warn('Font %r referenced in @font-face rule not found'
-                              %name)
-                continue
             rule['src'] = name
             normalize_font_properties(rule)
             rule['width'] = widths[rule['font-stretch']]

src/calibre/ebooks/oeb/polish/subset.py

@@ -15,7 +15,7 @@ from calibre.ebooks.oeb.polish.container import OEB_FONTS
 from calibre.utils.fonts.sfnt.subset import subset
 from calibre.utils.fonts.utils import get_font_names
 
-def remove_font_face_rules(container, sheet, remove_names):
+def remove_font_face_rules(container, sheet, remove_names, base):
     changed = False
     for rule in tuple(sheet.cssRules):
         if rule.type != rule.FONT_FACE_RULE:
@@ -24,7 +24,7 @@ def remove_font_face_rules(container, sheet, remove_names):
             uri = rule.style.getProperty('src').propertyValue[0].uri
         except (IndexError, KeyError, AttributeError, TypeError, ValueError):
             continue
-        name = container.href_to_name(uri)
+        name = container.href_to_name(uri, base)
         if name in remove_names:
             sheet.deleteRule(rule)
             changed = True
@@ -54,7 +54,10 @@ def subset_all_fonts(container, font_stats, report):
                 olen = sum(old_sizes.itervalues())
                 nlen = sum(new_sizes.itervalues())
                 total_new += len(nraw)
-                report('Decreased the font %s to %.1f%% of its original size'%
+                if nlen == olen:
+                    report('The font %s was already subset'%font_name)
+                else:
+                    report('Decreased the font %s to %.1f%% of its original size'%
                        (font_name, nlen/olen * 100))
                 f.seek(0), f.truncate(), f.write(nraw)
 
@@ -65,13 +68,13 @@ def subset_all_fonts(container, font_stats, report):
         for name, mt in container.mime_map.iteritems():
             if mt in OEB_STYLES:
                 sheet = container.parsed(name)
-                if remove_font_face_rules(container, sheet, remove):
+                if remove_font_face_rules(container, sheet, remove, name):
                     container.dirty(name)
             elif mt in OEB_DOCS:
                 for style in XPath('//h:style')(container.parsed(name)):
                     if style.get('type', 'text/css') == 'text/css' and style.text:
                         sheet = container.parse_css(style.text, name)
-                        if remove_font_face_rules(container, sheet, remove):
+                        if remove_font_face_rules(container, sheet, remove, name):
                             style.text = sheet.cssText
                             container.dirty(name)
     if total_old > 0:

src/calibre/ebooks/oeb/transforms/metadata.py

@@ -146,7 +146,7 @@ class MergeMetadata(object):
                     return item.id
                 self.remove_old_cover(item)
             elif not cdata:
-                id = self.oeb.manifest.generate(id='cover')
+                id = self.oeb.manifest.generate(id='cover')[0]
                 self.oeb.manifest.add(id, old_cover.href, 'image/jpeg')
                 return id
         if cdata:

src/calibre/ebooks/oeb/transforms/split.py

@@ -117,8 +117,7 @@ class Split(object):
                 continue
 
         page_breaks = list(page_breaks)
-        page_breaks.sort(cmp=
+        page_breaks.sort(key=lambda x:int(x.get('pb_order')))
-              lambda x,y : cmp(int(x.get('pb_order')), int(y.get('pb_order'))))
         page_break_ids, page_breaks_ = [], []
         for i, x in enumerate(page_breaks):
             x.set('id', x.get('id', 'calibre_pb_%d'%i))
@@ -235,7 +234,8 @@ class FlowSplitter(object):
         for pattern, before in ordered_ids:
             elem = pattern(tree)
             if elem:
-                self.log.debug('\t\tSplitting on page-break')
+                self.log.debug('\t\tSplitting on page-break at %s'%
+                               elem[0].get('id'))
                 before, after = self.do_split(tree, elem[0], before)
                 self.trees.append(before)
                 tree = after
@@ -292,14 +292,9 @@ class FlowSplitter(object):
 
         return npath
 
-
-
     def do_split(self, tree, split_point, before):
         '''
-        Split ``tree`` into a *before* and *after* tree at ``split_point``,
+        Split ``tree`` into a *before* and *after* tree at ``split_point``.
-        preserving tag structure, but not duplicating any text.
-        All tags that have had their text and tail
-        removed have the attribute ``calibre_split`` set to 1.
 
         :param before: If True tree is split before split_point, otherwise after split_point
         :return: before_tree, after_tree
@@ -315,8 +310,9 @@ class FlowSplitter(object):
 
 
         def nix_element(elem, top=True):
+            # Remove elem unless top is False in which case replace elem by its
+            # children
             parent = elem.getparent()
-            index = parent.index(elem)
             if top:
                 parent.remove(elem)
             else:
@@ -325,27 +321,38 @@ class FlowSplitter(object):
 
         # Tree 1
         hit_split_point = False
-        for elem in list(body.iterdescendants()):
+        keep_descendants = False
+        split_point_descendants = frozenset(split_point.iterdescendants())
+        for elem in tuple(body.iterdescendants()):
             if elem is split_point:
                 hit_split_point = True
                 if before:
                     nix_element(elem)
+                else:
+                    # We want to keep the descendants of the split point in
+                    # Tree 1
+                    keep_descendants = True
 
                 continue
             if hit_split_point:
+                if keep_descendants:
+                    if elem in split_point_descendants:
+                        # elem is a descendant keep it
+                        continue
+                    else:
+                        # We are out of split_point, so prevent further set
+                        # lookups of split_point_descendants
+                        keep_descendants = False
                 nix_element(elem)
 
-
         # Tree 2
-        hit_split_point = False
+        for elem in tuple(body2.iterdescendants()):
-        for elem in list(body2.iterdescendants()):
             if elem is split_point2:
-                hit_split_point = True
                 if not before:
-                    nix_element(elem, top=False)
+                    nix_element(elem)
-                continue
+                break
-            if not hit_split_point:
+            nix_element(elem, top=False)
-                nix_element(elem, top=False)
+
         body2.text = '\n'
 
         return tree, tree2
@@ -478,8 +485,7 @@ class FlowSplitter(object):
 
     def commit(self):
         '''
-        Commit all changes caused by the split. This removes the previously
+        Commit all changes caused by the split. Calculates an *anchor_map* for
-        introduced ``calibre_split`` attribute and calculates an *anchor_map* for
         all anchors in the original tree. Internal links are re-directed. The
         original file is deleted and the split files are saved.
         '''

src/calibre/ebooks/pdf/render/from_html.py

@@ -16,6 +16,7 @@ from PyQt4.Qt import (QObject, QPainter, Qt, QSize, QString, QTimer,
 from PyQt4.QtWebKit import QWebView, QWebPage, QWebSettings
 
 from calibre import fit_image
+from calibre.constants import iswindows
 from calibre.ebooks.oeb.display.webview import load_html
 from calibre.ebooks.pdf.render.common import (inch, cm, mm, pica, cicero,
                                               didot, PAPER_SIZES)
@@ -251,16 +252,33 @@ class PDFWriter(QObject):
     def current_page_num(self):
         return self.doc.current_page_num
 
+    def load_mathjax(self):
+        evaljs = self.view.page().mainFrame().evaluateJavaScript
+        mjpath = P(u'viewer/mathjax').replace(os.sep, '/')
+        if iswindows:
+            mjpath = u'/' + mjpath
+        if evaljs('''
+                    window.mathjax.base = %s;
+                    mathjax.check_for_math(); mathjax.math_present
+                    '''%(json.dumps(mjpath, ensure_ascii=False))).toBool():
+            self.log.debug('Math present, loading MathJax')
+            while not evaljs('mathjax.math_loaded').toBool():
+                self.loop.processEvents(self.loop.ExcludeUserInputEvents)
+            evaljs('document.getElementById("MathJax_Message").style.display="none";')
+
     def do_paged_render(self):
         if self.paged_js is None:
-            from calibre.utils.resources import compiled_coffeescript
+            from calibre.utils.resources import compiled_coffeescript as cc
-            self.paged_js =  compiled_coffeescript('ebooks.oeb.display.utils')
+            self.paged_js =  cc('ebooks.oeb.display.utils')
-            self.paged_js += compiled_coffeescript('ebooks.oeb.display.indexing')
+            self.paged_js += cc('ebooks.oeb.display.indexing')
-            self.paged_js += compiled_coffeescript('ebooks.oeb.display.paged')
+            self.paged_js += cc('ebooks.oeb.display.paged')
+            self.paged_js += cc('ebooks.oeb.display.mathjax')
 
         self.view.page().mainFrame().addToJavaScriptWindowObject("py_bridge", self)
         evaljs = self.view.page().mainFrame().evaluateJavaScript
         evaljs(self.paged_js)
+        self.load_mathjax()
+
         evaljs('''
         py_bridge.__defineGetter__('value', function() {
             return JSON.parse(this._pass_json_value);

src/calibre/gui2/actions/polish.py

@@ -7,35 +7,52 @@ __license__   = 'GPL v3'
 __copyright__ = '2013, Kovid Goyal <kovid at kovidgoyal.net>'
 __docformat__ = 'restructuredtext en'
 
-import os, weakref, shutil
+import os, weakref, shutil, textwrap
 from collections import OrderedDict
+from functools import partial
 
 from PyQt4.Qt import (QDialog, QGridLayout, QIcon, QCheckBox, QLabel, QFrame,
                       QApplication, QDialogButtonBox, Qt, QSize, QSpacerItem,
-                      QSizePolicy, QTimer)
+                      QSizePolicy, QTimer, QModelIndex, QTextEdit,
+                      QInputDialog, QMenu)
 
-from calibre.gui2 import error_dialog, Dispatcher
+from calibre.gui2 import error_dialog, Dispatcher, gprefs
 from calibre.gui2.actions import InterfaceAction
 from calibre.gui2.convert.metadata import create_opf_file
 from calibre.gui2.dialogs.progress import ProgressDialog
 from calibre.ptempfile import PersistentTemporaryDirectory
 from calibre.utils.config_base import tweaks
 
-
+class Polish(QDialog): # {{{
-class Polish(QDialog):
 
     def __init__(self, db, book_id_map, parent=None):
         from calibre.ebooks.oeb.polish.main import HELP
         QDialog.__init__(self, parent)
         self.db, self.book_id_map = weakref.ref(db), book_id_map
         self.setWindowIcon(QIcon(I('polish.png')))
-        self.setWindowTitle(ngettext(
+        title = _('Polish book')
-            'Polish book', _('Polish %d books')%len(book_id_map), len(book_id_map)))
+        if len(book_id_map) > 1:
+            title = _('Polish %d books')%len(book_id_map)
+        self.setWindowTitle(title)
 
         self.help_text = {
             'polish': _('<h3>About Polishing books</h3>%s')%HELP['about'],
 
             'subset':_('<h3>Subsetting fonts</h3>%s')%HELP['subset'],
+
+            'smarten_punctuation':
+            _('<h3>Smarten punctuation</h3>%s')%HELP['smarten_punctuation'],
+
+            'metadata':_('<h3>Updating metadata</h3>'
+                         '<p>This will update all metadata and covers in the'
+                         ' ebook files to match the current metadata in the'
+                         ' calibre library.</p><p>If the ebook file does not have'
+                         ' an identifiable cover, a new cover is inserted.</p>'
+                         ' <p>Note that most ebook'
+                         ' formats are not capable of supporting all the'
+                         ' metadata in calibre.</p>'),
+            'jacket':_('<h3>Book Jacket</h3>%s')%HELP['jacket'],
+            'remove_jacket':_('<h3>Remove Book Jacket</h3>%s')%HELP['remove_jacket'],
         }
 
         self.l = l = QGridLayout()
@@ -45,12 +62,19 @@ class Polish(QDialog):
         l.addWidget(la, 0, 0, 1, 2)
 
         count = 0
-        self.actions = OrderedDict([
+        self.all_actions = OrderedDict([
             ('subset', _('Subset all embedded fonts')),
+            ('smarten_punctuation', _('Smarten punctuation')),
+            ('metadata', _('Update metadata in book files')),
+            ('jacket', _('Add metadata as a "book jacket" page')),
+            ('remove_jacket', _('Remove a previously inserted book jacket')),
         ])
-        for name, text in self.actions.iteritems():
+        prefs = gprefs.get('polishing_settings', {})
+        for name, text in self.all_actions.iteritems():
             count += 1
             x = QCheckBox(text, self)
+            x.setChecked(prefs.get(name, False))
+            x.stateChanged.connect(partial(self.option_toggled, name))
             l.addWidget(x, count, 0, 1, 1)
             setattr(self, 'opt_'+name, x)
             la = QLabel(' <a href="#%s">%s</a>'%(name, _('About')))
@@ -72,28 +96,106 @@ class Polish(QDialog):
         l.addWidget(la, 0, 2, count+1, 1)
         l.setColumnStretch(2, 1)
 
+        self.show_reports = sr = QCheckBox(_('Show &report'), self)
+        sr.setChecked(gprefs.get('polish_show_reports', True))
+        sr.setToolTip(textwrap.fill(_('Show a report of all the actions performed'
+                        ' after polishing is completed')))
+        l.addWidget(sr, count+1, 0, 1, 1)
         self.bb = bb = QDialogButtonBox(QDialogButtonBox.Ok|QDialogButtonBox.Cancel)
         bb.accepted.connect(self.accept)
         bb.rejected.connect(self.reject)
-        l.addWidget(bb, count+1, 0, 1, -1)
+        self.save_button = sb = bb.addButton(_('&Save Settings'), bb.ActionRole)
+        sb.clicked.connect(self.save_settings)
+        self.load_button = lb = bb.addButton(_('&Load Settings'), bb.ActionRole)
+        self.load_menu = QMenu(lb)
+        lb.setMenu(self.load_menu)
+        self.all_button = b = bb.addButton(_('Select &all'), bb.ActionRole)
+        b.clicked.connect(partial(self.select_all, True))
+        self.none_button = b = bb.addButton(_('Select &none'), bb.ActionRole)
+        b.clicked.connect(partial(self.select_all, False))
+        l.addWidget(bb, count+1, 1, 1, -1)
+        self.setup_load_button()
 
-        self.resize(QSize(800, 600))
+        self.resize(QSize(950, 600))
+
+    def select_all(self, enable):
+        for action in self.all_actions:
+            x = getattr(self, 'opt_'+action)
+            x.blockSignals(True)
+            x.setChecked(enable)
+            x.blockSignals(False)
+
+    def save_settings(self):
+        if not self.something_selected:
+            return error_dialog(self, _('No actions selected'),
+                _('You must select at least one action before saving'),
+                                show=True)
+        name, ok = QInputDialog.getText(self, _('Choose name'),
+                _('Choose a name for these settings'))
+        if ok:
+            name = unicode(name).strip()
+            if name:
+                settings = {ac:getattr(self, 'opt_'+ac).isChecked() for ac in
+                            self.all_actions}
+                saved = gprefs.get('polish_settings', {})
+                saved[name] = settings
+                gprefs.set('polish_settings', saved)
+                self.setup_load_button()
+
+    def setup_load_button(self):
+        saved = gprefs.get('polish_settings', {})
+        m = self.load_menu
+        m.clear()
+        self.__actions = []
+        a = self.__actions.append
+        for name in sorted(saved):
+            a(m.addAction(name, partial(self.load_settings, name)))
+        m.addSeparator()
+        a(m.addAction(_('Remove saved settings'), self.clear_settings))
+        self.load_button.setEnabled(bool(saved))
+
+    def clear_settings(self):
+        gprefs.set('polish_settings', {})
+        self.setup_load_button()
+
+    def load_settings(self, name):
+        saved = gprefs.get('polish_settings', {}).get(name, {})
+        for action in self.all_actions:
+            checked = saved.get(action, False)
+            x = getattr(self, 'opt_'+action)
+            x.blockSignals(True)
+            x.setChecked(checked)
+            x.blockSignals(False)
+
+    def option_toggled(self, name, state):
+        if state == Qt.Checked:
+            self.help_label.setText(self.help_text[name])
 
     def help_link_activated(self, link):
         link = unicode(link)[1:]
         self.help_label.setText(self.help_text[link])
 
+    @property
+    def something_selected(self):
+        for action in self.all_actions:
+            if getattr(self, 'opt_'+action).isChecked():
+                return True
+        return False
+
     def accept(self):
         self.actions = ac = {}
+        saved_prefs = {}
+        gprefs['polish_show_reports'] = bool(self.show_reports.isChecked())
         something = False
-        for action in self.actions:
+        for action in self.all_actions:
-            ac[action] = bool(getattr(self, 'opt_'+action).isChecked())
+            ac[action] = saved_prefs[action] = bool(getattr(self, 'opt_'+action).isChecked())
             if ac[action]:
                 something = True
         if not something:
             return error_dialog(self, _('No actions selected'),
                 _('You must select at least one action, or click Cancel.'),
                                 show=True)
+        gprefs['polishing_settings'] = saved_prefs
         self.queue_files()
         return super(Polish, self).accept()
 
@@ -123,6 +225,7 @@ class Polish(QDialog):
             self.do_book(num, book_id, self.book_id_map[book_id])
         except:
             self.pd.reject()
+            raise
         else:
             self.pd.set_value(num)
             QTimer.singleShot(0, self.do_one)
@@ -148,13 +251,107 @@ class Polish(QDialog):
 
         desc = ngettext(_('Polish %s')%mi.title,
                         _('Polish book %(nums)s of %(tot)s (%(title)s)')%dict(
-                            num=num, tot=len(self.book_id_map),
+                            nums=num, tot=len(self.book_id_map),
                             title=mi.title), len(self.book_id_map))
         if hasattr(self, 'pd'):
             self.pd.set_msg(_('Queueing book %(nums)s of %(tot)s (%(title)s)')%dict(
-                            num=num, tot=len(self.book_id_map), title=mi.title))
+                            nums=num, tot=len(self.book_id_map), title=mi.title))
 
         self.jobs.append((desc, data, book_id, base))
+# }}}
+
+class Report(QDialog): # {{{
+
+    def __init__(self, parent):
+        QDialog.__init__(self, parent)
+        self.gui = parent
+        self.setAttribute(Qt.WA_DeleteOnClose, False)
+        self.setWindowIcon(QIcon(I('polish.png')))
+        self.reports = []
+
+        self.l = l = QGridLayout()
+        self.setLayout(l)
+        self.view = v = QTextEdit(self)
+        v.setReadOnly(True)
+        l.addWidget(self.view, 0, 0, 1, 2)
+
+        self.backup_msg = la = QLabel('')
+        l.addWidget(la, 1, 0, 1, 2)
+        la.setVisible(False)
+        la.setWordWrap(True)
+
+        self.ign_msg = _('Ignore remaining %d reports')
+        self.ign = QCheckBox(self.ign_msg, self)
+        l.addWidget(self.ign, 2, 0)
+
+        bb = self.bb = QDialogButtonBox(QDialogButtonBox.Close)
+        bb.accepted.connect(self.accept)
+        bb.rejected.connect(self.reject)
+        b = self.log_button = bb.addButton(_('View full &log'), bb.ActionRole)
+        b.clicked.connect(self.view_log)
+        bb.button(bb.Close).setDefault(True)
+        l.addWidget(bb, 2, 1)
+
+        self.finished.connect(self.show_next, type=Qt.QueuedConnection)
+
+        self.resize(QSize(800, 600))
+
+    def setup_ign(self):
+        self.ign.setText(self.ign_msg%len(self.reports))
+        self.ign.setVisible(bool(self.reports))
+        self.ign.setChecked(False)
+
+    def __call__(self, *args):
+        self.reports.append(args)
+        self.setup_ign()
+        if not self.isVisible():
+            self.show_next()
+
+    def show_report(self, book_title, book_id, fmts, job, report):
+        from calibre.ebooks.markdown.markdown import markdown
+        self.current_log = job.details
+        self.setWindowTitle(_('Polishing of %s')%book_title)
+        self.view.setText(markdown('# %s\n\n'%book_title + report,
+                                   output_format='html4'))
+        self.bb.button(self.bb.Close).setFocus(Qt.OtherFocusReason)
+        self.backup_msg.setVisible(bool(fmts))
+        if fmts:
+            m = ngettext('The original file has been saved as %s.',
+                     'The original files have been saved as %s.', len(fmts))%(
+                _(' and ').join('ORIGINAL_'+f for f in fmts)
+                     )
+            self.backup_msg.setText(m + ' ' + _(
+                'If you polish again, the polishing will run on the originals.')%(
+                ))
+
+    def view_log(self):
+        self.view.setPlainText(self.current_log)
+        self.view.verticalScrollBar().setValue(0)
+
+    def show_next(self, *args):
+        if not self.reports:
+            return
+        if not self.isVisible():
+            self.show()
+        self.show_report(*self.reports.pop(0))
+        self.setup_ign()
+
+    def accept(self):
+        if self.ign.isChecked():
+            self.reports = []
+        if self.reports:
+            self.show_next()
+            return
+        super(Report, self).accept()
+
+    def reject(self):
+        if self.ign.isChecked():
+            self.reports = []
+        if self.reports:
+            self.show_next()
+            return
+        super(Report, self).reject()
+# }}}
 
 class PolishAction(InterfaceAction):
 
@@ -165,6 +362,7 @@ class PolishAction(InterfaceAction):
 
     def genesis(self):
         self.qaction.triggered.connect(self.polish_books)
+        self.report = Report(self.gui)
 
     def location_selected(self, loc):
         enabled = loc == 'library'
@@ -205,21 +403,28 @@ class PolishAction(InterfaceAction):
             return
         d = Polish(self.gui.library_view.model().db, book_id_map, parent=self.gui)
         if d.exec_() == d.Accepted and d.jobs:
-            for desc, data, book_id, base, files in reversed(d.jobs):
+            show_reports = bool(d.show_reports.isChecked())
+            for desc, data, book_id, base in reversed(d.jobs):
                 job = self.gui.job_manager.run_job(
                     Dispatcher(self.book_polished), 'gui_polish', args=(data,),
                     description=desc)
-                job.polish_args = (book_id, base, data['files'])
+                job.polish_args = (book_id, base, data['files'], show_reports)
+            if d.jobs:
+                self.gui.jobs_pointer.start()
+                self.gui.status_bar.show_message(
+                    _('Start polishing of %d book(s)') % len(d.jobs), 2000)
 
     def book_polished(self, job):
         if job.failed:
             self.gui.job_exception(job)
             return
         db = self.gui.current_db
-        book_id, base, files = job.polish_args
+        book_id, base, files, show_reports = job.polish_args
+        fmts = set()
         for path in files:
             fmt = path.rpartition('.')[-1].upper()
-            if tweaks['save_original_format']:
+            if tweaks['save_original_format_when_polishing']:
+                fmts.add(fmt)
                 db.save_original_format(book_id, fmt, notify=False)
             with open(path, 'rb') as f:
                 db.add_format(book_id, fmt, f, index_is_id=True)
@@ -231,6 +436,13 @@ class PolishAction(InterfaceAction):
             os.rmdir(parent)
         except:
             pass
+        self.gui.tags_view.recount()
+        if self.gui.current_view() is self.gui.library_view:
+            current = self.gui.library_view.currentIndex()
+            if current.isValid():
+                self.gui.library_view.model().current_changed(current, QModelIndex())
+        if show_reports:
+            self.report(db.title(book_id, index_is_id=True), book_id, fmts, job, job.result)
 
 if __name__ == '__main__':
     app = QApplication([])

src/calibre/gui2/dialogs/metadata_bulk.py

@@ -512,7 +512,7 @@ class MetadataBulkDialog(ResizableDialog, Ui_MetadataBulkDialog):
         self.test_text.editTextChanged[str].connect(self.s_r_paint_results)
         self.comma_separated.stateChanged.connect(self.s_r_paint_results)
         self.case_sensitive.stateChanged.connect(self.s_r_paint_results)
-        self.s_r_src_ident.currentIndexChanged[int].connect(self.s_r_paint_results)
+        self.s_r_src_ident.currentIndexChanged[int].connect(self.s_r_identifier_type_changed)
         self.s_r_dst_ident.textChanged.connect(self.s_r_paint_results)
         self.s_r_template.lost_focus.connect(self.s_r_template_changed)
         self.central_widget.setCurrentIndex(0)
@@ -576,9 +576,9 @@ class MetadataBulkDialog(ResizableDialog, Ui_MetadataBulkDialog):
             elif not fm['is_multiple']:
                 val = [val]
             elif fm['datatype'] == 'composite':
-                val = [v.strip() for v in val.split(fm['is_multiple']['ui_to_list'])]
+                val = [v2.strip() for v2 in val.split(fm['is_multiple']['ui_to_list'])]
             elif field == 'authors':
-                val = [v.replace('|', ',') for v in val]
+                val = [v2.replace('|', ',') for v2 in val]
         else:
             val = []
         if not val:
@@ -591,6 +591,10 @@ class MetadataBulkDialog(ResizableDialog, Ui_MetadataBulkDialog):
     def s_r_template_changed(self):
         self.s_r_search_field_changed(self.search_field.currentIndex())
 
+    def s_r_identifier_type_changed(self, idx):
+        self.s_r_search_field_changed(self.search_field.currentIndex())
+        self.s_r_paint_results(idx)
+
     def s_r_search_field_changed(self, idx):
         self.s_r_template.setVisible(False)
         self.template_label.setVisible(False)

src/calibre/gui2/dialogs/scheduler.py

@@ -283,6 +283,7 @@ class SchedulerDialog(QDialog, Ui_Dialog):
         urn = self.current_urn
         if urn is not None:
             self.initialize_detail_box(urn)
+        self.recipes.scrollTo(current)
 
     def accept(self):
         if not self.commit():

src/calibre/gui2/dialogs/template_dialog.py

@@ -197,8 +197,7 @@ class TemplateHighlighter(QSyntaxHighlighter):
 
 class TemplateDialog(QDialog, Ui_TemplateDialog):
 
-    def __init__(self, parent, text, mi=None, fm=None, color_field=None,
+    def __init__(self, parent, text, mi=None, fm=None, color_field=None):
-                 rule_kind='color'):
         QDialog.__init__(self, parent)
         Ui_TemplateDialog.__init__(self)
         self.setupUi(self)
@@ -265,8 +264,7 @@ class TemplateDialog(QDialog, Ui_TemplateDialog):
         self.function.setCurrentIndex(0)
         self.function.currentIndexChanged[str].connect(self.function_changed)
         self.textbox_changed()
-        self.rule = (rule_kind, None, '')
+        self.rule = (None, '')
-        self.rule_kind = rule_kind
 
         tt = _('Template language tutorial')
         self.template_tutorial.setText(
@@ -325,5 +323,5 @@ class TemplateDialog(QDialog, Ui_TemplateDialog):
                     _('The template box cannot be empty'), show=True)
                 return
 
-        self.rule = (self.rule_kind, unicode(self.colored_field.currentText()), txt)
+        self.rule = (unicode(self.colored_field.currentText()), txt)
         QDialog.accept(self)

src/calibre/gui2/main.py

@@ -96,6 +96,7 @@ def init_qt(args):
     QCoreApplication.setApplicationName(APP_UID)
     override = 'calibre-gui' if islinux else None
     app = Application(args, override_program_name=override)
+    app.file_event_hook = EventAccumulator()
     actions = tuple(Main.create_application_menubar())
     app.setWindowIcon(QIcon(I('lt.png')))
     return app, opts, args, actions
@@ -150,6 +151,14 @@ def repair_library(library_path):
     from calibre.gui2.dialogs.restore_library import repair_library_at
     return repair_library_at(library_path)
 
+class EventAccumulator(object):
+
+    def __init__(self):
+        self.events = []
+
+    def __call__(self, ev):
+        self.events.append(ev)
+
 class GuiRunner(QObject):
     '''Make sure an event loop is running before starting the main work of
     initialization'''
@@ -187,6 +196,8 @@ class GuiRunner(QObject):
                 prints('Ignoring directories passed as command line arguments')
             if files:
                 add_filesystem_book(files)
+        for event in self.app.file_event_hook.events:
+            add_filesystem_book(event)
         self.app.file_event_hook = add_filesystem_book
         self.main = main
 
@@ -312,6 +323,8 @@ def run_gui(opts, args, actions, listener, app, gui_debug=None):
                 app = os.path.dirname(os.path.dirname(sys.frameworks_dir))
                 subprocess.Popen('sleep 3s; open '+app, shell=True)
             else:
+                if iswindows and hasattr(winutil, 'prepare_for_restart'):
+                    winutil.prepare_for_restart()
                 subprocess.Popen([e] + sys.argv[1:])
     else:
         if iswindows:

src/calibre/gui2/preferences/coloring.py

@@ -56,7 +56,12 @@ class ConditionEditor(QWidget): # {{{
                 (_('is equal to'), 'eq'),
                 (_('is less than'), 'lt'),
                 (_('is greater than'), 'gt'),
-                (_('is not more days ago than'), 'count_days')
+                (_('is set'), 'is set'),
+                (_('is not set'), 'is not set'),
+                (_('is more days ago than'), 'older count days'),
+                (_('is fewer days ago than'), 'count_days'),
+                (_('is more days from now than'), 'newer future days'),
+                (_('is fewer days from now than'), 'older future days')
             ),
             'multiple' : (
                 (_('has'), 'has'),
@@ -127,7 +132,7 @@ class ConditionEditor(QWidget): # {{{
 
         for b in (self.column_box, self.action_box):
             b.setSizeAdjustPolicy(b.AdjustToMinimumContentsLengthWithIcon)
-            b.setMinimumContentsLength(15)
+            b.setMinimumContentsLength(20)
 
     @dynamic_property
     def current_col(self):
@@ -240,7 +245,20 @@ class ConditionEditor(QWidget): # {{{
         elif dt == 'datetime':
             if action == 'count_days':
                 self.value_box.setValidator(QIntValidator(self.value_box))
-                tt = _('Enter the number of days old the item can be. Zero is today')
+                tt = _('Enter the maximum days old the item can be. Zero is today. '
+                       'Dates in the future always match')
+            elif action == 'older count days':
+                self.value_box.setValidator(QIntValidator(self.value_box))
+                tt = _('Enter the minimum days old the item can be. Zero is today. '
+                       'Dates in the future never match')
+            elif action == 'older future days':
+                self.value_box.setValidator(QIntValidator(self.value_box))
+                tt = _('Enter the maximum days in the future the item can be. '
+                       'Zero is today. Dates in the past always match')
+            elif action == 'newer future days':
+                self.value_box.setValidator(QIntValidator(self.value_box))
+                tt = _('Enter the minimum days in the future the item can be. '
+                       'Zero is today. Dates in the past never match')
             else:
                 self.value_box.setInputMask('9999-99-99')
                 tt = _('Enter a date in the format YYYY-MM-DD')
@@ -640,14 +658,32 @@ class RulesModel(QAbstractListModel): # {{{
             ''') % dict(kind=trans_kind, col=col, color=rule.color, rule=''.join(conditions))
 
     def condition_to_html(self, condition):
-        c, a, v = condition
+        col, a, v = condition
-        c = self.fm[c]['name']
+        dt = self.fm[col]['datatype']
+        c = self.fm[col]['name']
         action_name = a
-        for actions in ConditionEditor.ACTION_MAP.itervalues():
+        if col in ConditionEditor.ACTION_MAP:
-            for trans, ac in actions:
+            # look for a column-name-specific label
+            for trans, ac in ConditionEditor.ACTION_MAP[col]:
                 if ac == a:
                     action_name = trans
-
+                    break
+        elif dt in ConditionEditor.ACTION_MAP:
+            # Look for a type-specific label
+            for trans, ac in ConditionEditor.ACTION_MAP[dt]:
+                if ac == a:
+                    action_name = trans
+                    break
+        else:
+            # Wasn't a type-specific or column-specific label. Look for a text-type
+            for dt in ['single', 'multiple']:
+                for trans, ac in ConditionEditor.ACTION_MAP[dt]:
+                    if ac == a:
+                        action_name = trans
+                        break
+                else:
+                    continue
+                break
         return (
             _('<li>If the <b>%(col)s</b> column <b>%(action)s</b> value: <b>%(val)s</b>') %
                 dict(col=c, action=action_name, val=prepare_string_for_xml(v)))
@@ -742,7 +778,7 @@ class EditRules(QWidget): # {{{
 
     def add_advanced(self):
         td = TemplateDialog(self, '', mi=self.mi, fm=self.fm, color_field='')
-        self._add_rule(td)
+        self._add_rule(('color', td[0], td[1]))
 
     def edit_rule(self, index):
         try:
@@ -755,6 +791,8 @@ class EditRules(QWidget): # {{{
         else:
             d = TemplateDialog(self, rule, mi=self.mi, fm=self.fm, color_field=col)
         if d.exec_() == d.Accepted:
+            if len(d.rule) == 2: # Convert template dialog rules to a triple
+                d.rule = ('color', d.rule[0], d.rule[1])
             kind, col, r = d.rule
             if kind and r is not None and col:
                 self.model.replace_rule(index, kind, col, r)

src/calibre/gui2/store/loader.py

@@ -188,10 +188,12 @@ class Stores(OrderedDict):
 
 if __name__ == '__main__':
     st = time.time()
+    count = 0
     for name, code in download_updates():
+        count += 1
         print(name)
         print(code)
         print('\n', '_'*80, '\n', sep='')
-    print ('Time to download all plugins: %.2f'%( time.time() - st))
+    print ('Time to download all %d plugins: %.2f'%(count, time.time() - st))
 
 

src/calibre/gui2/store/stores/empik_plugin.py

@@ -1,10 +1,10 @@
 # -*- coding: utf-8 -*-
 
 from __future__ import (unicode_literals, division, absolute_import, print_function)
-store_version = 1 # Needed for dynamic plugin loading
+store_version = 2 # Needed for dynamic plugin loading
 
 __license__ = 'GPL 3'
-__copyright__ = '2011-2012, Tomasz Długosz <tomek3d@gmail.com>'
+__copyright__ = '2011-2013, Tomasz Długosz <tomek3d@gmail.com>'
 __docformat__ = 'restructuredtext en'
 
 import re
@@ -51,7 +51,7 @@ class EmpikStore(BasicStoreConfig, StorePlugin):
                 if not id:
                     continue
 
-                cover_url = ''.join(data.xpath('.//div[@class="productBox-450Pic"]/a/img/@src'))
+                cover_url = ''.join(data.xpath('.//div[@class="productBox-450Pic"]/a/img/@data-original'))
                 title = ''.join(data.xpath('.//a[@class="productBox-450Title"]/text()'))
                 title = re.sub(r' \(ebook\)', '', title)
                 author = ''.join(data.xpath('.//div[@class="productBox-450Author"]/a/text()'))

src/calibre/gui2/store/stores/legimi_plugin.py

@@ -1,10 +1,10 @@
 # -*- coding: utf-8 -*-
 
 from __future__ import (unicode_literals, division, absolute_import, print_function)
-store_version = 1 # Needed for dynamic plugin loading
+store_version = 2 # Needed for dynamic plugin loading
 
 __license__ = 'GPL 3'
-__copyright__ = '2011, Tomasz Długosz <tomek3d@gmail.com>'
+__copyright__ = '2011-2013, Tomasz Długosz <tomek3d@gmail.com>'
 __docformat__ = 'restructuredtext en'
 
 import re
@@ -61,8 +61,6 @@ class LegimiStore(BasicStoreConfig, StorePlugin):
                 cover_url = ''.join(data.xpath('.//img[1]/@src'))
                 title = ''.join(data.xpath('.//span[@class="bookListTitle ellipsis"]/text()'))
                 author = ''.join(data.xpath('.//span[@class="bookListAuthor ellipsis"]/text()'))
-                author = re.sub(',','',author)
-                author = re.sub(';',',',author)
                 price = ''.join(data.xpath('.//div[@class="bookListPrice"]/span/text()'))
                 formats = []
                 with closing(br.open(id.strip(), timeout=timeout/4)) as nf:

src/calibre/gui2/store/stores/nexto_plugin.py

@@ -1,10 +1,10 @@
 # -*- coding: utf-8 -*-
 
 from __future__ import (unicode_literals, division, absolute_import, print_function)
-store_version = 1 # Needed for dynamic plugin loading
+store_version = 2 # Needed for dynamic plugin loading
 
 __license__ = 'GPL 3'
-__copyright__ = '2011-2012, Tomasz Długosz <tomek3d@gmail.com>'
+__copyright__ = '2011-2013, Tomasz Długosz <tomek3d@gmail.com>'
 __docformat__ = 'restructuredtext en'
 
 import re
@@ -66,6 +66,8 @@ class NextoStore(BasicStoreConfig, StorePlugin):
                     price = ''.join(data.xpath('.//strong[@class="nprice"]/text()'))
 
                     cover_url = ''.join(data.xpath('.//img[@class="cover"]/@src'))
+                    cover_url = re.sub(r'%2F', '/', cover_url)
+                    cover_url = re.sub(r'\widthMax=120&heightMax=200', 'widthMax=64&heightMax=64', cover_url)
                     title = ''.join(data.xpath('.//a[@class="title"]/text()'))
                     title = re.sub(r' - ebook$', '', title)
                     formats = ', '.join(data.xpath('.//ul[@class="formats_available"]/li//b/text()'))
@@ -80,7 +82,7 @@ class NextoStore(BasicStoreConfig, StorePlugin):
                     counter -= 1
 
                     s = SearchResult()
-                    s.cover_url = cover_url
+                    s.cover_url = 'http://www.nexto.pl' + cover_url
                     s.title = title.strip()
                     s.author = author.strip()
                     s.price = price

src/calibre/gui2/store/stores/virtualo_plugin.py

@@ -1,10 +1,10 @@
 # -*- coding: utf-8 -*-
 
 from __future__ import (unicode_literals, division, absolute_import, print_function)
-store_version = 1 # Needed for dynamic plugin loading
+store_version = 2 # Needed for dynamic plugin loading
 
 __license__ = 'GPL 3'
-__copyright__ = '2011, Tomasz Długosz <tomek3d@gmail.com>'
+__copyright__ = '2011-2013, Tomasz Długosz <tomek3d@gmail.com>'
 __docformat__ = 'restructuredtext en'
 
 import re
@@ -55,10 +55,10 @@ class VirtualoStore(BasicStoreConfig, StorePlugin):
                     continue
 
                 price = ''.join(data.xpath('.//span[@class="price"]/text() | .//span[@class="price abbr"]/text()'))
-                cover_url = ''.join(data.xpath('.//div[@class="list_middle_left"]//a/img/@src'))
+                cover_url = ''.join(data.xpath('.//div[@class="list_middle_left"]//a//img/@src'))
                 title = ''.join(data.xpath('.//div[@class="list_title list_text_left"]/a/text()'))
                 author = ', '.join(data.xpath('.//div[@class="list_authors list_text_left"]/a/text()'))
-                formats = [ form.split('_')[-1].replace('.png', '') for form in data.xpath('.//div[@style="width:55%;float:left;text-align:left;height:18px;"]//img/@src')]
+                formats = [ form.split('_')[-1].replace('.png', '') for form in data.xpath('.//div[@style="width:55%;float:left;text-align:left;height:18px;"]//a/img/@src')]
                 nodrm = no_drm_pattern.search(''.join(data.xpath('.//div[@style="width:45%;float:right;text-align:right;height:18px;"]/div/div/text()')))
 
                 counter -= 1

src/calibre/gui2/viewer/config.py

@@ -82,6 +82,7 @@ def config(defaults=None):
     fonts('default_font_size', default=20, help=_('The standard font size in px'))
     fonts('mono_font_size', default=16, help=_('The monospaced font size in px'))
     fonts('standard_font', default='serif', help=_('The standard font type'))
+    fonts('minimum_font_size', default=8, help=_('The minimum font size in px'))
 
     return c
 
@@ -194,6 +195,7 @@ class ConfigDialog(QDialog, Ui_Dialog):
         self.sans_family.setCurrentFont(QFont(opts.sans_family))
         self.mono_family.setCurrentFont(QFont(opts.mono_family))
         self.default_font_size.setValue(opts.default_font_size)
+        self.minimum_font_size.setValue(opts.minimum_font_size)
         self.mono_font_size.setValue(opts.mono_font_size)
         self.standard_font.setCurrentIndex(
                 {'serif':0, 'sans':1, 'mono':2}[opts.standard_font])
@@ -266,6 +268,7 @@ class ConfigDialog(QDialog, Ui_Dialog):
         c.set('sans_family', unicode(self.sans_family.currentFont().family()))
         c.set('mono_family', unicode(self.mono_family.currentFont().family()))
         c.set('default_font_size', self.default_font_size.value())
+        c.set('minimum_font_size', self.minimum_font_size.value())
         c.set('mono_font_size', self.mono_font_size.value())
         c.set('standard_font', {0:'serif', 1:'sans', 2:'mono'}[
             self.standard_font.currentIndex()])

src/calibre/gui2/viewer/config.ui

@@ -210,6 +210,29 @@ QToolBox::tab:hover {
              </property>
             </widget>
            </item>
+           <item row="7" column="0">
+            <widget class="QLabel" name="label_22">
+             <property name="text">
+              <string>M&amp;inimum font size:</string>
+             </property>
+             <property name="buddy">
+              <cstring>minimum_font_size</cstring>
+             </property>
+            </widget>
+           </item>
+           <item row="7" column="1">
+            <widget class="QSpinBox" name="minimum_font_size">
+             <property name="toolTip">
+              <string>The minimum font size in pixels</string>
+             </property>
+             <property name="suffix">
+              <string> px</string>
+             </property>
+             <property name="minimum">
+              <number>8</number>
+             </property>
+            </widget>
+           </item>
           </layout>
          </widget>
          <widget class="QWidget" name="page_5">
@@ -573,8 +596,8 @@ QToolBox::tab:hover {
            <rect>
             <x>0</x>
             <y>0</y>
-            <width>811</width>
+            <width>352</width>
-            <height>352</height>
+            <height>151</height>
            </rect>
           </property>
           <attribute name="label">

src/calibre/gui2/viewer/documentview.py

@@ -39,8 +39,8 @@ class Document(QWebPage): # {{{
         settings = self.settings()
         settings.setFontSize(QWebSettings.DefaultFontSize, opts.default_font_size)
         settings.setFontSize(QWebSettings.DefaultFixedFontSize, opts.mono_font_size)
-        settings.setFontSize(QWebSettings.MinimumLogicalFontSize, 8)
+        settings.setFontSize(QWebSettings.MinimumLogicalFontSize, opts.minimum_font_size)
-        settings.setFontSize(QWebSettings.MinimumFontSize, 8)
+        settings.setFontSize(QWebSettings.MinimumFontSize, opts.minimum_font_size)
         settings.setFontFamily(QWebSettings.StandardFont, {'serif':opts.serif_family, 'sans':opts.sans_family, 'mono':opts.mono_family}[opts.standard_font])
         settings.setFontFamily(QWebSettings.SerifFont, opts.serif_family)
         settings.setFontFamily(QWebSettings.SansSerifFont, opts.sans_family)
@@ -790,7 +790,8 @@ class DocumentView(QWebView): # {{{
                 self.manager.load_started()
 
         load_html(path, self, codec=getattr(path, 'encoding', 'utf-8'), mime_type=getattr(path,
-            'mime_type', 'text/html'), pre_load_callback=callback)
+            'mime_type', 'text/html'), pre_load_callback=callback,
+                  force_as_html=True)
         entries = set()
         for ie in getattr(path, 'index_entries', []):
             if ie.start_anchor:

src/calibre/gui2/viewer/main.py

@@ -1,7 +1,7 @@
 __license__   = 'GPL v3'
 __copyright__ = '2008, Kovid Goyal <kovid at kovidgoyal.net>'
 
-import traceback, os, sys, functools, collections, textwrap
+import traceback, os, sys, functools, textwrap
 from functools import partial
 from threading import Thread
 
@@ -48,39 +48,57 @@ class Worker(Thread):
             self.exception = err
             self.traceback = traceback.format_exc()
 
-class History(collections.deque):
+class History(list):
 
     def __init__(self, action_back, action_forward):
         self.action_back = action_back
         self.action_forward = action_forward
-        collections.deque.__init__(self)
+        super(History, self).__init__(self)
-        self.pos = 0
+        self.insert_pos = 0
+        self.back_pos = None
+        self.forward_pos = None
         self.set_actions()
 
     def set_actions(self):
-        self.action_back.setDisabled(self.pos < 1)
+        self.action_back.setDisabled(self.back_pos is None)
-        self.action_forward.setDisabled(self.pos + 1 >= len(self))
+        self.action_forward.setDisabled(self.forward_pos is None)
 
     def back(self, from_pos):
-        if self.pos - 1 < 0: return None
+        # Back clicked
-        if self.pos == len(self):
+        if self.back_pos is None:
-            self.append([])
+            return None
-        self[self.pos] = from_pos
+        item = self[self.back_pos]
-        self.pos -= 1
+        self.forward_pos = self.back_pos+1
+        if self.forward_pos >= len(self):
+            self.append(from_pos)
+            self.forward_pos = len(self) - 1
+        self.insert_pos = self.forward_pos
+        self.back_pos = None if self.back_pos == 0 else self.back_pos - 1
         self.set_actions()
-        return self[self.pos]
+        return item
 
-    def forward(self):
+    def forward(self, from_pos):
-        if self.pos + 1 >= len(self): return None
+        if self.forward_pos is None:
-        self.pos += 1
+            return None
+        item = self[self.forward_pos]
+        self.back_pos = self.forward_pos - 1
+        if self.back_pos < 0: self.back_pos = None
+        self.insert_pos = self.back_pos or 0
+        self.forward_pos = None if self.forward_pos > len(self) - 2 else self.forward_pos + 1
         self.set_actions()
-        return self[self.pos]
+        return item
 
     def add(self, item):
-        while len(self) > self.pos+1:
+        self[self.insert_pos:] = []
-            self.pop()
+        while self.insert_pos > 0 and self[self.insert_pos-1] == item:
-        self.append(item)
+            self.insert_pos -= 1
-        self.pos += 1
+            self[self.insert_pos:] = []
+        self.insert(self.insert_pos, item)
+        # The next back must go to item
+        self.back_pos = self.insert_pos
+        self.insert_pos += 1
+        # There can be no forward
+        self.forward_pos = None
         self.set_actions()
 
 class Metadata(QLabel):
@@ -665,7 +683,7 @@ class EbookViewer(MainWindow, Ui_EbookViewer):
         self.goto_page(num)
 
     def forward(self, x):
-        pos = self.history.forward()
+        pos = self.history.forward(self.pos.value())
         if pos is not None:
             self.goto_page(pos)
 

src/calibre/gui2/wizard/__init__.py

@@ -255,17 +255,11 @@ class PocketBookPro912(PocketBook):
 
 class iPhone(Device):
 
-    name = 'iPhone/iTouch'
+    name = 'iPhone/iPad/iPod Touch'
     output_format = 'EPUB'
     manufacturer = 'Apple'
     id = 'iphone'
     supports_color = True
-    output_profile = 'ipad'
-
-class iPad(iPhone):
-
-    name = 'iPad'
-    id = 'ipad'
     output_profile = 'ipad3'
 
 class Android(Device):

src/calibre/gui2/wizard/stanza.ui

@@ -6,8 +6,8 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>400</width>
+    <width>530</width>
-    <height>300</height>
+    <height>316</height>
    </rect>
   </property>
   <property name="windowTitle">
@@ -23,7 +23,7 @@
    <item>
     <widget class="QLabel" name="label">
      <property name="text">
-      <string>&lt;p&gt;If you use the &lt;a href=&quot;http://www.lexcycle.com/download&quot;&gt;Stanza&lt;/a&gt; e-book app on your iPhone/iTouch, you can access your calibre book collection directly on the device. To do this you have to turn on the calibre content server.</string>
+      <string>&lt;p&gt;If you use the &lt;a href=&quot;http://www.lexcycle.com/download&quot;&gt;Stanza&lt;/a&gt; or &lt;a href=&quot;http://marvinapp.com/&quot;&gt;Marvin&lt;/a&gt; e-book reading app on your Apple iDevice, you can access your calibre book collection wirelessly, directly on the device. To do this you have to turn on the calibre content server.</string>
      </property>
      <property name="wordWrap">
       <bool>true</bool>
@@ -70,7 +70,7 @@
     <widget class="QLabel" name="instructions">
      <property name="text">
       <string>&lt;p&gt;Remember to leave calibre running as the server only runs as long as calibre is running.
-&lt;p&gt;Stanza should see your calibre collection automatically. If not, try adding the URL http://myhostname:8080 as a new catalog in the Stanza reader on your iPhone. Here myhostname should be the fully qualified hostname or the IP address of the computer calibre is running on.</string>
+&lt;p&gt;The reader app should see your calibre collection automatically. If not, try adding the URL http://myhostname:8080 as a new catalog in the reader on your iDevice. Here myhostname should be the fully qualified hostname or the IP address of the computer calibre is running on. See &lt;a href=&quot;http://manual.calibre-ebook.com/faq.html#how-do-i-use-app-with-my-ipad-iphone-ipod-touch&quot;&gt;the User Manual&lt;/a&gt; for more information.</string>
      </property>
      <property name="wordWrap">
       <bool>true</bool>

src/calibre/library/coloring.py

@@ -134,8 +134,29 @@ class Rule(object): # {{{
 
     def date_condition(self, col, action, val):
         if action == 'count_days':
-            return (("cmp(add(%s, 1), days_between(today(), format_date(raw_field('%s'), 'yyyy-MM-dd')), '', '1', '1')")
+            return (("test(field('%s'), cmp(%s, "
-                     %(val, col))
+                            "days_between(format_date(today(), 'yyyy-MM-dd'),"
+                            "format_date(raw_field('%s'), 'yyyy-MM-dd')), '', '1', '1'), '')")
+                     %(col, val, col))
+        if action == 'older count days':
+            return (("test(field('%s'), cmp(%s, "
+                            "days_between(format_date(today(), 'yyyy-MM-dd'),"
+                            "format_date(raw_field('%s'), 'yyyy-MM-dd')), '1', '', ''), '')")
+                     %(col, val, col))
+        if action == 'older future days':
+            return (("test(field('%s'), cmp(%s, "
+                            "days_between(format_date(raw_field('%s'), 'yyyy-MM-dd'), "
+                            "format_date(today(), 'yyyy-MM-dd')), '', '1', '1'), '')")
+                     %(col, val, col))
+        if action == 'newer future days':
+            return (("test(field('%s'), cmp(%s, "
+                            "days_between(format_date(raw_field('%s'), 'yyyy-MM-dd'), "
+                            "format_date(today(), 'yyyy-MM-dd')), '1', '', ''), '')")
+                     %(col, val, col))
+        if action == 'is set':
+            return (("test(field('%s'), '1', '')"%(col)))
+        if action == 'is not set':
+            return (("test(field('%s'), '', '1')"%(col)))
         lt, eq, gt = {
                 'eq': ('', '1', ''),
                 'lt': ('1', '', ''),

src/calibre/linux.py

@@ -19,6 +19,7 @@ entry_points = {
              'ebook-device       = calibre.devices.cli:main',
              'ebook-meta         = calibre.ebooks.metadata.cli:main',
              'ebook-convert      = calibre.ebooks.conversion.cli:main',
+             'ebook-polish       = calibre.ebooks.oeb.polish.main:main',
              'markdown-calibre   = calibre.ebooks.markdown.markdown:main',
              'web2disk           = calibre.web.fetch.simple:main',
              'calibre-server     = calibre.library.server.main:main',
@@ -30,7 +31,6 @@ entry_points = {
              'calibre-customize  = calibre.customize.ui:main',
              'calibre-complete   = calibre.utils.complete:main',
              'fetch-ebook-metadata = calibre.ebooks.metadata.sources.cli:main',
-             'epub-fix           = calibre.ebooks.epub.fix.main:main',
              'calibre-smtp = calibre.utils.smtp:main',
         ],
         'gui_scripts'    : [
@@ -226,7 +226,7 @@ class PostInstall:
             from calibre.gui2.main import option_parser as guiop
             from calibre.utils.smtp import option_parser as smtp_op
             from calibre.library.server.main import option_parser as serv_op
-            from calibre.ebooks.epub.fix.main import option_parser as fix_op
+            from calibre.ebooks.oeb.polish.main import option_parser as polish_op, SUPPORTED
             from calibre.ebooks import BOOK_EXTENSIONS
             input_formats = sorted(all_input_formats())
             bc = os.path.join(os.path.dirname(self.opts.staging_sharedir),
@@ -250,13 +250,17 @@ class PostInstall:
                 f.write('# calibre Bash Shell Completion\n')
                 f.write(opts_and_exts('calibre', guiop, BOOK_EXTENSIONS))
                 f.write(opts_and_exts('lrf2lrs', lrf2lrsop, ['lrf']))
-                f.write(opts_and_exts('ebook-meta', metaop, list(meta_filetypes())))
+                f.write(opts_and_exts('ebook-meta', metaop,
+                    list(meta_filetypes()), cover_opts=['--cover', '-c'],
+                    opf_opts=['--to-opf', '--from-opf']))
+                f.write(opts_and_exts('ebook-polish', polish_op,
+                    [x.lower() for x in SUPPORTED], cover_opts=['--cover', '-c'],
+                    opf_opts=['--opf', '-o']))
                 f.write(opts_and_exts('lrfviewer', lrfviewerop, ['lrf']))
                 f.write(opts_and_exts('ebook-viewer', viewer_op, input_formats))
                 f.write(opts_and_words('fetch-ebook-metadata', fem_op, []))
                 f.write(opts_and_words('calibre-smtp', smtp_op, []))
                 f.write(opts_and_words('calibre-server', serv_op, []))
-                f.write(opts_and_exts('epub-fix', fix_op, ['epub']))
                 f.write(textwrap.dedent('''
                 _ebook_device_ls()
                 {
@@ -478,11 +482,23 @@ def opts_and_words(name, op, words):
 complete -F _'''%(opts, words) + fname + ' ' + name +"\n\n").encode('utf-8')
 
 
-def opts_and_exts(name, op, exts):
+def opts_and_exts(name, op, exts, cover_opts=('--cover',), opf_opts=()):
     opts = ' '.join(options(op))
     exts.extend([i.upper() for i in exts])
     exts='|'.join(exts)
     fname = name.replace('-', '_')
+    special_exts_template = '''\
+      %s )
+           _filedir %s
+           return 0
+         ;;
+    '''
+    extras = []
+    for eopts, eexts in ((cover_opts, "${pics}"), (opf_opts, "'@(opf)'")):
+        for opt in eopts:
+            extras.append(special_exts_template%(opt, eexts))
+    extras = '\n'.join(extras)
+
     return '_'+fname+'()'+\
 '''
 {
@@ -490,33 +506,28 @@ def opts_and_exts(name, op, exts):
     COMPREPLY=()
     cur="${COMP_WORDS[COMP_CWORD]}"
     prev="${COMP_WORDS[COMP_CWORD-1]}"
-    opts="%s"
+    opts="%(opts)s"
     pics="@(jpg|jpeg|png|gif|bmp|JPG|JPEG|PNG|GIF|BMP)"
 
     case "${prev}" in
-      --cover )
+%(extras)s
-           _filedir "${pics}"
-           return 0
-           ;;
     esac
 
     case "${cur}" in
-      --cover )
+%(extras)s
-         _filedir "${pics}"
-         return 0
-         ;;
       -* )
          COMPREPLY=( $(compgen -W "${opts}" -- ${cur}) )
          return 0
          ;;
       *  )
-        _filedir '@(%s)'
+        _filedir '@(%(exts)s)'
         return 0
         ;;
     esac
 
 }
-complete -o filenames -F _'''%(opts,exts) + fname + ' ' + name +"\n\n"
+complete -o filenames -F _'''%dict(
+    opts=opts, extras=extras, exts=exts) + fname + ' ' + name +"\n\n"
 
 
 VIEWER = '''\