Diff for /loncom/interface/lonpickcourse.pm between versions 1.71 and 1.72

version 1.71, 2008/05/19 21:40:26 version 1.72, 2008/05/31 00:50:00
Line 267  sub display_matched_courses { Line 267  sub display_matched_courses {
                  .'<th>'.&mt('Course Description').'</th>'                   .'<th>'.&mt('Course Description').'</th>'
                  .'<th>'.&mt('Domain').'</th>'                   .'<th>'.&mt('Domain').'</th>'
                  .'<th>'.&mt('Course Code').'</th>'                   .'<th>'.&mt('Course Code').'</th>'
                  .'<th>'.&mt('Owner:Domain').'</th>'                   .'<th>'.&mt('Owner/Co-owner(s)').'</th>'
                  .'<th>'.&mt('Type').'</th>'                   .'<th>'.&mt('Type').'</th>'
                  );                   );
         $r->print(&Apache::loncommon::end_data_table_header_row());          $r->print(&Apache::loncommon::end_data_table_header_row());
     }      }
     foreach my $description (sort { lc($a) cmp lc($b) } (keys(%by_descrip))) {      foreach my $description (sort { lc($a) cmp lc($b) } (keys(%by_descrip))) {
         $r->print(&Apache::loncommon::start_data_table_row());  
         foreach my $course (@{$by_descrip{$description}}) {          foreach my $course (@{$by_descrip{$description}}) {
               $r->print(&Apache::loncommon::start_data_table_row());
             my $cleandesc=&HTML::Entities::encode($description,'<>&"');              my $cleandesc=&HTML::Entities::encode($description,'<>&"');
             $cleandesc=~s/'/\\'/g;              $cleandesc=~s/'/\\'/g;
             my ($cdom,$cnum)=split(/\_/,$course);              my ($cdom,$cnum)=split(/\_/,$course);
Line 294  sub display_matched_courses { Line 294  sub display_matched_courses {
                 ($descr,$instcode,$singleowner,$ttype)=split(/:/,$courses{$course});                  ($descr,$instcode,$singleowner,$ttype)=split(/:/,$courses{$course});
                 push(@owners,&unescape($singleowner));                  push(@owners,&unescape($singleowner));
             }              }
             my $owner = join(', ',@owners);              my $ownerstr = join(', ',@owners);
             $r->print('<td>'.&course_chooser($multiple,$cdom,$cnum,$cleandesc).'</td>');              $r->print('<td>'.&course_chooser($multiple,$cdom,$cnum,$cleandesc).'</td>');
             $r->print('<td>'.$description.'</td>');              $r->print('<td>'.$description.'</td>');
             $r->print('<td>');              $r->print('<td>');
Line 308  sub display_matched_courses { Line 308  sub display_matched_courses {
                 $r->print('&nbsp;');                  $r->print('&nbsp;');
             }              }
             $r->print('</td>');              $r->print('</td>');
             $r->print('<td>');              $r->print('<td>'.$ownerstr.'</td>');
             if (@owners > 1) {  
 #               $r->print(', '.&mt('owners').' - ',join(', ',@owners));   
                 $r->print(join(', ',@owners));   
             } elsif (@owners == 1) {  
 #               $r->print(', '.&mt('owner').' - '.$owner);  
                 $r->print($owner);  
             }  
             $r->print('</td>');  
             $r->print('<td>');              $r->print('<td>');
             if ($ttype ne '') {              if ($ttype ne '') {
                 $r->print(&mt(&unescape($ttype)));                  $r->print(&mt(&unescape($ttype)));
Line 325  sub display_matched_courses { Line 317  sub display_matched_courses {
             }              }
             $r->print('</td>');              $r->print('</td>');
             if ($multiple) { $r->print("</label>\n"); }              if ($multiple) { $r->print("</label>\n"); }
               $r->print(&Apache::loncommon::end_data_table_row());
             # $r->print("<br />\n");              # $r->print("<br />\n");
         }          }
         $r->print(&Apache::loncommon::end_data_table_row());  
     }      }
     $r->print(&Apache::loncommon::end_data_table());      if (%courses) {
           $r->print(&Apache::loncommon::end_data_table());
       }
   
     if (!%courses) {      if (!%courses) {
         $r->print(&mt('None found'));          $r->print(&mt('None found'));
Line 449  sub build_filters { Line 443  sub build_filters {
                 $list->{'instcodefilter'}.'" />';                  $list->{'instcodefilter'}.'" />';
     }      }
     foreach my $posstype ('Course','Non-standard Course') {      foreach my $posstype ('Course','Non-standard Course') {
         $typeselectform.='<option value="'.$posstype.'" "'.          $typeselectform.='<option value="'.$posstype.'"'.
             ($posstype eq $type ? 'selected="selected" ' : ''). ">$posstype</option>\n";              ($posstype eq $type ? ' selected="selected" ' : ''). ">$posstype</option>\n";
     }      }
     $typeselectform.="</select>";      $typeselectform.="</select>";
     my $output = qq|      my $output = qq|

Removed from v.1.71  
changed lines
  Added in v.1.72


FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>