--- loncom/interface/lonsupportreq.pm	2005/02/17 08:29:43	1.23
+++ loncom/interface/lonsupportreq.pm	2006/03/15 19:41:26	1.30
@@ -1,12 +1,39 @@
+#
+# $Id: lonsupportreq.pm,v 1.30 2006/03/15 19:41:26 albertel 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,23 +62,24 @@ 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 ($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'};
+    $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 {
@@ -87,6 +115,7 @@ function validmail(field) {
     }
 }
 END
+    #" stupid emacs
     if ($cid =~ m/_/) {
         ($cdom,$cnum) = split/_/,$cid;
     }
@@ -96,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 = ();
@@ -116,8 +145,8 @@ 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) {
@@ -161,21 +190,23 @@ function initialize_codes() {
             if ($format_reply eq 'ok') {
                 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);
+                &javascript_code_selections($formname,$numtypes,\%cat_titles,\$jscript,\%idlist,\%idnums,\%idlist_titles,\@codetitles);
                 $loaditems = '';
             }
         }
     }
+	
     my $html=&Apache::lonxml::xmlbegin();
+    my $head = &Apache::loncommon::headtag('LON-CAPA support request');
+    my $end_head = &Apache::loncommon::endheadtag();
     $r->print(<<ENDHEAD);
 $html
-<head>
- <title>LON-CAPA support request</title>
+$head
 <script type"text/javascript">
 $scripttag
 $jscript
 </script>
-</head>
+$end_head
 $bodytag
 ENDHEAD
     if ($r->uri eq '/adm/helpdesk') {
@@ -538,7 +569,7 @@ END
 	    </td>
 	   </tr>
 END
-    if (defined($ENV{'user.name'})) {
+    if (defined($env{'user.name'})) {
         $r->print(<<END);
            <tr>
             <td width="140" bgcolor="$tablecolor">
@@ -566,6 +597,7 @@ END
            </tr>
 END
     }
+
     $r->print(<<END);
            <tr>
             <td width="140" bgcolor="$tablecolor">
@@ -602,16 +634,17 @@ END
   </tr>
  </table>
 </form>
-</body>
-</html>
 END
+    $r->print(&Apache::loncommon::end_page());
     return;
 }
 
 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'};
@@ -624,52 +657,51 @@ 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();
+    my $head = &Apache::loncommon::head('LON-CAPA support request recorded');
     $r->print(<<"END");
 $html
-<head>
- <title>LON-CAPA support request recorded</title>
-</head>
+$head
 $bodytag
 <form name="logproblem">
 <input type="hidden" name="action" value="result" />
@@ -693,13 +725,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;
@@ -710,9 +742,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 {
@@ -721,13 +753,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(
@@ -747,24 +791,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);
@@ -817,11 +861,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>
@@ -838,9 +892,8 @@ END
    </td>
   </tr>
  </table>
-</body>
-</html>
     ");
+    $r->print(&Apache::loncommon::end_page());
 }
 
 sub print_header {
@@ -915,7 +968,7 @@ Please review the information in "Log-in
 
 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);
@@ -1134,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++) {
@@ -1155,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]) {
@@ -1222,55 +1275,59 @@ END
     }
     $$script_tag .= (<<END_OF_BLOCK);
  var display = new Array($numtitles)
- if (caller == "") {
-    document.logproblem.Year.length = 0
-    document.logproblem.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]
-          }
-      }
-      document.logproblem.Year.options[i+1] = new Option(display[0][i],idyears[i],false,false)
-    }
-    document.logproblem.Year.selectedIndex = 0;
- }
- 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)
-    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.logproblem.Semester.options[i+1] = new Option(display[1][i],idsems[idyr-1][i],false,false)
-    }
-   }
-   document.logproblem.Semester.selectedIndex = 0;
+ 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]
@@ -1279,18 +1336,18 @@ END
               display[2][i] = idcodeslongs[idyr-1][idsem-1][i]
           }
       }
-      document.logproblem.Department.options[i+1] = new Option(display[2][i],idcodes[idyr-1][idsem-1][i],false,false)
+      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]
@@ -1299,10 +1356,10 @@ END
             display[3][i] = idcourseslongs[idyr-1][idsem-1][iddept-1][i]
         }
       }
-      document.logproblem.Number.options[i+1] = new Option(display[3][i],idcourses[idyr-1][idsem-1][iddept-1][i],false,false)
+      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
  }
 }