From bbd57ef79b6fbbd8b923c9367c84fe1d18b9ad52 Mon Sep 17 00:00:00 2001
From: Victor Shnayder <victor@mitx.mit.edu>
Date: Fri, 7 Sep 2012 15:10:26 -0400
Subject: [PATCH] don't show empty list if there is no error data

---
 .../courseware/instructor_dashboard.html      | 20 +++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/lms/templates/courseware/instructor_dashboard.html b/lms/templates/courseware/instructor_dashboard.html
index 2d9ab853eb..8568490e5e 100644
--- a/lms/templates/courseware/instructor_dashboard.html
+++ b/lms/templates/courseware/instructor_dashboard.html
@@ -64,17 +64,17 @@ table.stat_table td {
 %if instructor_access:
     <hr width="40%" style="align:left">
     <p>
-    <input type="submit" name="action" value="List course staff members"> 
+    <input type="submit" name="action" value="List course staff members">
     <p>
-    <input type="text" name="staffuser"> <input type="submit" name="action" value="Remove course staff"> 
-    <input type="submit" name="action" value="Add course staff"> 
+    <input type="text" name="staffuser"> <input type="submit" name="action" value="Remove course staff">
+    <input type="submit" name="action" value="Add course staff">
     <hr width="40%" style="align:left">
  %endif
- 
+
 %if settings.MITX_FEATURES['ENABLE_MANUAL_GIT_RELOAD'] and admin_access:
     <p>
-    <input type="submit" name="action" value="Reload course from XML files"> 
-    <input type="submit" name="action" value="GIT pull and Reload course"> 
+    <input type="submit" name="action" value="Reload course from XML files">
+    <input type="submit" name="action" value="GIT pull and Reload course">
 %endif
 
     </form>
@@ -101,7 +101,7 @@ table.stat_table td {
     </p>
 
 %if msg:
-    <p>${msg}</p>    
+    <p>${msg}</p>
 %endif
 
 % if course_errors is not UNDEFINED:
@@ -110,13 +110,17 @@ table.stat_table td {
       <ul>
         % for (summary, err) in course_errors:
         <li>${summary | h}
+        % if err:
           <ul><li><pre>${err | h}</pre></li></ul>
+        % else:
+          <p>&nbsp;</p>
+        % endif
         </li>
         % endfor
         </ul>
       </div>
 % endif
-      
+
   </section>
 </div>
 </section>