--- loncom/interface/lonpickcourse.pm	2003/03/26 16:39:40	1.3
+++ loncom/interface/lonpickcourse.pm	2003/11/04 16:33:10	1.12
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Pick a course
 #
-# $Id: lonpickcourse.pm,v 1.3 2003/03/26 16:39:40 www Exp $
+# $Id: lonpickcourse.pm,v 1.12 2003/11/04 16:33:10 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -33,10 +33,11 @@ use Apache::Constants qw(:common);
 use Apache::loncommon;
 use Apache::loncoursedata;
 use Apache::lonnet;
+use Apache::lonlocal;
 
 sub handler {
     my $r = shift;
-    $r->content_type('text/html');
+    &Apache::loncommon::content_type($r,'text/html');
     $r->send_http_header;
     return OK if $r->header_only;
 
@@ -47,22 +48,42 @@ sub handler {
 <title>The LearningOnline Network with CAPA</title>
 </head>
 ENDDOCUMENT
-
-
+# Get parameters from query string
     &Apache::loncommon::get_unprocessed_cgi
         ($ENV{'QUERY_STRING'},['domainfilter','descriptfilter',
                  'sincefilter','form','cnumelement','cdomelement']);
+# domain filter and selection
     my $domainfilter=$ENV{'form.domainfilter'};
     $domainfilter=~s/\W//g;
-    unless ($domainfilter) { $domainfilter='.'; }
+    unless ($domainfilter) { $domainfilter=''; }
+    my $domainselectform=&Apache::loncommon::select_dom_form($domainfilter,
+                                                          'domainfilter',1);
+# description filter
     my $descriptfilter=$ENV{'form.descriptfilter'};
-    $descriptfilter=~s/\W//g;
-    unless ($descriptfilter) { $descriptfilter='.'; }
+    $descriptfilter=~s/[^\w\s]//g;
+    unless ($descriptfilter) { $descriptfilter=''; }
+
+# last course activity filter and selection
     my $sincefilter=$ENV{'form.sincefilter'};
-    $sincefilter=~s/\D//g;
-    unless ($sincefilter) { $sincefilter=0; }
+    $sincefilter=~s/[^\d-]//g;
+    unless ($sincefilter) { $sincefilter=-1; }
+    my $sincefilterform=&Apache::loncommon::select_form($sincefilter,
+       'sincefilter',('-1'=>'',
+                      '86400' => 'today',
+                      '604800' => 'last week',
+                      '2592000' => 'last month',
+                      '7776000' => 'last three months',
+                      '15552000' => 'last six months',
+                      '31104000' => 'last year',
+                      'select_form_order' =>
+                      ['-1','86400','604800','2592000','7776000',
+		      '15552000','31104000']));
        
     $r->print(&Apache::loncommon::bodytag("Selecting a Course"));
+    my %lt=&Apache::lonlocal::texthash(
+				       'cac' => 'Course Activity',
+				       'cde' => 'Course Description',
+				       'cdo' => 'Course Domain');
     $r->print(<<ENDSCRIPT);
 <script>
 function gochoose(cname,cdom) {
@@ -70,25 +91,53 @@ function gochoose(cname,cdom) {
     var slct=opener.document.$ENV{'form.form'}.$ENV{'form.cdomelement'};
     var i;
     for (i=0;i<slct.length;i++) {
-        if (slct[i]==cdom) { slct.selectedIndex=i; }
+        if (slct.options[i].value==cdom) { slct.selectedIndex=i; }
     }
     self.close();
 }
 </script>
+<form method="post">
+<input type="hidden" name="cnumelement" value="$ENV{'form.cnumelement'}" />
+<input type="hidden" name="cdomelement" value="$ENV{'form.cdomelement'}" />
+<input type="hidden" name="form" value="$ENV{'form.form'}" />
+
+$lt{'cac'}: $sincefilterform
+<br />
+$lt{'cdo'}: $domainselectform
+<br />
+$lt{'cde'}:
+<input type="text" name="descriptfilter" size="40" value="$descriptfilter" />
+<p><input type="submit" name="gosearch" value="Search" /></p>
+</form>
+<hr />
 ENDSCRIPT
-    $r->rflush();
 # ---------------------------------------------------------------- Get the data
-    my %courses=
-     &Apache::lonnet::courseiddump($domainfilter,$descriptfilter,$sincefilter);
-    $r->print('<form>');
-    foreach (keys %courses) {
-        my ($cdom,$cnum)=split(/\_/,$_);
-	$r->print('<input type="button" value="Select" onClick="gochoose('.
-		  "'".$cnum."','".$cdom."')".'" /> '.$courses{$_}.' ('.
+    if ($ENV{'form.gosearch'}) {
+        $r->print(&mt('Searching').' ...<br />&nbsp;<br />');
+        $r->rflush();
+	unless ($descriptfilter) { $descriptfilter='.'; }
+        my $timefilter=($sincefilter==-1?1:time-$sincefilter);
+	my %courses=
+	    &Apache::lonnet::courseiddump
+	    ($domainfilter,$descriptfilter,$timefilter);
+	$r->print('<form>');
+	my %by_descrip;
+	foreach my $course (keys %courses) {
+	    $by_descrip{lc($courses{$course})}=$course;
+	}
+	foreach my $description (sort(keys(%by_descrip))) {
+	    my $course=$by_descrip{$description};
+	    my ($cdom,$cnum)=split(/\_/,$course);
+	    $r->print('<input type="button" value="Select" onClick="gochoose('.
+		  "'".$cnum."','".$cdom."')".'" /> '.$courses{$course}.' ('.
 		  ($Apache::lonnet::domaindescription{$cdom}?
-                   $Apache::lonnet::domaindescription{$cdom}:$cdom).')<br />');
+                   $Apache::lonnet::domaindescription{$cdom}:$cdom).
+                 ")<br />\n");
+	}
+	if (!%courses) { $r->print(&mt('None found')); }
+        $r->print('</form>');
     }
-    $r->print('</form></body></html>');
+    $r->print('</body></html>');
     return OK;
 }