From 115a572fd0c93a9fcd5aef40379e8ea93cef2a13 Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Tue, 10 Jul 2012 09:23:40 -0400 Subject: [PATCH] Set defaults for org and course, and log error messages when they are used --- common/lib/xmodule/xmodule/modulestore/xml.py | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/common/lib/xmodule/xmodule/modulestore/xml.py b/common/lib/xmodule/xmodule/modulestore/xml.py index 96e20aa1be..76056bc069 100644 --- a/common/lib/xmodule/xmodule/modulestore/xml.py +++ b/common/lib/xmodule/xmodule/modulestore/xml.py @@ -57,8 +57,20 @@ class XMLModuleStore(ModuleStore): course_data = etree.parse(course_file).getroot() org = course_data.get('org') + + if org is None: + log.error("No 'org' attribute set for course in {dir}. Using default 'edx'".format(dir=course_dir)) + org = 'edx' + course = course_data.get('course') + if course is None: + log.error("No 'course' attribute set for course in {dir}. Using default '{default}'".format( + dir=course_dir, + default=course_dir + )) + course = course_dir + class ImportSystem(XMLParsingSystem, MakoDescriptorSystem): def __init__(self, modulestore): """