From b5dbd25b62ad929fdbf86bbb6961b14d51cbd831 Mon Sep 17 00:00:00 2001
From: Baptiste Delpey <bdelpey@budget-insight.com>
Date: Thu, 29 Mar 2018 14:40:53 +0200
Subject: [PATCH] factorize state dumping

---
 weboob/tools/backend.py | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/weboob/tools/backend.py b/weboob/tools/backend.py
index 22fe8c05cc..3738a8c620 100644
--- a/weboob/tools/backend.py
+++ b/weboob/tools/backend.py
@@ -282,6 +282,11 @@ def __init__(self, weboob, name, config=None, storage=None, logger=None, nofail=
         self.storage = BackendStorage(self.name, storage)
         self.storage.load(self.STORAGE)
 
+    def dump_state(self):
+        if hasattr(self.browser, 'dump_state'):
+            self.storage.set('browser_state', self.browser.dump_state())
+            self.storage.save()
+
     def deinit(self):
         """
         This abstract method is called when the backend is unloaded.
@@ -289,9 +294,8 @@ def deinit(self):
         if self._browser is None:
             return
 
-        if hasattr(self.browser, 'dump_state'):
-            self.storage.set('browser_state', self.browser.dump_state())
-            self.storage.save()
+        self.dump_state()
+
         if hasattr(self.browser, 'deinit'):
             self.browser.deinit()
 
-- 
GitLab