From 2ce5a6ceedadbcf4f82a3ae5ca0107a9355bdaa8 Mon Sep 17 00:00:00 2001 From: stv Date: Sat, 8 Nov 2014 18:13:29 -0800 Subject: [PATCH] Fix PEP8: E125 continuation line does not distinguish itself from next logical line --- common/lib/capa/capa/capa_problem.py | 2 +- common/lib/sandbox-packages/verifiers/draganddrop.py | 2 +- common/lib/xmodule/xmodule/errortracker.py | 2 +- common/lib/xmodule/xmodule/x_module.py | 2 +- common/xml_cleanup.py | 2 +- lms/djangoapps/courseware/access.py | 2 +- lms/djangoapps/open_ended_grading/open_ended_notifications.py | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/common/lib/capa/capa/capa_problem.py b/common/lib/capa/capa/capa_problem.py index a854e1109f..7b3dbbe79f 100644 --- a/common/lib/capa/capa/capa_problem.py +++ b/common/lib/capa/capa/capa_problem.py @@ -694,7 +694,7 @@ class LoncapaProblem(object): return if (problemtree.tag == 'script' and problemtree.get('type') - and 'javascript' in problemtree.get('type')): + and 'javascript' in problemtree.get('type')): # leave javascript intact. return deepcopy(problemtree) diff --git a/common/lib/sandbox-packages/verifiers/draganddrop.py b/common/lib/sandbox-packages/verifiers/draganddrop.py index 28ac146d4f..66c3835be9 100644 --- a/common/lib/sandbox-packages/verifiers/draganddrop.py +++ b/common/lib/sandbox-packages/verifiers/draganddrop.py @@ -98,7 +98,7 @@ class PositionsCompare(list): return False if (isinstance(self[0], (list, int, float)) and - isinstance(other[0], (list, int, float))): + isinstance(other[0], (list, int, float))): return self.coordinate_positions_compare(other) elif (isinstance(self[0], (unicode, str)) and diff --git a/common/lib/xmodule/xmodule/errortracker.py b/common/lib/xmodule/xmodule/errortracker.py index 80e6d288f8..e39604805a 100644 --- a/common/lib/xmodule/xmodule/errortracker.py +++ b/common/lib/xmodule/xmodule/errortracker.py @@ -39,7 +39,7 @@ def make_error_tracker(): # don't display irrelevant gunicorn sync error if (('python2.7/site-packages/gunicorn/workers/sync.py' in exc_str) and - ('[Errno 11] Resource temporarily unavailable' in exc_str)): + ('[Errno 11] Resource temporarily unavailable' in exc_str)): exc_str = '' errors.append((msg, exc_str)) diff --git a/common/lib/xmodule/xmodule/x_module.py b/common/lib/xmodule/xmodule/x_module.py index 3a7d5699d1..5332296e15 100644 --- a/common/lib/xmodule/xmodule/x_module.py +++ b/common/lib/xmodule/xmodule/x_module.py @@ -202,7 +202,7 @@ class XModuleMixin(XBlockMixin): # an XModule context, not an XModuleDescriptor context, # so we should use the xmodule_runtime (ModuleSystem) as the runtime. if (not isinstance(self, (XModule, XModuleDescriptor)) and - self.xmodule_runtime is not None): + self.xmodule_runtime is not None): return PureSystem(self.xmodule_runtime, self._runtime) else: return self._runtime diff --git a/common/xml_cleanup.py b/common/xml_cleanup.py index 9fc8fd00f6..050189a7a3 100755 --- a/common/xml_cleanup.py +++ b/common/xml_cleanup.py @@ -76,7 +76,7 @@ def cleanup(filepath, remove_meta): print "WARNING: {0} has both slug and url_name".format(node) if ('url_name' in attrs and 'filename' in attrs and - len(attrs) == 2 and attrs['url_name'] == attrs['filename']): + len(attrs) == 2 and attrs['url_name'] == attrs['filename']): # This is a pointer tag in disguise. Get rid of the filename. print 'turning {0}.{1} into a pointer tag'.format(node.tag, attrs['url_name']) del attrs['filename'] diff --git a/lms/djangoapps/courseware/access.py b/lms/djangoapps/courseware/access.py index 93bcbf7c1f..4dae9f3161 100644 --- a/lms/djangoapps/courseware/access.py +++ b/lms/djangoapps/courseware/access.py @@ -154,7 +154,7 @@ def _has_access_course_desc(user, action, course): # if using registration method to restrict (say shibboleth) if settings.FEATURES.get('RESTRICT_ENROLL_BY_REG_METHOD') and course.enrollment_domain: if user is not None and user.is_authenticated() and \ - ExternalAuthMap.objects.filter(user=user, external_domain=course.enrollment_domain): + ExternalAuthMap.objects.filter(user=user, external_domain=course.enrollment_domain): debug("Allow: external_auth of " + course.enrollment_domain) reg_method_ok = True else: diff --git a/lms/djangoapps/open_ended_grading/open_ended_notifications.py b/lms/djangoapps/open_ended_grading/open_ended_notifications.py index 11993e15be..bfa9e337bd 100644 --- a/lms/djangoapps/open_ended_grading/open_ended_notifications.py +++ b/lms/djangoapps/open_ended_grading/open_ended_notifications.py @@ -162,7 +162,7 @@ def combined_notifications(course, user): last_time_viewed) if notifications.get('success'): if (notifications.get('staff_needs_to_grade') or - notifications.get('student_needs_to_peer_grade')): + notifications.get('student_needs_to_peer_grade')): pending_grading = True except: #Non catastrophic error, so no real action