From d2b078f401ebbabc6154076fb92a4fc5c2a31a64 Mon Sep 17 00:00:00 2001 From: Heinz-Alexander Fuetterer Date: Wed, 15 Nov 2023 15:52:40 +0100 Subject: [PATCH] chore: fix ruff errors (PGH002, UP032) --- rdmo/projects/utils.py | 5 +---- rdmo/services/providers.py | 8 ++++---- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/rdmo/projects/utils.py b/rdmo/projects/utils.py index 8a964f7334..5e5c374d43 100644 --- a/rdmo/projects/utils.py +++ b/rdmo/projects/utils.py @@ -102,10 +102,7 @@ def save_import_snapshot_values(project, snapshots, checked): for value in snapshot.snapshot_values: if value.attribute: - value_key = '{value.attribute.uri}[{snapshot_index}][{value.set_prefix}][{value.set_index}][{value.collection_index}]'.format( # noqa: E501 - value=value, - snapshot_index=snapshot.snapshot_index - ) + value_key = f"{value.attribute.uri}[{snapshot.snapshot_index}][{value.set_prefix}][{value.set_index}][{value.collection_index}]" # noqa: E501 if value_key in checked: # assert that this is a new value diff --git a/rdmo/services/providers.py b/rdmo/services/providers.py index 41b6530224..ea06629425 100644 --- a/rdmo/services/providers.py +++ b/rdmo/services/providers.py @@ -25,14 +25,14 @@ def get(self, request, url): response = requests.get(url, headers=self.get_authorization_headers(access_token)) if response.status_code == 401: - logger.warn('get forbidden: %s (%s)', response.content, response.status_code) + logger.warning('get forbidden: %s (%s)', response.content, response.status_code) else: try: response.raise_for_status() return self.get_success(request, response) except requests.HTTPError: - logger.warn('get error: %s (%s)', response.content, response.status_code) + logger.warning('get error: %s (%s)', response.content, response.status_code) return render(request, 'core/error.html', { 'title': _('OAuth error'), @@ -53,14 +53,14 @@ def post(self, request, url, data): response = requests.post(url, json=data, headers=self.get_authorization_headers(access_token)) if response.status_code == 401: - logger.warn('post forbidden: %s (%s)', response.content, response.status_code) + logger.warning('post forbidden: %s (%s)', response.content, response.status_code) else: try: response.raise_for_status() return self.post_success(request, response) except requests.HTTPError: - logger.warn('post error: %s (%s)', response.content, response.status_code) + logger.warning('post error: %s (%s)', response.content, response.status_code) return render(request, 'core/error.html', { 'title': _('OAuth error'),