Skip to content

Commit e960537

Browse files
committed
fix: moving direct imports from openedx.core into a wrapper
1 parent 5b76b0b commit e960537

File tree

6 files changed

+28
-10
lines changed

6 files changed

+28
-10
lines changed

backend/openedx_ai_extensions/edxapp_wrapper/__init__.py

Whitespace-only changes.

backend/openedx_ai_extensions/edxapp_wrapper/backends/__init__.py

Whitespace-only changes.
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
""" Backend abstraction. """
2+
from openedx.core.djangoapps import content_libraries # pylint: disable=import-error
3+
4+
5+
def get_content_libraries():
6+
""" Get content_libraries module. """
7+
return content_libraries
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
""" Backend abstraction. """
2+
from importlib import import_module
3+
4+
from django.conf import settings
5+
6+
7+
def get_content_libraries():
8+
""" Get content_libraries module. """
9+
backend_function = settings.CONTENT_LIBRARIES_MODULE_BACKEND
10+
backend = import_module(backend_function)
11+
return backend.get_content_libraries()

backend/openedx_ai_extensions/processors/openedx/content_libraries_processor.py

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@
77
from django.db import transaction
88
from opaque_keys.edx.locator import LibraryLocatorV2
99

10+
from openedx_ai_extensions.edxapp_wrapper.content_libraries_module import get_content_libraries
11+
1012
logger = logging.getLogger(__name__)
1113

1214

@@ -50,9 +52,8 @@ def create_collection_and_add_items(self, items, title, description="") -> str:
5052

5153
def create_block(self, data):
5254
"""Create a library block."""
53-
# pylint: disable=import-error, import-outside-toplevel
54-
from openedx.core.djangoapps.content_libraries import api
55-
from openedx.core.djangoapps.content_libraries.rest_api import serializers
55+
api = get_content_libraries().api
56+
serializers = get_content_libraries().rest_api.serializers
5657

5758
serializer = serializers.LibraryXBlockCreationSerializer(data=data)
5859
serializer.is_valid(raise_exception=True)
@@ -66,15 +67,13 @@ def create_block(self, data):
6667

6768
def modify_block_olx(self, usage_key, data):
6869
"""Modify the OLX of a library block."""
69-
# pylint: disable=import-error, import-outside-toplevel
70-
from openedx.core.djangoapps.content_libraries import api
71-
70+
api = get_content_libraries().api
7271
api.set_library_block_olx(usage_key, data)
7372

7473
def create_collection(self, title, description="") -> None:
7574
"""Create a collection in the library."""
76-
# pylint: disable=import-error, import-outside-toplevel
77-
from openedx.core.djangoapps.content_libraries import api, permissions
75+
api = get_content_libraries().api
76+
permissions = get_content_libraries().permissions
7877

7978
content_library = api.require_permission_for_library_key(
8079
self.library_key, self.user, permissions.CAN_EDIT_THIS_CONTENT_LIBRARY
@@ -93,8 +92,8 @@ def create_collection(self, title, description="") -> None:
9392
return collection
9493

9594
def update_library_collection_items(self, collection_key, item_keys) -> None:
96-
# pylint: disable=import-error, import-outside-toplevel
97-
from openedx.core.djangoapps.content_libraries import api
95+
"""Modifies the list of items in a collection."""
96+
api = get_content_libraries().api
9897

9998
api.update_library_collection_items(
10099
library_key=self.library_key,

backend/openedx_ai_extensions/settings/common.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ def plugin_settings(settings):
1414
Args:
1515
settings (dict): Django settings object
1616
"""
17+
settings.CONTENT_LIBRARIES_MODULE_BACKEND = "openedx_ai_extensions.edxapp_wrapper.backends.content_libraries_module_t_v1"
1718

1819
if not hasattr(settings, "AI_EXTENSIONS_MODEL_PROXY"):
1920
settings.AI_EXTENSIONS_MODEL_PROXY = [

0 commit comments

Comments
 (0)