From bbd7b66413ef6b0c128e71d7f3dc49b1942afc4a Mon Sep 17 00:00:00 2001 From: Romain Bignon Date: Thu, 18 Nov 2010 12:23:36 +0100 Subject: [PATCH] don't need to logout from dlfp if the browser hasn't been used --- weboob/backends/dlfp/backend.py | 4 ++++ weboob/tools/backend.py | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/weboob/backends/dlfp/backend.py b/weboob/backends/dlfp/backend.py index 3ca5b5ce..976815d0 100644 --- a/weboob/backends/dlfp/backend.py +++ b/weboob/backends/dlfp/backend.py @@ -50,6 +50,10 @@ class DLFPBackend(BaseBackend, ICapMessages, ICapMessagesPost): return self.create_browser(self.config['username'], self.config['password']) def deinit(self): + # don't need to logout if the browser hasn't been used. + if not self._browser: + return + with self.browser: self.browser.close_session() diff --git a/weboob/tools/backend.py b/weboob/tools/backend.py index 89ef09e5..364ae94b 100644 --- a/weboob/tools/backend.py +++ b/weboob/tools/backend.py @@ -152,6 +152,8 @@ class BaseBackend(object): else: return xdg.IconTheme.getIconPath(klass.NAME) + _browser = None + @property def browser(self): """ @@ -160,7 +162,7 @@ class BaseBackend(object): Note that the 'create_default_browser' method is called to create it. """ - if not hasattr(self, '_browser'): + if self._browser is None: self._browser = self.create_default_browser() return self._browser