--- loncom/interface/coursecatalog.pm	2010/09/07 03:00:38	1.64
+++ loncom/interface/coursecatalog.pm	2012/10/04 13:41:10	1.71
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler for displaying the course catalog interface
 #
-# $Id: coursecatalog.pm,v 1.64 2010/09/07 03:00:38 raeburn Exp $
+# $Id: coursecatalog.pm,v 1.71 2012/10/04 13:41:10 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -283,7 +283,6 @@ sub instcode_course_selector {
         $r->print('<input type="hidden" name="coursenum" value="" />'."\n".
                   '<input type="hidden" name="sortby" value="" />'."\n".
                   '<input type="hidden" name="state" value="listing" />'."\n".
-                  '<input type="hidden" name="form.currcat_0" value="instcode::0" />'."\n".
                   '<input type="submit" name="catalogfilter" value="'.
                   &mt('Display courses').'" />'.
                   '<input type="hidden" name="numtitles" value="'.$numtitles.
@@ -395,7 +394,7 @@ sub category_breadcrumbs {
                     $catlinks .= '<option value="">'.&mt('De-select').'</option>';
                     for (my $j=0; $j<@{$cats[$shallower]{$container}}; $j++) {
                         my $name = $cats[$shallower]{$container}[$j];
-                        my $item = &escape($name).':'.$container.':'.$shallower;
+                        my $item = &escape($name).':'.&escape($container).':'.$shallower;
                         my $selected = '';
                         if ($item eq $env{'form.currcat_'.$shallower}) {
                             $selected = ' selected="selected"';
@@ -523,9 +522,9 @@ sub additional_filters {
     }
     my $selfenroll_text;
     if ($env{'form.currcat_0'} eq 'communities::0') {
-        $selfenroll_text = &mt('Only show communities which allow self-enrollment');
+        $selfenroll_text = &mt('Only show communities which currently allow self-enrollment (or will allow it in the future)');
     } else {
-        $selfenroll_text = &mt('Only show courses which allow self-enrollment');
+        $selfenroll_text = &mt('Only show courses which currently allow self-enrollment (or will allow it in the future)');
     }
     $output .= '<tr><td>'.
                '<label><input type="checkbox" name="showselfenroll" value="1" '.
@@ -663,9 +662,15 @@ sub print_course_listing {
         }
         if (keys(%courses) == 0) {
             if ($env{'form.currcat_0'} eq 'communities::0') {
-                $output = &mt('No communities match the criteria you selected.');
+                $output =
+                    '<p class="LC_info">'
+                   .&mt('No communities match the criteria you selected.')
+                   .'</p>';
             } else {
-                $output = &mt('No courses match the criteria you selected.');
+                $output =
+                    '<p class="LC_info">'
+                   .&mt('No courses match the criteria you selected.')
+                   .'</p>';
             }
             return $output;
         }
@@ -694,7 +699,7 @@ sub construct_data_table {
         $sortname{'Code'} = 'code';
         $sortname{'Categories'} = 'cats';
         $sortname{'Title'} = 'title';
-        $sortname{'Owner(s)'} = 'owner';
+        $sortname{'Owner & Co-owner(s)'} = 'owner';
     }
     my $output = &Apache::loncommon::start_data_table().
                  &Apache::loncommon::start_data_table_header_row();
@@ -704,7 +709,7 @@ sub construct_data_table {
     } else {
         push(@coltitles,'Categories');
     }
-    push(@coltitles,('Sections','Crosslisted','Title','Owner(s)'));
+    push(@coltitles,('Sections','Crosslisted','Title','Owner & Co-owner(s)'));
     if (ref($usersections) eq 'HASH') {
        $coltitles[1] = 'Your Section';
     }
@@ -799,8 +804,8 @@ sub build_courseinfo_hash {
             $selfenroll_end = $courses->{$course}{'selfenroll_end_date'};
             $categories = $courses->{$course}{'categories'};
             push(@owners,$singleowner);
-            if (ref($courses->{$course}{'co-owners'}) eq 'ARRAY') {
-                foreach my $item (@{$courses->{$course}{'co-owners'}}) {
+            if ($courses->{$course}{'co-owners'} ne '') {
+                foreach my $item (split(/,/,$courses->{$course}{'co-owners'})) {
                     push(@owners,$item);
                 }
             }
@@ -1017,7 +1022,13 @@ sub courseinfo_row {
             if (($info->{'selfenroll_start'} > 0) && ($info->{'selfenroll_start'} > $now)) {
                 $output .= '<td>'.&mt('Starts: [_1]','<span class="LC_cusr_emph">'.$showstart.'</span>').'<br />'.&mt('Ends: [_1]','<span class="LC_cusr_emph">'.$showend.'</span>').'</td>';
             } else { 
-                $output .= '<td><a href="javascript:ToSelfenroll('."'$course'".')">'.&mt('Enroll in course').'</a></td>';
+                $output .= '<td><a href="javascript:ToSelfenroll('."'$course'".')">'.&mt('Enroll in course').'</a><br />';
+                if ($info->{'selfenroll_end'} == 0) {
+                    $output .= &mt('Available permanently');
+                } elsif ($info->{'selfenroll_end'} > $now) {
+                    $output .= &mt('Self-enrollment ends: [_1]','<span class="LC_cusr_emph">'.$showend.'</span>');
+                }
+                $output .= '</td>';
             }
             $selfenroll = 1;
         }