--- loncom/interface/lonsupportreq.pm	2004/12/30 21:16:38	1.20
+++ loncom/interface/lonsupportreq.pm	2005/10/13 17:18:14	1.29
@@ -1,12 +1,39 @@
+#
+# $Id: lonsupportreq.pm,v 1.29 2005/10/13 17:18:14 raeburn Exp $
+#
+# Copyright Michigan State University Board of Trustees
+#
+# This file is part of the LearningOnline Network with CAPA (LON-CAPA).
+#
+# LON-CAPA is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# LON-CAPA is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with LON-CAPA; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+#
+# /home/httpd/html/adm/gpl.txt
+#
+# http://www.lon-capa.org/
+#
+
 package Apache::lonsupportreq;
 
 use strict;
 use lib qw(/home/httpd/lib/perl);
 use MIME::Types;
 use MIME::Lite;
+use CGI::Cookie();
 use Apache::Constants qw(:common);
 use Apache::loncommon();
-use Apache::lonnet();
+use Apache::lonnet;
 use Apache::lonlocal;
 
 sub handler {
@@ -21,9 +48,9 @@ sub handler {
     if ($r->uri eq '/adm/helpdesk') {
         &Apache::loncommon::get_posted_cgi($r);
     }
-    my $function = $ENV{'form.function'};
-    my $origurl = &Apache::lonnet::unescape($ENV{'form.origurl'});
-    my $action = $ENV{'form.action'};
+    my $function = $env{'form.function'};
+    my $origurl = &Apache::lonnet::unescape($env{'form.origurl'});
+    my $action = $env{'form.action'};
 
     if ($action eq 'process') {
         &print_request_receipt($r,$origurl,$function);
@@ -35,24 +62,29 @@ sub handler {
     
 sub print_request_form {
     my ($r,$origurl,$function) = @_;
-    my ($os,$browser,$bversion,$uhost,$uname,$udom,$uhome,$urole,$usec,$email,$cid,$cdom,$cnum,$ctitle,$ccode,$sectionlist,$lastname,$firstname,$server);
-    my $bodytag = &Apache::loncommon::bodytag('',$function,'topmargin="0",marginheight="0"',1);
+    my ($os,$browser,$bversion,$uhost,$uname,$udom,$uhome,$urole,$usec,$email,$cid,$cdom,$cnum,$ctitle,$ccode,$sectionlist,$lastname,$firstname,$server,$formname);
+    my $bodytag = &Apache::loncommon::bodytag('',$function,'topmargin="0" marginheight="0" onLoad="initialize_codes()"',1);
     my $tablecolor = &Apache::loncommon::designparm($function.'.tabbg');
     if (($tablecolor eq '') || ($tablecolor eq '#FFFFFF')) {
         $tablecolor = '#EEEE99';
     }
     $ccode = '';
-    $os = $ENV{'browser.os'};
-    $browser = $ENV{'browser.type'};
-    $bversion = $ENV{'browser.version'};
-    $uhost = $ENV{'request.host'};
-    $uname = $ENV{'user.name'};
-    $udom = $ENV{'user.domain'};
-    $uhome = $ENV{'user.home'};
-    $urole = $ENV{'request.role'};
-    $usec = $ENV{'request.course.sec'};
-    $cid = $ENV{'request.course.id'};
-    $server = $ENV{'SERVER_NAME'};
+    $os = $env{'browser.os'};
+    $browser = $env{'browser.type'};
+    $bversion = $env{'browser.version'};
+    $uhost = $env{'request.host'};
+    $uname = $env{'user.name'};
+    $udom = $env{'user.domain'};
+    $uhome = $env{'user.home'};
+    $urole = $env{'request.role'};
+    $usec = $env{'request.course.sec'};
+    $cid = $env{'request.course.id'};
+    $formname = 'logproblem';
+    if ($origurl =~ m-^http://-) {
+        $server = $origurl;
+    } else {
+        $server = 'http://'.$ENV{'SERVER_NAME'}.$origurl;
+    }
     my $scripttag = (<<'END');
 function validate() {
     if (validmail(document.logproblem.email) == false) {
@@ -83,6 +115,7 @@ function validmail(field) {
     }
 }
 END
+    #" stupid emacs
     if ($cid =~ m/_/) {
         ($cdom,$cnum) = split/_/,$cid;
     }
@@ -92,17 +125,17 @@ END
         $ccode = $csettings{'internal.coursecode'};
         $sectionlist = $csettings{'internal.sectionnums'};
     }
-    if ($ENV{'environment.critnotification'}) {
-        $email = $ENV{'environment.critnotification'};
+    if ($env{'environment.critnotification'}) {
+        $email = $env{'environment.critnotification'};
     }
-    if (!$email && $ENV{'environment.notification'}) {
-        $email = $ENV{'environment.notification'};
+    if (!$email && $env{'environment.notification'}) {
+        $email = $env{'environment.notification'};
     }
-    if ($ENV{'environment.lastname'}) {
-        $lastname = $ENV{'environment.lastname'};
+    if ($env{'environment.lastname'}) {
+        $lastname = $env{'environment.lastname'};
     }
-    if ($ENV{'environment.firstname'}) {
-        $firstname = $ENV{'environment.firstname'};
+    if ($env{'environment.firstname'}) {
+        $firstname = $env{'environment.firstname'};
     }
     my @sections = split/,/,$sectionlist;
     my %groupid = ();
@@ -112,8 +145,12 @@ END
     }
     my $codedom = $Apache::lonnet::perlvar{'lonDefDomain'};
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['codedom']);
-    if (exists($ENV{'form.codedom'})) {
-        $codedom = $ENV{'form.codedom'};
+    if (exists($env{'form.codedom'})) {
+        $codedom = $env{'form.codedom'};
+    }
+    my $details_title;
+    if ($codedom) {
+        $details_title = '<br />('.$codedom.')';
     }
     my %coursecodes = ();
     my %codes = ();
@@ -127,7 +164,11 @@ END
     my $totcodes = 0;
     my $format_reply;
     my $jscript = '';
-
+    my $loaditems = qq|
+function initialize_codes() {
+    return;
+}
+    |;
     if ($cdom) {
         $codedom = $cdom;
     }
@@ -144,15 +185,19 @@ END
         $totcodes = &retrieve_instcodes(\%coursecodes,$codedom,$totcodes);
     }
     if ($totcodes > 0) {
-        $format_reply = &Apache::lonnet::auto_instcode_format($caller,$codedom,\%coursecodes,\%codes,\@codetitles,\%cat_titles,\%cat_order);
         if ($ccode eq '') {
-            my $numtypes = @codetitles;
-            &build_code_selections(\%codes,\@codetitles,\%cat_titles,\%cat_order,\%idlist,\%idnums,\%idlist_titles);
-            &javascript_code_selections($numtypes,\%cat_titles,\$jscript,\%idlist,\%idnums,\%idlist_titles,\@codetitles);
+            $format_reply = &Apache::lonnet::auto_instcode_format($caller,$codedom,\%coursecodes,\%codes,\@codetitles,\%cat_titles,\%cat_order);
+            if ($format_reply eq 'ok') {
+                my $numtypes = @codetitles;
+                &build_code_selections(\%codes,\@codetitles,\%cat_titles,\%cat_order,\%idlist,\%idnums,\%idlist_titles);
+                &javascript_code_selections($formname,$numtypes,\%cat_titles,\$jscript,\%idlist,\%idnums,\%idlist_titles,\@codetitles);
+                $loaditems = '';
+            }
         }
     }
+    my $html=&Apache::lonxml::xmlbegin();
     $r->print(<<ENDHEAD);
-<html>
+$html
 <head>
  <title>LON-CAPA support request</title>
 <script type"text/javascript">
@@ -298,7 +343,7 @@ END
              <table width="100%" border="0" cellpadding="8" cellspacing="0">
               <tr>
                <td>
-                http://$server$origurl<input type="hidden" name="sourceurl" value="http://$server$origurl" />
+                $server<input type="hidden" name="sourceurl" value="$server" />
                </td>
               </tr>
              </table>
@@ -337,7 +382,7 @@ END
             <td width="140" bgcolor="$tablecolor">
              <table width="140" border="0" cellpadding="8" cellspacing="0">
               <tr>
-               <td align="right"><b>Course Details:</b>
+               <td align="right"><b>Course Details:</b>$details_title
                </td>
               </tr>
              </table>
@@ -384,6 +429,11 @@ END
                     } else {
                         $longitems[0] = $idlist_titles{$codetitles[0]};
                     }
+                    for (my $i=0; $i<@longitems; $i++) {
+                        if ($longitems[$i] eq '') {
+                            $longitems[$i] = $items[$i];
+                        }
+                    }
                 } else {
                     @longitems = @items;
                 }
@@ -444,12 +494,13 @@ END
                <td>
 END
     if ($sectionlist) {
-        $r->print("<select name=\"section\">");
+        $r->print("<select name=\"section\"\n>".
+                  "  <option value=\"\" selected=\"selected\">Select</option>\n");
         foreach (sort keys %groupid) {
             if ($_ eq $groupid{$_} || $groupid{$_} eq '') {
-                $r->print("<option value=\"$_\" />$_");
+                $r->print("  <option value=\"$_\" >$_</option>\n");
             } else {
-                $r->print("<option value=\"$_\" />$_ - (LON-CAPA sec: $groupid{$_})");
+                $r->print("  <option value=\"$_\" >$_ - (LON-CAPA sec: $groupid{$_})</option>\n");
             }
         }
         $r->print("</select>");
@@ -516,7 +567,7 @@ END
 	    </td>
 	   </tr>
 END
-    if (defined($ENV{'user.name'})) {
+    if (defined($env{'user.name'})) {
         $r->print(<<END);
            <tr>
             <td width="140" bgcolor="$tablecolor">
@@ -588,8 +639,10 @@ END
 
 sub print_request_receipt {
     my ($r,$url,$function) = @_;
-    my @envvars = ('lonID','HTTP_HOST','HTTP_USER_AGENT','REMOTE_ADDR','SERVER_ADDR','SERVER_NAME','browser.os','browser.type','browser.version','user.home','request.role');
+    my @ENVvars = ('HTTP_HOST','HTTP_USER_AGENT','REMOTE_ADDR','SERVER_ADDR','SERVER_NAME');
+    my @envvars = ('browser.os','browser.type','browser.version','user.home','request.role');
     my @loncvars = ('user.name','user.domain','request.course.sec','request.course.id');
+    my @cookievars = ('lonID');
 
     my $bodytag = &Apache::loncommon::bodytag('',$function,'topmargin="0" marginheight="0"',1);
     my $admin = $Apache::lonnet::perlvar{'lonAdminMail'};
@@ -602,48 +655,49 @@ sub print_request_receipt {
     my @formvars = ('username','email','uname','udom','sourceurl','phone','section','coursecode','title','subject','description','screenshot');
 
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},\@formvars);
-    my $coursecode = $ENV{'form.coursecode'};
+    my $coursecode = $env{'form.coursecode'};
     if ($coursecode eq '') {
-        if (defined($ENV{'form.Year'})) {
-            $coursecode .= $ENV{'form.Year'};
+        if (defined($env{'form.Year'})) {
+            $coursecode .= $env{'form.Year'};
         }
-        if (defined($ENV{'form.Semester'})) {
-            $coursecode .= $ENV{'form.Semester'};
+        if (defined($env{'form.Semester'})) {
+            $coursecode .= $env{'form.Semester'};
         }
-        if (defined($ENV{'form.Department'})) {
-            $coursecode .= $ENV{'form.Department'};
+        if (defined($env{'form.Department'})) {
+            $coursecode .= $env{'form.Department'};
         }
-        if (defined($ENV{'form.Number'})) {
-            $coursecode .= $ENV{'form.Number'};
+        if (defined($env{'form.Number'})) {
+            $coursecode .= $env{'form.Number'};
         }
     }
     my $supportmsg = qq|
-Name: $ENV{'form.username'}
-Email: $ENV{'form.email'}
-Username/domain: $ENV{'form.uname'} - $ENV{'form.udom'}
-Tel: $ENV{'form.phone'}
-Course Information: $ENV{'form.title'} - $coursecode - section: $ENV{'form.section'}
-Subject: $ENV{'form.subject'}
-Description: $ENV{'form.description'}
-URL: $ENV{'form.sourceurl'}
+Name: $env{'form.username'}
+Email: $env{'form.email'}
+Username/domain: $env{'form.uname'} - $env{'form.udom'}
+Tel: $env{'form.phone'}
+Course Information: $env{'form.title'} - $coursecode - section: $env{'form.section'}
+Subject: $env{'form.subject'}
+Description: $env{'form.description'}
+URL: $env{'form.sourceurl'}
 Date/Time: $reporttime
 
     |;
-    my $descrip = $ENV{'form.description'};
+    my $descrip = $env{'form.description'};
     $descrip =~ s#\n#<br />#g;
     my $displaymsg = qq|
-<font color="$fontcolor">Name:</font><font color="$vlinkcolor"> $ENV{'form.username'}</font><br />
-<font color="$fontcolor">Email: </font><font color="$vlinkcolor">$ENV{'form.email'}</font><br />
-<font color="$fontcolor">Username/domain: </font><font color="$vlinkcolor">$ENV{'form.uname'} - $ENV{'form.udom'}</font><br />
-<font color="$fontcolor">Tel: </font><font color="$vlinkcolor">$ENV{'form.phone'}</font><br />
-<font color="$fontcolor">Course Information: </font><font color="$vlinkcolor">$ENV{'form.title'} - $coursecode - section: $ENV{'form.section'}</font><br />
-<font color="$fontcolor">Subject: </font><font color="$vlinkcolor">$ENV{'form.subject'}</font><br />
+<font color="$fontcolor">Name:</font><font color="$vlinkcolor"> $env{'form.username'}</font><br />
+<font color="$fontcolor">Email: </font><font color="$vlinkcolor">$env{'form.email'}</font><br />
+<font color="$fontcolor">Username/domain: </font><font color="$vlinkcolor">$env{'form.uname'} - $env{'form.udom'}</font><br />
+<font color="$fontcolor">Tel: </font><font color="$vlinkcolor">$env{'form.phone'}</font><br />
+<font color="$fontcolor">Course Information: </font><font color="$vlinkcolor">$env{'form.title'} - $coursecode - section: $env{'form.section'}</font><br />
+<font color="$fontcolor">Subject: </font><font color="$vlinkcolor">$env{'form.subject'}</font><br />
 <font color="$fontcolor">Description: </font><font color="$vlinkcolor">$descrip</font><br />
-<font color="$fontcolor">URL: </font><font color="$vlinkcolor">$ENV{'form.sourceurl'}</font><br />
+<font color="$fontcolor">URL: </font><font color="$vlinkcolor">$env{'form.sourceurl'}</font><br />
 <font color="$fontcolor">Date/Time: </font><font color="$vlinkcolor">$reporttime</font><br />
     |;
+    my $html=&Apache::lonxml::xmlbegin();
     $r->print(<<"END");
-<html>
+$html
 <head>
  <title>LON-CAPA support request recorded</title>
 </head>
@@ -670,13 +724,13 @@ END
             $to = 'helpdesk@lon-capa.org';
         }
     }
-    if (defined($ENV{'form.email'})) {
-        if ($ENV{'form.email'} =~ m/^[^\@]+\@[^\@]+$/) {
-            $from = $ENV{'form.email'};
+    if (defined($env{'form.email'})) {
+        if ($env{'form.email'} =~ m/^[^\@]+\@[^\@]+$/) {
+            $from = $env{'form.email'};
         }
     }
 
-    my $subject = $ENV{'form.subject'};
+    my $subject = $env{'form.subject'};
     $subject =~ s#(`)#'#g;
     $subject =~ s#\$#\(\$\)#g;
     $supportmsg =~ s#(`)#'#g;
@@ -687,9 +741,9 @@ END
 
     my $attachmentpath = '';
     my $attachmentsize = '';
-    if (defined($ENV{'user.name'})) {
-        if ($ENV{'form.screenshot.filename'}) {
-            $attachmentsize = length($ENV{'form.screenshot'});
+    if (defined($env{'user.name'})) {
+        if ($env{'form.screenshot.filename'}) {
+            $attachmentsize = length($env{'form.screenshot'});
             if ($attachmentsize > 131072) {
                 $displaymsg .= "<br />The uploaded screenshot file ($attachmentsize bytes) included with your request exceeded the maximum allowed size - 128 KB, and has therefore been discarded.";
             } else {
@@ -698,13 +752,25 @@ END
         }
     }
 
+    my %cookies = ();
+    my $cookie=CGI::Cookie->parse($r->header_in('Cookie'));
+    if ($$cookie{'lonID'} =~ /lonID=(\w+);/) {
+        $cookies{'lonID'} = $1;
+    }
+
     if ($attachmentpath =~ m-/([^/]+)$-) {
         $fname = $1;
-        $displaymsg .= "<br />An uploaded screenshot file - $fname ($attachmentsize bytes) was included in the request sent by $ENV{'user.name'} from LON-CAPA domain: $ENV{'user.domain'}";
+        $displaymsg .= "<br />An uploaded screenshot file - $fname ($attachmentsize bytes) was included in the request sent by $env{'user.name'} from LON-CAPA domain: $env{'user.domain'}";
         $supportmsg .= "\n";
-        foreach (@envvars) {
+        foreach (@cookievars) {
+            $supportmsg .= "$_: $cookies{$_}\n";
+        }
+        foreach (@ENVvars) {
             $supportmsg .= "$_: $ENV{$_}\n";
         }
+        foreach (@envvars) {
+            $supportmsg .= "$_: $env{$_}\n";
+        }
     }
  
     my $msg = MIME::Lite->new(
@@ -724,24 +790,24 @@ END
 
     } else {
         my $envdata = '';
-        foreach (@envvars) {
+        foreach (@cookievars) {
+            $envdata .= "$_: $cookies{$_}\n";
+        }
+        foreach (@ENVvars) {
             $envdata .= "$_: $ENV{$_}\n";
         }
+        foreach (@envvars) {
+            $envdata .= "$_: $env{$_}\n";
+        }
         foreach (@loncvars) {
-            $envdata .= "$_: $ENV{$_}\n";
+            $envdata .= "$_: $env{$_}\n";
         }
         $msg->attach(Type => 'TEXT',
                      Data => $envdata);
     }
 
 ### Send it:
-    # ->send can cause an sh launch which can pass all of %ENV along
-    # which can be to large for /bin/sh's little mind
-    my %oldENV=%ENV;
-    undef(%ENV);
     $msg->send('sendmail');
-    %ENV=%oldENV;
-    undef(%oldENV);
 
     if ($attachmentpath =~ m#$Apache::lonnet::perlvar{'lonDaemons'}/tmp/helprequests/(\d+)/[^/]+#) {
         unlink($attachmentpath);
@@ -794,11 +860,21 @@ END
               <tr>
                <td>
     |);
-    foreach (@envvars) {
-        unless($ENV{$_} eq '') { 
+    foreach (@cookievars) {
+        unless($cookies{$_} eq '') {
+            $r->print("$_:&nbsp;<font color='$vlinkcolor'>$cookies{$_}</font>, ");
+        }
+    }
+    foreach (@ENVvars) {
+        unless($ENV{$_} eq '') {
             $r->print("$_:&nbsp;<font color='$vlinkcolor'>$ENV{$_}</font>, ");
         }
     }
+    foreach (@envvars) {
+        unless($env{$_} eq '') { 
+            $r->print("$_:&nbsp;<font color='$vlinkcolor'>$env{$_}</font>, ");
+        }
+    }
     $r->print("
                </td>
               </tr>
@@ -834,7 +910,12 @@ sub print_header {
                                            ask   => 'Ask helpdesk',
                                            getst => 'Getting started guide',
                                            back =>  'Back to last location'
-                                         );  
+                                         );
+    my ($getstartlink,$getstarttext);
+    if (-e $Apache::lonnet::perlvar{'lonDocRoot'}.'/adm/gettingstarted.html') {
+        $getstartlink = qq|<td align="center">&nbsp;<b><a href="/adm/gettingstarted.html">$lt{'getst'}</a></td>|;
+        $getstarttext = ' '.&mt('and the "Getting started" guide').' ';
+    }
     $r->print(<<END);
 <table width="620" border="0" cellspacing="0" cellpadding="0" height="55">   <tr height="50">    <td width='5'>&nbsp;</td>
    <td>
@@ -851,8 +932,7 @@ sub print_header {
           <table width="100%" border="0" cellpadding="0" cellspacing="1" bgcolor="#ffffff">
            <tr bgcolor="$tablecolor">
             <td align="center"><img src="$location/help/gif/smallHelp.gif" border="0" alt="(Login help)" valign="middle" />&nbsp;<b><a href="/adm/loginproblems.html">$lt{'login'}</a></td>
-            <td align="center">&nbsp;<b>$helpdesk_link<img src="$location/lonIcons/helpdesk.gif" border="0" alt="(Ask helpdesk)" valign="middle" />&nbsp;$lt{'ask'}</a></b>&nbsp;</td>
-            <td align="center">&nbsp;<b><a href="/adm/gettingstarted.html">$lt{'getst'}</a></td>
+            <td align="center">&nbsp;<b>$helpdesk_link<img src="$location/lonIcons/helpdesk.gif" border="0" alt="(Ask helpdesk)" valign="middle" />&nbsp;$lt{'ask'}</a></b>&nbsp;</td>$getstartlink
             <td align="center">&nbsp;<b><a href="$origurl" target="_top"><img src="$location/lonIcons/move_up.gif" border="0" alt="(Back to last location)" valign="middle" />&nbsp;$lt{'back'}</a></b>&nbsp;</td>
            </tr>
           </table>
@@ -877,7 +957,7 @@ END
         $r->print('
  <tr>
   <td colspan="3">'.&mt('
-Please read the "Log-in help" and "Getting started guide" if you can not log-in').'.  '.&mt('If your problem is still unresolved, the form below can be used to send a question to the LON-CAPA helpdesk').'.<br /><font size="-1"><b>'.&mt('Note').':</b> '.&mt('Student questions about course content should be directed to the course instructor').'.</font><br /><br />
+Please review the information in "Log-in help"').$getstarttext.' '.&mt('if you are unable to log-in').'.  '.&mt('If your problem is still unresolved, the form below can be used to send a question to the LON-CAPA helpdesk').'.<br /><font size="-1"><b>'.&mt('Note').':</b> '.&mt('Student questions about course content should be directed to the course instructor').'.</font><br /><br />
   </td>
  </tr>');
     }
@@ -888,7 +968,7 @@ Please read the "Log-in help" and "Getti
 
 sub retrieve_instcodes {
     my ($coursecodes,$codedom,$totcodes) = @_;
-    my %courses = &Apache::lonnet::courseiddump($codedom,'.',1,'.','.');
+    my %courses = &Apache::lonnet::courseiddump($codedom,'.',1,'.','.','.');
     foreach my $course (keys %courses) {
         if ($courses{$course} =~ m/^[^:]*:([^:]+)/) {
             $$coursecodes{$course} = &Apache::lonnet::unescape($1);
@@ -1107,7 +1187,7 @@ sub recurse_list {
 }
 
 sub javascript_code_selections {
-    my ($numcats,$cat_titles,$script_tag,$idlist,$idnums,$idlist_titles,$codetitles) = @_;
+    my ($formname,$numcats,$cat_titles,$script_tag,$idlist,$idnums,$idlist_titles,$codetitles) = @_;
     my $numtitles = @{$codetitles};
     my @seltitles = ();
     for (my $j=0; $j<$numtitles; $j++) {
@@ -1128,10 +1208,10 @@ function courseSet(caller) {
     var ids = new Array ("$seltitle_str");
     var formitems = new Array ($numtitles);
     var longtitles = new Array ("$longtitles_str");
-    var idyr = document.logproblem.Year.selectedIndex
-    var idsem  = document.logproblem.Semester.selectedIndex
-    var iddept = document.logproblem.Department.selectedIndex
-    var idclass = document.logproblem.Number.selectedIndex
+    var idyr = document.$formname.Year.selectedIndex
+    var idsem  = document.$formname.Semester.selectedIndex
+    var iddept = document.$formname.Department.selectedIndex
+    var idclass = document.$formname.Number.selectedIndex
     var idyears = new Array("$$idlist{$$codetitles[0]}");
 END
     if ($longtitles[0]) {
@@ -1194,68 +1274,99 @@ END
         }
     }
     $$script_tag .= (<<END_OF_BLOCK);
- if (caller == "$$codetitles[0]") {
-   document.logproblem.Department.length = 0
-   document.logproblem.Number.length = 0
-   document.logproblem.Department.options[0] = new Option("<-Pick $$codetitles[1]","-1",true,true)
-   document.logproblem.Number.options[0] = new Option("<-Pick $$codetitles[2]","-1",true,true)
-   if (idyr == 0) {
-    document.logproblem.Semester.length = 0
-    document.logproblem.Semester.options[0] = new Option("<-Pick $$codetitles[0]","-1",true,true)
-   }
-   else {
-    document.logproblem.Semester.length = 0
-    document.logproblem.Semester.options[0] = new Option("Select","-1",true,true)
-    for (var i=0; i<idsems[idyr-1].length; i++) {
-      if (longtitles[1] == 1) {
-          document.logproblem.Semester.options[i+1] = new Option(idsemslongs[idyr-1][i],idsems[idyr-1][i],false,false)
-      }
-      else {
-          document.logproblem.Semester.options[i+1] = new Option(idsems[idyr-1][i],idsems[idyr-1][i],false,false)
-      }
-    }
-   }
-   document.logproblem.Semester.selectedIndex = 0;
+ var display = new Array($numtitles)
+ if (caller == "" || caller == "$$codetitles[0]") {
+     if (caller == "") {
+         document.$formname.Year.length = 0
+         document.$formname.Year.options[0] = new Option("Select","-1",true,true)
+         display[0] = new Array(idyears.length)
+         for (var i=0; i<idyears.length; i++) {
+             display[0][i] = idyears[i]
+             if (longtitles[0] == 1) {
+                 if (idyearslongs[i] != "") {
+                     display[0][i] = idyearslongs[i]
+                 }
+             }
+             else {
+                 if (idyearslongs[i] != "") {
+                     display[0][i] = idyears[i]
+                 }
+             }
+             document.$formname.Year.options[i+1] = new Option(display[0][i],idyears[i],false,false)
+         }
+         document.$formname.Year.selectedIndex = 0;
+     }
+     document.$formname.Semester.length = 0
+     document.$formname.Department.length = 0;
+     document.$formname.Number.length = 0
+     document.$formname.Department.options[0] = new Option("<-Pick $$codetitles[1]","-1",true,true)
+     document.$formname.Number.options[0] = new Option("<-Pick $$codetitles[2]","-1",true,true)
+     if (idyr == 0 || caller == "") {
+         document.$formname.Semester.options[0] = new Option("<-Pick $$codetitles[0]","-1",true,true)
+     }
+     else {
+         document.$formname.Semester.options[0] = new Option("Select","-1",true,true)
+         display[1] = new Array(idsems[idyr-1].length)
+         for (var i=0; i<idsems[idyr-1].length; i++) {
+             display[1][i] = idsems[idyr-1][i]
+             if (longtitles[1] == 1) {
+                 if (idsemslongs[idyr-1][i] != "") {
+                     display[1][i] = idsemslongs[idyr-1][i]
+                 }
+             }
+             document.$formname.Semester.options[i+1] = new Option(display[1][i],idsems[idyr-1][i],false,false)
+         }
+     }
+     document.$formname.Semester.selectedIndex = 0;
  }
  if (caller == "$$codetitles[1]") {
-   document.logproblem.Department.length = 0
-   document.logproblem.Number.length = 0
-   document.logproblem.Number.options[0] = new Option("<-Pick $$codetitles[2]","-1",true,true)
+   document.$formname.Department.length = 0
+   document.$formname.Number.length = 0
+   document.$formname.Number.options[0] = new Option("<-Pick $$codetitles[2]","-1",true,true)
    if (idsem == 0) {
-     document.logproblem.Department.options[0] = new Option("<-Pick $$codetitles[1]","-1",true,true)
+     document.$formname.Department.options[0] = new Option("<-Pick $$codetitles[1]","-1",true,true)
    }
    else {
-    document.logproblem.Department.options[0] = new Option("Select","-1",true,true)    
+    document.$formname.Department.options[0] = new Option("Select","-1",true,true)    
+    display[2] = new Array(idcodes[idyr-1][idsem-1].length)
     for (var i=0; i<idcodes[idyr-1][idsem-1].length; i++) {
+      display[2][i] = idcodes[idyr-1][idsem-1][i]
       if (longtitles[2] == 1) {
-          document.logproblem.Department.options[i+1] = new Option(idcodeslongs[idyr-1][idsem-1][i],idcodes[idyr-1][idsem-1][i],false,false)
-      }
-      else {
-          document.logproblem.Department.options[i+1] = new Option(idcodes[idyr-1][idsem-1][i],idcodes[idyr-1][idsem-1][i],false,false)
+          if (idcodeslongs[idyr-1][idsem-1][i] != "") {
+              display[2][i] = idcodeslongs[idyr-1][idsem-1][i]
+          }
       }
+      document.$formname.Department.options[i+1] = new Option(display[2][i],idcodes[idyr-1][idsem-1][i],false,false)
     }
    }
-   document.logproblem.Department.selectedIndex = 0
+   document.$formname.Department.selectedIndex = 0
  }
  if (caller == "$$codetitles[2]") {
-   document.logproblem.Number.length = 0
+   document.$formname.Number.length = 0
    if (iddept == 0) {
-     document.logproblem.Number.options[0] = new Option("<-Pick $$codetitles[2]","-1",true,true)
+     document.$formname.Number.options[0] = new Option("<-Pick $$codetitles[2]","-1",true,true)
    }
    else {
-    document.logproblem.Number.options[0] = new Option("Select","-1",true,true)
+    document.$formname.Number.options[0] = new Option("Select","-1",true,true)
+    display[3] = new Array (idcourses[idyr-1][idsem-1][iddept-1].length)
     for (var i=0; i<idcourses[idyr-1][idsem-1][iddept-1].length; i++) {
+      display[3][i] = idcourses[idyr-1][idsem-1][iddept-1][i]
       if (longtitles[3] == 1) {
-        document.logproblem.Number.options[i+1] = new Option(idcourseslongs[idyr-1][idsem-1][iddept-1][i],idcourses[idyr-1][idsem-1][iddept-1][i],false,false)
-      }
-      else {
-        document.logproblem.Number.options[i+1] = new Option(idcourses[idyr-1][idsem-1][iddept-1][i],idcourses[idyr-1][idsem-1][iddept-1][i],false,false)
+        if (idcourseslongs[idyr-1][idsem-1][iddept-1][i] != "") {
+            display[3][i] = idcourseslongs[idyr-1][idsem-1][iddept-1][i]
+        }
       }
+      document.$formname.Number.options[i+1] = new Option(display[3][i],idcourses[idyr-1][idsem-1][iddept-1][i],false,false)
     }
    }
-   document.logproblem.Number.selectedIndex = 0
+   document.$formname.Number.selectedIndex = 0
  }
 }
+
+function initialize_codes() {
+    courseSet();
+    return;
+}
 END_OF_BLOCK
 }