--- loncom/interface/lonblockingmenu.pm	2016/10/23 17:06:49	1.14.2.3
+++ loncom/interface/lonblockingmenu.pm	2016/10/23 14:35:32	1.22
@@ -2,7 +2,7 @@
 # Routines for configuring blocking of access to collaborative functions, 
 # and specific resources during an exam
 #
-# $Id: lonblockingmenu.pm,v 1.14.2.3 2016/10/23 17:06:49 raeburn Exp $
+# $Id: lonblockingmenu.pm,v 1.22 2016/10/23 14:35:32 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -116,7 +116,7 @@ Outputs: 2
     $readonly - true if modification of blocking events is prohibited.
 
     $allowed  - true if blocking events information can be shown.
-
+ 
 
 =item &get_timed_items()
 
@@ -252,7 +252,7 @@ Inputs: 8
       - $blockcount - number of existing blocking events in course
 
       - $readonly - if true, modification not allowed.
-
+ 
 
 Output: None
 
@@ -345,7 +345,7 @@ Outputs: 1 - $intervalform - web form el
 =item &interval_details()
 
 Creates name/scope of current interval and expandable/collapsible
-showing which interval parameters apply to the current folder/resource
+showing which interval parameters apply to the current folder/resource 
 
 Inputs: 6
 
@@ -362,7 +362,7 @@ Inputs: 6
     - $parmcount - unique ID for current element.
 
 
-Outputs: 2
+Outputs: 2 
 
    - $itemname - name/scope of interval (timer) parameter
 
@@ -435,10 +435,10 @@ containing details about item with timed
 Inputs: none
 
 Output: 1 Javascript (with <script></script> tags) for functions used to:
-          toggle visibility of unordered list for display of detailed
+          toggle visibility of unordered list for display of detailed 
           information about intervals.
-
-=back
+ 
+=back  
 
 =cut
 
@@ -628,8 +628,8 @@ sub handler {
 
 # ------------------------ Choose between modifying existing block or adding new
     $r->print('<form name="blockform" method="post" action="/adm/setblock?action=store">');
-
-    unless ($readonly) {
+    
+    unless ($readonly) { 
         if ($blockcount > 0) {
             $r->print(<<"END");
 <div class="LC_left_float">
@@ -658,19 +658,20 @@ END
         &Apache::loncourserespicker::get_navmap_object($crstype,'examblock');
 
 # --------------------------------------------- Interface for adding a new block
-    unless ($readonly) {
+    if ($readonly) {
+        if (!$blockcount) {
+            $r->print($lt{'ncbc'}.'<br />');
+        }
+    } else {
         &display_addblocker_table($r,$blockcount,\%ltext,\%intervals,
                                   $navmap,$errormsg);
         if ($blockcount > 0) {
             $r->print('</div>');
         }
     }
+
 # ------------------------------------------------ Interface for existing blocks
-    if (!$blockcount) {
-        if ($readonly) {
-            $r->print($lt{'ncbc'}.'<br />');
-        }
-    } else {
+    if ($blockcount > 0) {
         &display_blocker_status($r,\%records,\%ltext,\%intervals,
                                 $navmap,$errormsg,$blockcount,$readonly);
     }
@@ -1436,7 +1437,7 @@ sub create_interval_form {
                                 if (ref($resobj)) {
                                     $title = $resobj->compTitle();
                                 }
-                                if ($title eq '') {
+                                if ($title eq '') {                      
                                     $title = &Apache::lonnet::gettitle($resource);
                                 }
                                 $hierarchy = &show_timer_path($type,$resource,$navmap);