--- loncom/interface/coursecatalog.pm	2010/10/07 14:45:04	1.53.4.7
+++ loncom/interface/coursecatalog.pm	2009/11/23 21:45:18	1.57
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler for displaying the course catalog interface
 #
-# $Id: coursecatalog.pm,v 1.53.4.7 2010/10/07 14:45:04 raeburn Exp $
+# $Id: coursecatalog.pm,v 1.57 2009/11/23 21:45:18 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -190,34 +190,17 @@ sub course_details {
                 ({href=>"/adm/coursecatalog",
                   text=>"Course/Community Catalog"});
     }
-    my $brtextone = 'Course listing';
-    my $brtexttwo = 'Course details';
-    if ($env{'form.currcat_0'} eq 'communities::0') {
-        $brtextone = 'Community listing';
-        $brtexttwo = 'Community details';
-    }
     &Apache::lonhtmlcommon::add_breadcrumb
              ({href=>"javascript:document.$formname.submit()",
-              text=>$brtextone},
-             {text=>$brtexttwo});
+              text=>"Course listing"},
+             {text=>"Course details"});
     $r->print(&Apache::lonhtmlcommon::breadcrumbs('Course/Community Catalog'));
-    $r->print('<br />');
-    if ($env{'form.currcat_0'} eq 'communities::0') {
-        $r->print(&mt('Detailed community information:'));
-    } else {
-        $r->print(&mt('Detailed course information:'));
-    }
-    $r->print('<br /><br />'.
+    $r->print('<br />'.&mt('Detailed course information:').'<br /><br />'.
               &print_course_listing($codedom,undef,$trails,$allitems).
               '<br /><br />');
     $r->print('<form name="'.$formname.'" method="post" action="/adm/coursecatalog">'.
-              '<a href = "javascript:document.coursecatalog.submit()">');
-    if ($env{'form.currcat_0'} eq 'communities::0') {
-        $r->print(&mt('Back to community listing'));
-    } else {
-        $r->print(&mt('Back to course listing'));
-    }
-    $r->print('</a>'.
+              '<a href = "javascript:document.coursecatalog.submit()">'.
+              &mt('Back to course listing').'</a>'.
               &Apache::lonhtmlcommon::echo_form_input(['coursenum','catalogfilter',
                                                        'showdetails','courseid']).'</form>');
     return;
@@ -306,24 +289,20 @@ sub instcode_course_selector {
 sub cat_header {
     my ($r,$codedom,$js,$add_entries,$catlinks,$numtitles) = @_;
     my $start_page =
-        &Apache::loncommon::start_page('Course/Community Catalog',$js,
+        &Apache::loncommon::start_page('Other',$js,
                                        {
                                          'add_entries' => $add_entries,
                                          'no_inline_link'   => 1,});
     $r->print($start_page);
-    my $brtext = 'Course listing';
-    if ($env{'form.currcat_0'} eq 'communities::0') {
-        $brtext = 'Community listing';
-    }
     if ($env{'form.state'} eq 'listing') {
         if ($numtitles > 0) {
             &Apache::lonhtmlcommon::add_breadcrumb
                 ({href=>"/adm/coursecatalog",
                   text=>"Course/Community Catalog"},
-                 {text=>$brtext});
+                 {text=>"Course listing"});
         } else {
             &Apache::lonhtmlcommon::add_breadcrumb
-            ({text=>$brtext});
+            ({text=>"Course listing"});
         }
     } else {
         &Apache::lonhtmlcommon::add_breadcrumb
@@ -332,7 +311,7 @@ sub cat_header {
     }
     $r->print(&Apache::lonhtmlcommon::breadcrumbs('Course/Community Catalog'));
     my $onchange;
-    unless (($env{'browser.interface'} eq 'textual') || ($env{'form.interface'} eq 'textual')) {
+    unless ($env{'form.interface'} eq 'textual') {
         $onchange = 'this.form.submit()';
     }
     $r->print('<form name="coursecatdom" method="post" action="/adm/coursecatalog">'.
@@ -355,68 +334,78 @@ sub category_breadcrumbs {
     my $currcat_str = 
         '<input type="hidden" name="catalog_maxdepth" value="'.$deeper.'" />'.
         '<input type="hidden" name="showdom" value="'.$dom.'" />';
-    my $catlinks = '<td valign="top"><b>'.&mt('Catalog:').'</b></td><td><table><tr><td>';
+    my $catlinks = '<td valign="top"><b>'.&mt('Catalog:').'</b></td><td><table><tr>';
     my $has_subcats;
     my $selitem;
-    if (ref($cats[0]) eq 'ARRAY') {
-        if (@{$cats[0]} == 0) {
-            $catlinks .= &mt('No categories defined in this domain'); 
-        } elsif (@{$cats[0]} == 1) {
-            if ($cats[0][0] eq 'instcode') {
-                $catlinks .= &mt('Official courses (with institutional codes)');
-                $env{'form.currcat_0'} = 'instcode::0';
-            } elsif ($cats[0][0] eq 'communities') {
-                $catlinks .= &mt('Communities');
-                $env{'form.currcat_0'} = 'communities::0';
-            } else {
-                my $name = $cats[0][0];
-                my $item = &escape($name).'::0';
-                $catlinks .= $name;
-                $env{'form.currcat_0'} = $item;
-            }
-            $currcat_str .= '<input type="hidden" name="currcat_0" value="'.$env{'form.currcat_0'}.'" />';
-        } else {
-            $catlinks .= &main_category_selector(@cats);
-            if (($env{'form.currcat_0'} ne '') && 
-                ($env{'form.currcat_0'} ne 'instcode::0')) {
-                $catlinks .= $crumbsymbol;
-            } else {
-                $catlinks .= '</td>';
+    for (my $i=0; $i<$deeper; $i++) {
+        $currcat_str .= '<input type="hidden" name="currcat_'.$i.'" value="'.$env{'form.currcat_'.$i}.'" />';
+        my ($cattitle,$shallower);
+        if ($i == 0) {
+            if (ref($cats[0]) eq 'ARRAY') {
+                if (@{$cats[0]} > 1) {
+                    $cattitle = &mt('Main Categories');
+                }
             }
+        } else {
+            $shallower = $i-1;
+            my ($cat,$container,$depth) = map { &unescape($_); } split(/:/,$env{'form.currcat_'.$shallower});
+            $cattitle = $cat;
+        }
+        if ($cattitle ne '') {
+            $catlinks .= '<td valign="top"><a href="javascript:setCatDepth('."'$shallower'".')">'.$cattitle.'</a>'.$crumbsymbol.'</td>';
         }
-    } else {
-        $catlinks .= &mt('Official courses (with institutional codes)');
-                     $env{'form.currcat_0'} = 'instcode::0';
-        $currcat_str .= '<input type="hidden" name="currcat_0" value="'.$env{'form.currcat_0'}.'" />';
     }
-    if ($deeper) {
-        for (my $i=1; $i<=$deeper; $i++) {
-            my $shallower = $i-1;
-            next if ($shallower == 0);
-            my ($cat,$container,$depth) = map { &unescape($_); } split(/:/,$env{'form.currcat_'.$shallower});
-            if ($cat ne '') {
-                $catlinks .= '<td valign="top">'.
-                             '<select name="currcat_'.$shallower.'" onchange="'.
-                             'setCatDepth('."'$shallower'".');this.form.submit();">';
-                if (ref($cats[$shallower]{$container}) eq 'ARRAY') {
-                    $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).':'.&escape($container).':'.$shallower;
-                        my $selected = '';
-                        if ($item eq $env{'form.currcat_'.$shallower}) {
-                            $selected = ' selected="selected"';
-                        }
-                        $catlinks .= 
-                           '<option value="'.$item.'"'.$selected.'>'.$name.'</option>';
+    if ($deeper == 0) {
+        $catlinks .= '<td>';
+        if (ref($cats[0]) eq 'ARRAY') {
+            if ((@{$cats[0]} == 1) && (@cats == 1)) {
+                if ($cats[0][0] eq 'instcode') {
+                    $catlinks .= &mt('Official courses (with institutional codes)').
+                                 '<input type="hidden" name="currcat_0" value="instcode::0" />';
+                    $env{'form.currcat_0'} = 'instcode::0';
+                } elsif ($cats[0][0] eq 'communities') {
+                    $catlinks .= &mt('Communities').
+                                 '<input type="hidden" name="currcat_0" value="instcode::0" />';
+                    $env{'form.currcat_0'} = 'communities::0';
+                } else {
+                    my $name = $cats[0][0];
+                    my $item = &escape($name).'::0';
+                    $catlinks .= $name.
+                             '<input type="hidden" name="currcat_0" value="'.$item.'" />';
+                    $env{'form.currcat_0'} = $item;
+                }
+            } else {
+                $has_subcats = 1;
+                my $buttontext = &mt('Show subcategories');
+                $selitem = 'currcat_0';
+                $catlinks .= '<select name="'.$selitem.'">'."\n";
+                if (@{$cats[0]} > 1) {
+                    $catlinks .= '<option value="" selected="selected">'.&mt('Select').'</option>'."\n";
+                    $buttontext = &mt('Pick main category');
+                }
+                for (my $i=0; $i<@{$cats[0]}; $i++) {
+                    my $name = $cats[0][$i];
+                    my $item = &escape($name).'::0';
+                    $catlinks .= '<option value="'.$item.'">';
+                    if ($name eq 'instcode') {
+                        $catlinks .= &mt('Official courses (with institutional codes)');
+                    } elsif ($name eq 'communities') {
+                        $catlinks .= &mt('Communities');
+                    } else {
+                        $catlinks .= $name;
                     }
+                    $catlinks .= '</option>'."\n";
                 }
-                $catlinks .= '</select>';
+                $catlinks .= '</select>'."\n".
+                             '&nbsp;<input type="submit" name="gocats" value="'.
+                             $buttontext.'" />';
             }
-            unless ($i == $deeper) {
-                $catlinks .= $crumbsymbol;
-            } 
+        } else {
+            $catlinks .= &mt('Official courses (with institutional codes)').
+                         '<input type="hidden" name="currcat_0" value="instcode::0" />';
+            $env{'form.currcat_0'} = 'instcode::0';
         }
+    } else {
         my ($cat,$container,$depth);
         if ($env{'form.currcat_'.$currdepth} eq '') {
             my $shallower = $currdepth - 1;
@@ -424,66 +413,40 @@ sub category_breadcrumbs {
         } else {
             ($cat,$container,$depth) = map { &unescape($_); } split(/:/,$env{'form.currcat_'.$currdepth});
         }
-        my $deeperlevel = $depth +1;
-        if (ref($cats[$deeperlevel]{$cat}) eq 'ARRAY') {
+        my $deeper = $depth +1;
+        my $currcat = $cat;
+        if ($cat eq 'instcode') {
+            $currcat = &mt('Official courses (with institutional codes)');
+        } elsif ($cat eq 'communities') {
+            $currcat = &mt('Communities');
+        }
+        $catlinks .= '<td><b>'.$currcat.'</b>';
+        if (ref($cats[$deeper]{$cat}) eq 'ARRAY') {
             $has_subcats = 1;
             my $buttontext = &mt('Show subcategories');
-            my $selitem = 'currcat_'.$deeperlevel;
-            $catlinks .= '&nbsp;<select name="'.$selitem.'" onchange="this.form.submit()">';
-            if (@{$cats[$deeperlevel]{$cat}}) {
+            $selitem = 'currcat_'.$deeper;
+            $catlinks .= ':&nbsp;<select name="'.$selitem.'">';
+            if (@{$cats[$deeper]{$cat}} > 1) {
                 $catlinks .= '<option value="" selected="selected">'.
-                             &mt('Subcategory ...').'</option>';
+                             &mt('Select').'</option>';
+                $buttontext = &mt('Pick subcategory');
             }
-            for (my $k=0; $k<@{$cats[$deeperlevel]{$cat}}; $k++) {
-                my $name = $cats[$deeperlevel]{$cat}[$k];
-                my $item = &escape($name).':'.&escape($cat).':'.$deeperlevel;
+            for (my $k=0; $k<@{$cats[$deeper]{$cat}}; $k++) {
+                my $name = $cats[$deeper]{$cat}[$k];
+                my $item = &escape($name).':'.&escape($cat).':'.$deeper;
                 $catlinks .= '<option value="'.$item.'">'.$name.'</option>'."\n";
             }
-            $catlinks .= '</select>'."\n";
+            $catlinks .= '</select>'."\n".
+                         '&nbsp;<input type="submit" name="gocats" value="'.
+                         $buttontext.'" />';
         } elsif ($cat ne 'instcode') {
             $catlinks .= '&nbsp;'.&mt('(No subcategories)');
         }
-    } else {
-        $selitem = 'currcat_0';
     }
     $catlinks .= $currcat_str.'</td></tr></table></td>';
     return ($catlinks,$has_subcats,$selitem);
 }
 
-sub main_category_selector {
-    my (@cats) = @_;
-    my $maincatlinks = '<select name="currcat_0" onchange="setCatDepth('."'0'".');this.form.submit();">'."\n";
-    if (ref($cats[0]) eq 'ARRAY') {
-        if (@{$cats[0]} > 1) {
-            my $selected = '';
-            if ($env{'form.currcat_0'} eq '') {
-                $selected = ' selected="selected"';    
-            }
-            $maincatlinks .= 
-                '<option value=""'.$selected.'>'.&mt('Select').'</option>'."\n";
-        }
-        for (my $i=0; $i<@{$cats[0]}; $i++) {
-            my $name = $cats[0][$i];
-            my $item = &escape($name).'::0';
-            my $selected;
-            if ($env{'form.currcat_0'} eq $item) {
-                $selected = ' selected="selected"';
-            }
-            $maincatlinks .= '<option value="'.$item.'"'.$selected.'>';
-            if ($name eq 'instcode') {
-                $maincatlinks .= &mt('Official courses (with institutional codes)');
-            } elsif ($name eq 'communities') {
-                $maincatlinks .= &mt('Communities');
-            } else {
-                $maincatlinks .= $name;
-            }
-            $maincatlinks .= '</option>'."\n";
-        }
-        $maincatlinks .= '</select>'."\n";
-    }
-    return $maincatlinks;
-}
-
 sub get_depth_values {
     my $currdepth = 0;
     my $deeper = 0;
@@ -655,11 +618,7 @@ sub print_course_listing {
                                                  $env{'form.coursenum'},
                                                  undef,undef,'.',1);
         if (keys(%courses) == 0) {
-            if ($env{'form.currcat_0'} eq 'communities::0') {
-                $output .= &mt('The courseID provided does not match a community in this domain.');
-            } else {
-                $output .= &mt('The courseID provided does not match a course in this domain.');
-            }
+            $output .= &mt('The courseID provided does not match a course in this domain.');
             return $output;
         }
     } else {
@@ -728,14 +687,10 @@ sub construct_data_table {
     }
     if ($knownuser) {
         if ($details) {
-            if ($env{'form.currcat_0'} eq 'communities::0') {
-                $output .= '<th>'.&mt('Default Access Dates for Members').'</th>';
-            } else {
-                $output .=
-                    '<th>'.&mt('Default Access Dates for Students').'</th>'.
-                    '<th>'.&mt('Student Counts').'</th>'.
-                    '<th>'.&mt('Auto-enrollment of[_1]registered students','<br />').'</th>';
-            }
+            $output .=
+              '<th>'.&mt('Default Access Dates for Students').'</th>'.
+              '<th>'.&mt('Student Counts').'</th>'.
+              '<th>'.&mt('Auto-enrollment of[_1]registered students','<br />').'</th>';
         } else {
             $output .= '<th>'.&mt('Details').'</th>';
         }
@@ -1004,14 +959,10 @@ sub courseinfo_row {
                '<td>'.$ownerlast.'</td>';
     if ($knownuser) {
         if ($details) {
-            if ($env{'form.currcat_0'} eq 'communities::0') {
-                $output .= '<td>'.$accessdates.'</td>';
-            } else {
-                $output .=
-                    '<td>'.$accessdates.'</td>'.
-                    '<td>'.$counts.'</td>'.
-                    '<td>'.$autoenrollment.'</td>';
-            }
+            $output .=
+               '<td>'.$accessdates.'</td>'. 
+               '<td>'.$counts.'</td>'.
+               '<td>'.$autoenrollment.'</td>';
         } else {
             $output .= "<td><a href=\"javascript:setCourseId('$cnum')\">".&mt('Show more details').'</a></td>';
         }