--- loncom/homework/inputtags.pm 2006/06/30 00:32:25 1.198 +++ loncom/homework/inputtags.pm 2006/06/30 04:56:56 1.199 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # input definitons # -# $Id: inputtags.pm,v 1.198 2006/06/30 00:32:25 albertel Exp $ +# $Id: inputtags.pm,v 1.199 2006/06/30 04:56:56 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -440,12 +440,12 @@ sub file_selector { ''. '
'; if ($Apache::lonhomework::history{"resource.$part.$id.portfiles"}=~/[^\s]/){ - my (@filelist,@bad_file_list); - foreach my $file (split(',',&unescape($Apache::lonhomework::history{"resource.$part.$id.portfiles"}))) { + my (@file_list,@bad_file_list); + foreach my $file (split(/\s*,\s*/,&unescape($Apache::lonhomework::history{"resource.$part.$id.portfiles"}))) { my (undef,undef,$domain,$user)=&Apache::lonxml::whichuser(); my $url="/uploaded/$domain/$user/portfolio$file"; my $icon=&Apache::loncommon::icon($url); - push(@filelist,''.$file.''); if (! &Apache::lonnet::stat_file($url)) { &Apache::lonnet::logthis("bad file is $url"); @@ -453,9 +453,15 @@ sub file_selector { '" border="0" />'.$file.''); } } - $result.=&mt("Portfolio files previously selected: [_1]",join(', ',@filelist)); + my $files = ''. + join(', ',@file_list). + ''; + $result.=&mt("Portfolio files previously selected: [_1]",$files); if (@bad_file_list) { - $result.='
'.&mt('These file(s) don\'t exist: [_1]',join(', ',@bad_file_list)); + my $bad_files = ''. + join(', ',@bad_file_list). + ''; + $result.='
'.&mt('These file(s) don\'t exist: [_1]',$bad_files); } } }