Skip to content

Commit 2a2a193

Browse files
committed
chore: fixed pycodestile suggestions
1 parent 0148696 commit 2a2a193

4 files changed

Lines changed: 9 additions & 5 deletions

File tree

openedx/core/djangoapps/content_libraries/api/libraries.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -716,7 +716,7 @@ def get_backup_task_status(library_key: LibraryLocatorV2, user_id: int | None =
716716

717717
if not task_status:
718718
return None
719-
719+
720720
result = {'state': task_status.state, 'url': None}
721721

722722
if task_status.state == UserTaskStatus.SUCCEEDED:
@@ -732,4 +732,4 @@ def get_backup_task_status(library_key: LibraryLocatorV2, user_id: int | None =
732732
else:
733733
result['url'] = artifact.file.storage.url(artifact.file.name)
734734

735-
return result
735+
return result

openedx/core/djangoapps/content_libraries/rest_api/libraries.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -761,7 +761,7 @@ def post(self, request, lib_key_str):
761761
result = {'task_id': async_result.task_id}
762762

763763
return Response(LibraryBackupResponseSerializer(result).data)
764-
764+
765765
@apidocs.schema(
766766
parameters=[
767767
apidocs.query_parameter(

openedx/core/djangoapps/content_libraries/rest_api/serializers.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -417,12 +417,14 @@ class ContainerHierarchySerializer(serializers.Serializer):
417417
components = serializers.ListField(child=ContainerHierarchyMemberSerializer(), allow_empty=True)
418418
object_key = OpaqueKeySerializer()
419419

420+
420421
class LibraryBackupResponseSerializer(serializers.Serializer):
421422
"""
422423
Serializer for the response after requesting a backup of a content library.
423424
"""
424425
task_id = serializers.CharField()
425426

427+
426428
class LibraryBackupTaskStatusSerializer(serializers.Serializer):
427429
"""
428430
Serializer for checking the status of a library backup task.

openedx/core/djangoapps/content_libraries/tasks.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -484,6 +484,7 @@ def _copy_overrides(
484484
)
485485
store.update_item(dest_block, user_id)
486486

487+
487488
class LibraryBackupTask(UserTask): # pylint: disable=abstract-method
488489
"""
489490
Base class for tasks related with Library backup functionality.
@@ -507,6 +508,7 @@ def generate_name(cls, arguments_dict) -> str:
507508
key = arguments_dict['library_key_str']
508509
return f'Export of {key}'
509510

511+
510512
@shared_task(base=LibraryBackupTask, bind=True)
511513
# Note: The decorator @set_code_owner_attribute cannot be used here because the UserTaskMixin
512514
# does stack inspection and can't handle additional decorators.
@@ -522,7 +524,7 @@ def backup_library(self, user_id: int, library_key_str: str) -> None:
522524
ensure_cms("backup_library may only be executed in a CMS context")
523525
set_code_owner_attribute_from_module(__name__)
524526
library_key = LibraryLocatorV2.from_string(library_key_str)
525-
527+
526528
try:
527529
self.status.set_state('Exporting')
528530
set_custom_attribute("exporting_started", str(library_key))
@@ -541,4 +543,4 @@ def backup_library(self, user_id: int, library_key_str: str) -> None:
541543
TASK_LOGGER.exception('Error exporting course %s', library_key, exc_info=True)
542544
if self.status.state != UserTaskStatus.FAILED:
543545
self.status.fail({'raw_error_msg': str(exception)})
544-
return
546+
return

0 commit comments

Comments
 (0)