[FIX] *_online_ponto: log parameters should be a single dict

OCA · Mar 16, 2023 · 6ac2db2 · 6ac2db2
1 parent 34a92dc
commit 6ac2db2
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 11 deletions.
diff --git a/account_statement_import_online_ponto/models/online_bank_statement_provider_ponto.py b/account_statement_import_online_ponto/models/online_bank_statement_provider_ponto.py
@@ -67,9 +67,11 @@ def _ponto_pull(self, date_since, date_until):
                     "Ponto obtain statement data for journal {journal}"
                     " from {date_since} to {date_until}"
                 ),
-                journal=self.journal_id.name,
-                date_since=date_since,
-                date_until=date_until,
+                dict(
+                    journal=self.journal_id.name,
+                    date_since=date_since,
+                    date_until=date_until,
+                ),
             )
         else:
             _logger.debug(

diff --git a/account_statement_import_online_ponto/models/ponto_interface.py b/account_statement_import_online_ponto/models/ponto_interface.py
@@ -35,7 +35,7 @@ def _login(self, username, password):
             "Accept": "application/json",
             "Authorization": "Basic {login}".format(login=login),
         }
-        _logger.debug(_("POST request on {url}"), url=url)
+        _logger.debug(_("POST request on {url}"), dict(url=url))
         response = requests.post(
             url,
             params={"grant_type": "client_credentials"},
@@ -71,7 +71,7 @@ def _get_request_headers(self, access_data):
     def _set_access_account(self, access_data, account_number):
         """Set ponto account for bank account in access_data."""
         url = PONTO_ENDPOINT + "/accounts"
-        _logger.debug(_("GET request on {}"), url)
+        _logger.debug(_("GET request on {url}"), dict(url=url))
         response = requests.get(
             url,
             params={"limit": 100},
@@ -119,8 +119,8 @@ def _get_transactions_from_data(self, data):
         transactions = data.get("data", [])
         if not transactions:
             _logger.debug(
-                _("No transactions where found in data {}"),
-                data,
+                _("No transactions where found in data {data}"),
+                dict(data=data),
             )
         else:
             _logger.debug(
@@ -134,9 +134,11 @@ def _get_request(self, access_data, url, params):
         headers = self._get_request_headers(access_data)
         _logger.debug(
             _("GET request to {url} with headers {headers} and params {params}"),
-            url=url,
-            headers=headers,
-            params=params,
+            dict(
+                url=url,
+                headers=headers,
+                params=params,
+            ),
         )
         response = requests.get(
             url,
@@ -150,7 +152,7 @@ def _get_response_data(self, response):
         """Get response data for GET or POST request."""
         _logger.debug(
             _("HTTP answer code {response_code} from Ponto"),
-            response_code=response.status_code,
+            dict(response_code=response.status_code),
         )
         if response.status_code not in (200, 201):
             raise UserError(