From 305167cd868b613e91566040e0576f91b1874321 Mon Sep 17 00:00:00 2001 From: Cory Lee Date: Thu, 18 Oct 2018 11:08:54 -0400 Subject: [PATCH] Removing unnecessary log statements --- .../xmodule/xmodule/library_content_module.py | 31 +------------------ .../content/block_structure/store.py | 2 -- 2 files changed, 1 insertion(+), 32 deletions(-) diff --git a/common/lib/xmodule/xmodule/library_content_module.py b/common/lib/xmodule/xmodule/library_content_module.py index ed7bc20a4f..13e505dc8b 100644 --- a/common/lib/xmodule/xmodule/library_content_module.py +++ b/common/lib/xmodule/xmodule/library_content_module.py @@ -185,10 +185,7 @@ class LibraryContentModule(LibraryContentFields, XModule, StudioEditableModule): else: raise NotImplementedError("Unsupported mode.") selected |= added_block_keys - # TODO: used for temporary logging for EDUCATOR-1290 - cls._log_if_mit_supply_chain( - valid_block_keys, selected, invalid_block_keys, overlimit_block_keys, added_block_keys, children - ) + return { 'selected': selected, 'invalid': invalid_block_keys, @@ -196,32 +193,6 @@ class LibraryContentModule(LibraryContentFields, XModule, StudioEditableModule): 'added': added_block_keys, } - @staticmethod - def _log_if_mit_supply_chain( - valid_block_keys, selected, invalid_block_keys, overlimit_block_keys, added_block_keys, children - ): - """ - Helper method to debug case where random block_keys are not assigned for particular courses. - TODO: Delete this before closing EDUCATOR-1290 - """ - if not selected: - return - course_key = '' - if children: - course_key = children[0].course_key - if selected and "MITx+CTL" in text_type(course_key): - logger.info( - "EDUCATOR-1290: LibraryContentModule.make_selection executed for course {0}: " - "valid_block_keys: {1} | selected: {2} | invalid: {3} | overlimit: {4} | added: {5}".format( - text_type(course_key), - valid_block_keys, - selected, - invalid_block_keys, - overlimit_block_keys, - added_block_keys - ) - ) - def _publish_event(self, event_name, result, **kwargs): """ Helper method to publish an event for analytics purposes diff --git a/openedx/core/djangoapps/content/block_structure/store.py b/openedx/core/djangoapps/content/block_structure/store.py index 7974a0d4f1..959a2fb11c 100644 --- a/openedx/core/djangoapps/content/block_structure/store.py +++ b/openedx/core/djangoapps/content/block_structure/store.py @@ -173,8 +173,6 @@ class BlockStructureStore(object): if not serialized_data: logger.info("BlockStructure: Not found in cache; %s.", bs_model) raise BlockStructureNotFound(bs_model.data_usage_key) - else: - logger.info("BlockStructure: Read from cache; %s, size: %d", bs_model, len(serialized_data)) return serialized_data def _get_from_store(self, bs_model):