From: W. Trevor King <wking@tremily.us>
Date: Wed, 19 Sep 2012 23:58:51 +0000 (-0400)
Subject: storage: don't try to load empty names.
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=c6125f35473cd31360f2a4473ed64f65ce9b9a8e;p=pygrader.git

storage: don't try to load empty names.

This avoids problems with configuration files like:

  [course]
  ...
  assistants:
  ...

where the `assistants` field exists but lists no names.
---

diff --git a/pygrader/storage.py b/pygrader/storage.py
index b69b798..a5516e9 100644
--- a/pygrader/storage.py
+++ b/pygrader/storage.py
@@ -64,7 +64,9 @@ def load_course(basedir):
     names = {'robot': [config.get('course', 'robot').strip()]}
     for option in ['assignments', 'professors', 'assistants', 'students']:
         names[option] = [
-        a.strip() for a in config.get('course', option).split(',')]
+            a.strip() for a in config.get('course', option).split(',')]
+        while '' in names[option]:
+            names[option].remove('')
     assignments = []
     for assignment in names['assignments']:
         _LOG.debug('loading assignment {}'.format(assignment))