style: pylint fixes

openedx · Nov 25, 2023 · b4a4c96 · b4a4c96
1 parent 42c2037
commit b4a4c96
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 3 deletions.
diff --git a/cms/djangoapps/contentstore/utils.py b/cms/djangoapps/contentstore/utils.py
@@ -1059,7 +1059,12 @@ def duplicate_block(
 
         if not children_handled:
             handle_children_duplication(
-                dest_block, store, source_item, store, user, duplication_function=duplicate_block, shallow=shallow
+                dest_block,
+                source_item,
+                store,
+                user,
+                duplication_function=duplicate_block,
+                shallow=shallow,
             )
 
         # pylint: disable=protected-access

diff --git a/cms/djangoapps/contentstore/xblock_storage_handlers/view_handlers.py b/cms/djangoapps/contentstore/xblock_storage_handlers/view_handlers.py
@@ -54,7 +54,6 @@
 from openedx.core.lib.cache_utils import request_cached
 from openedx.core.toggles import ENTRANCE_EXAMS
 from xmodule.course_block import DEFAULT_START_DATE
-from xmodule.library_tools import LibraryToolsService
 from xmodule.modulestore import EdxJSONEncoder, ModuleStoreEnum
 from xmodule.modulestore.django import modulestore
 from xmodule.modulestore.draft_and_published import DIRECT_ONLY_CATEGORIES

diff --git a/xmodule/library_content_block.py b/xmodule/library_content_block.py
@@ -8,7 +8,6 @@
 import random
 from copy import copy
 from gettext import ngettext, gettext
-from typing import Callable
 
 import bleach
 from django.conf import settings

diff --git a/xmodule/util/duplicate.py b/xmodule/util/duplicate.py
@@ -1,9 +1,13 @@
+"""
+Utility to aid xblock duplication
+"""
 from typing import Callable
 
 
 def handle_children_duplication(
     xblock, source_item, store, user, duplication_function: Callable[..., None], shallow: bool
 ):
+    """Implements default behaviour to handle children duplication."""
     if not source_item.has_children or shallow:
         return