Diff for /loncom/interface/coursecatalog.pm between versions 1.8 and 1.14

version 1.8, 2006/10/17 18:11:46 version 1.14, 2007/01/06 20:12:19
Line 47  sub handler { Line 47  sub handler {
     my $lonidsdir=$r->dir_config('lonIDsDir');      my $lonidsdir=$r->dir_config('lonIDsDir');
     my $handle;      my $handle;
     if ($lonid) {      if ($lonid) {
         $handle=$lonid->value;   $handle=&LONCAPA::clean_handle($lonid->value);
         $handle=~s/\W//g;  
     }      }
     if ((-e "$lonidsdir/$handle.id") && ($handle ne '')) {      if ((-e "$lonidsdir/$handle.id") && ($handle ne '')) {
         &Apache::lonnet::transfer_profile_to_env($lonidsdir,$handle);          &Apache::lonnet::transfer_profile_to_env($lonidsdir,$handle);
Line 456  sub build_courseinfo_hash { Line 455  sub build_courseinfo_hash {
         }          }
         my $cleandesc=&HTML::Entities::encode($descr,'<>&"');          my $cleandesc=&HTML::Entities::encode($descr,'<>&"');
         $cleandesc=~s/'/\\'/g;          $cleandesc=~s/'/\\'/g;
           $cleandesc =~ s/^\s+//;
         my ($cdom,$cnum)=split(/\_/,$course);          my ($cdom,$cnum)=split(/\_/,$course);
   
         my ($desc,$instcode,$owner,$ttype) = split(/:/,$courses{$course});          my ($desc,$instcode,$owner,$ttype) = split(/:/,$courses{$course});
Line 631  sub get_valid_classes { Line 631  sub get_valid_classes {
     @{$validations{'xlists'}} = ();      @{$validations{'xlists'}} = ();
     my $totalitems = 0;      my $totalitems = 0;
     if ($seclist) {      if ($seclist) {
         foreach my $sec (split(',',$seclist)) {          foreach my $sec (split(/, /,$seclist)) {
             my $class = $crscode.$sec;              my $class = $crscode.$sec;
             if (&Apache::lonnet::auto_validate_class_sec($cdom,$cnum,$owner,              if (&Apache::lonnet::auto_validate_class_sec($cdom,$cnum,$owner,
  $class) eq 'ok') {   $class) eq 'ok') {
Line 643  sub get_valid_classes { Line 643  sub get_valid_classes {
         }          }
     }      }
     if ($xlist_items) {      if ($xlist_items) {
         foreach my $item (split(',',$xlist_items)) {          foreach my $item (split(/, /,$xlist_items)) {
             if (&Apache::lonnet::auto_validate_class_sec($cdom,$cnum,$owner,              if (&Apache::lonnet::auto_validate_class_sec($cdom,$cnum,$owner,
  $item) eq 'ok') {   $item) eq 'ok') {
                 if (!grep(/^\Q$item$\E/,@{$validations{'xlists'}})) {                  if (!grep(/^\Q$item$\E/,@{$validations{'xlists'}})) {
Line 656  sub get_valid_classes { Line 656  sub get_valid_classes {
     if ($totalitems > 0) {      if ($totalitems > 0) {
         if (@{$validations{'sections'}}) {          if (@{$validations{'sections'}}) {
             $response = &mt('Sections: ').              $response = &mt('Sections: ').
                         join(',',@{$validations{'sections'}}).'<br />';                          join(', ',@{$validations{'sections'}}).'<br />';
         }          }
         if (@{$validations{'xlists'}}) {          if (@{$validations{'xlists'}}) {
             $response .= &mt('Courses: ').              $response .= &mt('Courses: ').
                         join(',',@{$validations{'xlists'}});                          join(', ',@{$validations{'xlists'}});
         }          }
     }      }
     return $response;      return $response;
Line 758  function courseSet() { Line 758  function courseSet() {
             }              }
         }          }
         else {          else {
             for (var k=0; k<idyears.length; k++) {              if (selDepartment > -1) {
                 if (idyears[k] == valyr) {                  for (var k=0; k<idyears.length; k++) {
                     idyr = k;                      if (idyears[k] == valyr) {
                           idyr = k;
                       }
                 }                  }
             }                  if (idyr != -1) {
             if (idyr != -1) {                      for (var k=0; k<idsems[idyr].length; k++) {
                 for (var k=0; k<idsems[idyr].length; k++) {                          if (idsems[idyr][k] == valsem) {
                     if (idsems[idyr][k] == valsem) {                              idsem = k;
                         idsem = k;                          }
                     }                      }
                 }                  }
             }                  if (idsem != -1) {
             if (idsem != -1) {                      for (var k=0; k<idcodes[idyr][idsem].length; k++) {
                 for (var k=0; k<idcodes[idyr][idsem].length; k++) {                          if (idcodes[idyr][idsem][k] == valdept) {
                     if (idcodes[idyr][idsem][k] == valdept) {                              iddept = k;
                         iddept = k;                          }
                     }                      }
                 }                  }
             }                  if (iddept != -1) {
             if (iddept != -1) {                      document.$formname.Number.options[0] =  new Option('All','0',false,false);
                 document.$formname.Number.options[0] =  new Option('All','0',false,false);                      for (var i=0; i<idcourses[idyr][idsem][iddept].length; i++) {
                 for (var i=0; i<idcourses[idyr][idsem][iddept].length; i++) {                          var display = idcourses[idyr][idsem][iddept][i];
                     var display = idcourses[idyr][idsem][iddept][i];                          if (longtitles[3] == 1) {
                     if (longtitles[3] == 1) {                              if (idcourseslongs[idyr][idsem][iddept][i] != "") {
                         if (idcourseslongs[idyr][idsem][iddept][i] != "") {                                  display = idcourseslongs[idyr][idsem][iddept][i]
                             display = idcourseslongs[idyr][idsem][iddept][i]                              }
                         }                          }
                           document.$formname.Number.options[i+1] = new Option(display,idcourses[idyr][idsem][iddept][i],false,false)
                     }                      }
                     document.$formname.Number.options[i+1] = new Option(display,idcourses[idyr][idsem][iddept][i],false,false)                  } 
                   else {
                       document.$formname.Number.options[0] =  new Option('No courses','0',true,true);
                 }                  }
             }              } 
             else {              else {
                 document.$formname.Number.options[0] =  new Option('No courses','0',true,true);                  document.$formname.Number.options[0] =  new Option('All','0',true,true);
             }              }
         }          }
         document.$formname.Number.selectedIndex = 0          document.$formname.Number.selectedIndex = 0

Removed from v.1.8  
changed lines
  Added in v.1.14


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