From cf715cb7278405bd8294c541b41951919d1d1d46 Mon Sep 17 00:00:00 2001 From: Chris Dodge Date: Thu, 15 Aug 2013 12:48:01 -0400 Subject: [PATCH] fix pylint violations --- common/lib/xmodule/xmodule/contentstore/content.py | 6 +++++- common/lib/xmodule/xmodule/modulestore/__init__.py | 6 ++++++ lms/envs/cms/mixed_dev.py | 6 +++++- 3 files changed, 16 insertions(+), 2 deletions(-) diff --git a/common/lib/xmodule/xmodule/contentstore/content.py b/common/lib/xmodule/xmodule/contentstore/content.py index cdecfd1af5..9d767482d5 100644 --- a/common/lib/xmodule/xmodule/contentstore/content.py +++ b/common/lib/xmodule/xmodule/contentstore/content.py @@ -102,7 +102,11 @@ class StaticContent(object): @staticmethod def convert_legacy_static_url_with_course_id(path, course_id): - org, course_num, run = course_id.split("/") + """ + Returns a path to a piece of static content when we are provided with a filepath and + a course_id + """ + org, course_num, __ = course_id.split("/") loc = StaticContent.compute_location(org, course_num, path) return StaticContent.get_url_path_from_location(loc) diff --git a/common/lib/xmodule/xmodule/modulestore/__init__.py b/common/lib/xmodule/xmodule/modulestore/__init__.py index eb9c44a441..707390d759 100644 --- a/common/lib/xmodule/xmodule/modulestore/__init__.py +++ b/common/lib/xmodule/xmodule/modulestore/__init__.py @@ -457,10 +457,16 @@ class ModuleStoreBase(ModuleStore): @property def metadata_inheritance_cache_subsystem(self): + """ + Exposes an accessor to the runtime configuration for the metadata inheritance cache + """ return self.modulestore_configuration.get('metadata_inheritance_cache_subsystem', None) @property def request_cache(self): + """ + Exposes an accessor to the runtime configuration for the request cache + """ return self.modulestore_configuration.get('request_cache', None) def set_modulestore_configuration(self, config_dict): diff --git a/lms/envs/cms/mixed_dev.py b/lms/envs/cms/mixed_dev.py index 9a1a3a4cb8..88e1f3d1f8 100644 --- a/lms/envs/cms/mixed_dev.py +++ b/lms/envs/cms/mixed_dev.py @@ -2,7 +2,11 @@ This configuration is to run the MixedModuleStore on a localdev environment """ -from .dev import * +# We intentionally define lots of variables that aren't used, and +# want to import all variables from base settings files +# pylint: disable=W0401, W0614 + +from .dev import *, DATA_DIR MODULESTORE = { 'default': {