From c6443b97259662624c8f2771e5146724a43450ac Mon Sep 17 00:00:00 2001
From: Romain Bignon <romain@peerfuse.org>
Date: Wed, 11 May 2011 16:50:07 +0200
Subject: [PATCH] fix when lefigaro's feed is wrong

---
 weboob/backends/lefigaro/browser.py                  | 7 ++++++-
 weboob/tools/capabilities/messages/GenericBackend.py | 6 +++++-
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/weboob/backends/lefigaro/browser.py b/weboob/backends/lefigaro/browser.py
index aec76028..d55df381 100644
--- a/weboob/backends/lefigaro/browser.py
+++ b/weboob/backends/lefigaro/browser.py
@@ -20,8 +20,10 @@
 
 from .pages.article import ArticlePage
 from .pages.flashactu import FlashActuPage
-from weboob.tools.browser import BaseBrowser
+from weboob.tools.browser import BaseBrowser, BasePage
 
+class IndexPage(BasePage):
+    pass
 
 
 class NewspaperFigaroBrowser(BaseBrowser):
@@ -30,6 +32,7 @@ class NewspaperFigaroBrowser(BaseBrowser):
              "http://www.lefigaro.fr/flash-.*/(\d{4})/(\d{2})/(\d{2})/(.*$)": FlashActuPage,
              "http://www.lefigaro.fr/bd/(\d{4})/(\d{2})/(\d{2})/(.*$)": FlashActuPage,
              "http://www.lefigaro.fr/(?!flash-|bd).+/(\d{4})/(\d{2})/(\d{2})/(.*$)": ArticlePage,
+             "http://www.lefigaro.fr/": IndexPage,
             }
 
     def is_logged(self):
@@ -44,4 +47,6 @@ class NewspaperFigaroBrowser(BaseBrowser):
     def get_content(self, _id):
         "return page article content"
         self.location(_id)
+        if self.is_on_page(IndexPage):
+            return None
         return self.page.get_article(_id)
diff --git a/weboob/tools/capabilities/messages/GenericBackend.py b/weboob/tools/capabilities/messages/GenericBackend.py
index bbca756b..8a4458c4 100644
--- a/weboob/tools/capabilities/messages/GenericBackend.py
+++ b/weboob/tools/capabilities/messages/GenericBackend.py
@@ -44,6 +44,9 @@ class GenericNewspaperBackend(BaseBackend, ICapMessages):
         with self.browser:
             content = self.browser.get_content(_id)
 
+        if content is None:
+            return None
+
         if not thread:
             thread = Thread(_id)
 
@@ -77,7 +80,8 @@ class GenericNewspaperBackend(BaseBackend, ICapMessages):
 
     def fill_thread(self, thread, fields):
         "fill the thread"
-        return self.get_thread(thread)
+        t = self.get_thread(thread)
+        return t or thread
 
     def iter_unread_messages(self, thread=None):
         for thread in self.iter_threads():