Commit 57909ce1 by Ned Batchelder

Fix all W0602, global used but no assignment done.

parent 75b39012
...@@ -37,7 +37,6 @@ rate -- messages per second ...@@ -37,7 +37,6 @@ rate -- messages per second
self.log_file.write(datetime.datetime.utcnow().isoformat() + ' -- ' + text + '\n') self.log_file.write(datetime.datetime.utcnow().isoformat() + ' -- ' + text + '\n')
def handle(self, *args, **options): def handle(self, *args, **options):
global log_file
(user_file, message_base, logfilename, ratestr) = args (user_file, message_base, logfilename, ratestr) = args
users = [u.strip() for u in open(user_file).readlines()] users = [u.strip() for u in open(user_file).readlines()]
......
...@@ -18,8 +18,6 @@ def load_function(path): ...@@ -18,8 +18,6 @@ def load_function(path):
def contentstore(name='default'): def contentstore(name='default'):
global _CONTENTSTORE
if name not in _CONTENTSTORE: if name not in _CONTENTSTORE:
class_ = load_function(settings.CONTENTSTORE['ENGINE']) class_ = load_function(settings.CONTENTSTORE['ENGINE'])
options = {} options = {}
......
...@@ -26,8 +26,6 @@ def load_function(path): ...@@ -26,8 +26,6 @@ def load_function(path):
def modulestore(name='default'): def modulestore(name='default'):
global _MODULESTORES
if name not in _MODULESTORES: if name not in _MODULESTORES:
class_ = load_function(settings.MODULESTORE[name]['ENGINE']) class_ = load_function(settings.MODULESTORE[name]['ENGINE'])
......
...@@ -73,21 +73,17 @@ def get_discussion_id_map(course): ...@@ -73,21 +73,17 @@ def get_discussion_id_map(course):
""" """
return a dict of the form {category: modules} return a dict of the form {category: modules}
""" """
global _DISCUSSIONINFO
initialize_discussion_info(course) initialize_discussion_info(course)
return _DISCUSSIONINFO[course.id]['id_map'] return _DISCUSSIONINFO[course.id]['id_map']
def get_discussion_title(course, discussion_id): def get_discussion_title(course, discussion_id):
global _DISCUSSIONINFO
initialize_discussion_info(course) initialize_discussion_info(course)
title = _DISCUSSIONINFO[course.id]['id_map'].get(discussion_id, {}).get('title', '(no title)') title = _DISCUSSIONINFO[course.id]['id_map'].get(discussion_id, {}).get('title', '(no title)')
return title return title
def get_discussion_category_map(course): def get_discussion_category_map(course):
global _DISCUSSIONINFO
initialize_discussion_info(course) initialize_discussion_info(course)
return filter_unstarted_categories(_DISCUSSIONINFO[course.id]['category_map']) return filter_unstarted_categories(_DISCUSSIONINFO[course.id]['category_map'])
...@@ -141,8 +137,6 @@ def sort_map_entries(category_map): ...@@ -141,8 +137,6 @@ def sort_map_entries(category_map):
def initialize_discussion_info(course): def initialize_discussion_info(course):
global _DISCUSSIONINFO
course_id = course.id course_id = course.id
discussion_id_map = {} discussion_id_map = {}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment