--- loncom/html/adm/helper/newslot.helper	2006/02/03 22:40:46	1.12
+++ loncom/html/adm/helper/newslot.helper	2006/03/07 16:29:14	1.14
@@ -1,5 +1,8 @@
 <helper title="Slot Creation" requiredpriv="mgq">
   <state name="START" title="Specify Required Attributes">
+
+    <nextstate>OPTIONAL</nextstate>
+
     <exec>
       if (!exists($helper->{'VARS'}{'name'}) ||
 	  $helper->{'VARS'}{'name'} !~ /\S/) {
@@ -26,7 +29,7 @@
 	  return $slot{$name}{$which};
       }
     </exec>
-    <message nextstate="OPTIONAL">
+    <message>
         Name: <br />&nbsp;&nbsp;&nbsp;
     </message>
     <string variable="name" size="30">
@@ -42,7 +45,7 @@
     </string>
     <condition>
       <clause>$helper->{DATA}{copy}</clause>
-      <message nextstate="OPTIONAL">
+      <message>
 	<p>Changing the Name will create a new slot with the new name, and not rename the existing slot.</p>
       </message>
     </condition>
@@ -74,6 +77,7 @@
     </choices>
   </state>
 
+
   <state name="OPTIONAL" title="Specify Optional Attributes">
     <message>
         <p>Description:<br />&nbsp;&nbsp;&nbsp;
@@ -159,9 +163,7 @@
       </defaultvalue>
     </choices>
     <message>
-       
-	</p><p>IP restrictions:<br />&nbsp;&nbsp;&nbsp;
-      
+	</p><p>IP restrictions:<br />&nbsp;&nbsp;&nbsp;      
     </message>
     <string variable="ip" size="30">
       <validator>
@@ -173,19 +175,33 @@
     </string>
     <message>  </p>  </message>
   </state>
+
+
   <state name="RESOURCESELECT" title="Specify Optional Attributes">
+
+    <nextstate>PROCTOR</nextstate>
+
     <resource variable="symb">
       <filterfunc>return $res->is_problem()</filterfunc>
       <valuefunc>return $res->symb()</valuefunc>
-      <nextstate>PROCTOR</nextstate>
       <defaultvalue>
         return &{$helper->{DATA}{origslot}}('symb');
       </defaultvalue>
     </resource>    
   </state>
+
+
   <state name="PROCTOR" title="Specify Proctors">
+
+    <nextstate>FINISH</nextstate>
+
+    <condition>
+       <clause>$helper->{'VARS'}{'type'} eq 'schedulable_student'</clause>
+       <nextstate>RESTRICTUSER</nextstate>
+    </condition>
+
     <student variable="proctor" multichoice="1" coursepersonnel="1"
-             nextstate="FINISH" activeonly="1" emptyallowed="1">
+             activeonly="1" emptyallowed="1">
       <defaultvalue>
         my @defaults;
         my $default=&{$helper->{DATA}{origslot}}('proctor');
@@ -197,6 +213,35 @@
       </defaultvalue>
     </student>
   </state>
+
+
+  <state name="RESTRICTUSER" title="Restrict slot availability">
+    <nextstate>FINISH</nextstate>
+    <message>
+      <p>Slots are by default available to all users in a course, if you would like this slot to be restricted to a subset of users you can specify restrictions.</p>
+    </message>
+
+    <message>
+      Select sections to limit slot availability to: <br />&nbsp;&nbsp;&nbsp;
+    </message>
+    <section variable="allowedsections" multichoice="1" allowempty="1">
+      <defaultvalue>
+        return join('|||',
+		    split(',',&{$helper->{DATA}{origslot}}('allowedsections')));
+      </defaultvalue>
+    </section>
+    <message>
+      Select users to limit slot availability to: <br />&nbsp;&nbsp;&nbsp;
+    </message>
+    <student variable="allowedusers" multichoice="1" coursepersonnel="1"
+             activeonly="1" emptyallowed="1">
+      <defaultvalue>
+	 return split(',',&{$helper->{DATA}{origslot}}('allowedusers'));
+      </defaultvalue>
+    </student>
+  </state>
+
+
   <state name="FINISH" title="Creating/Modfying Slot">
 	    <message> Created Slot  </message>	
     <final>
@@ -204,23 +249,27 @@
         my $result;
 	if ($helper->{'STATE'} ne 'FINISH') { return; }
 	my %slot;
+
 	foreach my $which ('type','starttime','endtime') {
 	    $slot{$which} = $helper->{'VARS'}{$which};
 	}
-	foreach my $which ('ip','description','maxspace',
-			   'secret','symb') {
+
+	foreach my $which ('ip','description','maxspace','secret','symb') {
 	    if ( $helper->{'VARS'}{$which} =~/\S/ ) {
 		$slot{$which} = $helper->{'VARS'}{$which};
 	    }
 	}
+
 	if ( $helper->{'VARS'}{'startreserve'} > 0) {
 		$slot{'startreserve'} = $helper->{'VARS'}{'startreserve'};
 	}
+
 	if ( $helper->{'VARS'}{'startunique'} > 0 &&
 	     $helper->{'VARS'}{'endunique'} > 0 ) {
 	    $slot{'uniqueperiod'} = [$helper->{'VARS'}{'startunique'},
 				     $helper->{'VARS'}{'endunique'}];
 	}
+
         if ( $helper->{'VARS'}{'proctor'} =~/\S/ ) {
 	    my @names;
 	    foreach my $user (split(/\|\|\|/, $helper->{'VARS'}{'proctor'})) {
@@ -231,6 +280,27 @@
 	    my %proctors = map { ($_,1) } @names;
 	    $slot{'proctor'}=join(',',sort(keys(%proctors)));
 	}
+
+        if ( $helper->{'VARS'}{'allowedsections'} =~/\S/ ) {
+	    $slot{'allowedsections'}=
+		join(',',sort(split(/\|\|\|/,
+				    $helper->{'VARS'}{'allowedsections'})));
+	}
+
+        if ( $helper->{'VARS'}{'allowedusers'} =~/\S/ ) {
+
+	    my @names;
+	    foreach my $item (split(/\|\|\|/,
+				    $helper->{'VARS'}{'allowedusers'})) {
+		my ($uname,$udomain)=split(/:/,$item);
+		push(@names,"$uname:$udomain");
+	    }
+
+	    # make sure the choices are unique
+	    my %users = map { ($_,1) } @names;
+	    $slot{'allowedusers'}=join(',',sort(keys(%users)));
+	}
+
         my $cname = $env{'course.'.$env{'request.course.id'}.'.num'};
         my $cdom  = $env{'course.'.$env{'request.course.id'}.'.domain'};
 
@@ -243,7 +313,8 @@
                  "\n".'<li> Type: '.$slot{'type'}.'</li>';
         my %labels =
              map {($_->[0],$_->[1])} &Apache::slotrequest::csvupload_fields();
-        foreach my $which ('ip','description','maxspace','secret','symb') {
+        foreach my $which ('ip','description','maxspace','secret','symb',
+			   'allowedsections','allowedusers') {
 	    if (exists($slot{$which})) {
 		$result.="\n".'<li> '.$labels{$which}.': '.
 		    &HTML::Entities::encode($slot{$which}).'</li>';