--- loncom/interface/loncourserespicker.pm 2013/11/13 01:40:07 1.8 +++ loncom/interface/loncourserespicker.pm 2013/11/13 13:32:33 1.9 @@ -1,6 +1,6 @@ # The LearningOnline Network # -# $Id: loncourserespicker.pm,v 1.8 2013/11/13 01:40:07 raeburn Exp $ +# $Id: loncourserespicker.pm,v 1.9 2013/11/13 13:32:33 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -466,7 +466,7 @@ sub create_picker { $display .= ''; my $scripttag = &respicker_javascript($startcount,$numcount,$context,$formname,\%children, - \%hierarchy,\@checked_maps); + \%hierarchy,\@checked_maps,$numhome); if ($context eq 'dumpdocs') { return $scripttag.$display; } @@ -498,7 +498,7 @@ sub create_picker { sub respicker_javascript { my ($startcount,$numitems,$context,$formname,$children,$hierarchy, - $checked_maps) = @_; + $checked_maps,$numhome) = @_; return unless ((ref($children) eq 'HASH') && (ref($hierarchy) eq 'HASH') && (ref($checked_maps) eq 'ARRAY')); my ($elem,$nested,$nameforelem); @@ -591,11 +591,13 @@ EXTRA } elsif ($context eq 'dumpdocs') { my $blankmsg = &mt('An item selected has no filename set in the "Save as ..." column.'); my $dupmsg = &mt('Items selected for copying need unique filenames in the "Save as ..." column.'); + my $homemsg = &mt('An authoring space needs to be selected.'); $scripttag .= <<"EXTRA"; function checkUnique(form,field) { var duplicate = 0; var blank = 0; + var numhome = '$numhome'; if (field.length > 0) { for (i=0; i 1) { + if (!form.authorspace.options[form.authorspace.selectedIndex].value) { + alert('$homemsg'); + return false; + } + } return true; }