--- loncom/homework/structuretags.pm	2011/07/26 17:14:51	1.493
+++ loncom/homework/structuretags.pm	2011/09/13 15:40:37	1.496
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA 
 # definition of tags that give a structure to a document
 #
-# $Id: structuretags.pm,v 1.493 2011/07/26 17:14:51 raeburn Exp $
+# $Id: structuretags.pm,v 1.496 2011/09/13 15:40:37 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -129,8 +129,16 @@ sub homework_js {
 	<<'JS';
 <script type="text/javascript">
 // <![CDATA[
-function setSubmittedPart (part) {
-   this.document.lonhomework.submitted.value="part_"+part;
+function setSubmittedPart (part,prefix) {
+    if (typeof(prefix) == 'undefined') {
+        this.document.lonhomework.submitted.value="part_"+part;
+    } else {
+        for (var i=0;i<this.document.lonhomework.elements.length;i++) {
+            if (this.document.lonhomework.elements[i].name == prefix+'submitted') {
+                this.document.lonhomework.elements[i].value="part_"+part;
+            }
+        }
+    }
 }
 
 function image_response_click (which, e) {
@@ -164,166 +172,6 @@ function setmode(form,probmode) {
 ENDSCRIPT
 }
 
-sub file_delchk_js {
-    my $delfilewarn = &mt('You have indicated you wish to remove some files previously included in your submission.').'\\n'.
-                      &mt('Continue submission with these files removed?');
-    return <<"ENDSCRIPT";
-<script type="text/javascript">
-// <![CDATA[
-function file_deletion_check(formname,uploadid) {
-    var elemnum = formname.elements.length;
-    if (elemnum == 0) {
-        return true;
-    }
-    var alluploads = new Array();
-    if ((uploadid != '') && (uploadid != undefined)) {
-        alluploads.push(uploadid);
-    } else {
-        var uploadstr = /^HWFILE.+\$/;
-        for (var i=0; i<formname.elements.length; i++) {
-            var id = formname.elements[i].id;
-            if (id != '') {
-                if (uploadstr.test(id)) {
-                    if (formname.elements[i].type == 'file') {
-                        alluploads.push(id);
-                    }
-                }
-            }
-        }
-    }
-    for (var i=0; i<alluploads.length; i++) {
-        var delstr = new RegExp('^'+alluploads[i]+'_\\\\d+_delete\$');
-        var delboxes = new Array();
-        for (var j=0; j<formname.elements.length; j++) {
-            var id = formname.elements[j].id;
-            if (id != '') {
-                if (delstr.test(id)) {
-                    if (formname.elements[j].type == 'checkbox') {
-                        if (formname.elements[j].checked) {
-                            delboxes.push(id);
-                        }
-                    }
-                }
-            }
-        }
-        if (delboxes.length > 0) {
-            if (!confirm("$delfilewarn")) {
-                for (var j=0; j<delboxes.length; j++) {
-                    formname.elements[delboxes[j]].checked = false;
-                }
-                return false;
-            }
-        }
-    }
-    return true;
-}
-// ]]>
-</script>
-ENDSCRIPT
-}
-
-sub file_overwritechk_js {
-    my $overwritewarn = &mt('File(s) you uploaded for your submission will overwrite existing file(s) submitted for this item').'\\n'.
-                      &mt('Continue submission and overwrite the file(s)?');
-    return <<"ENDSCRIPT";
-<script type="text/javascript">
-// <![CDATA[
-function file_overwrite_check(formname,path,multiresp) {
-    var elemnum = formname.elements.length;
-    if (elemnum == 0) {
-        return true;
-    }
-    var alluploads = new Array();
-    var uploaded = new Array();
-    var uploadstr = /^HWFILE.+\$/;
-    var fnametrim = /[^\\/\\\\]+\$/;
-    for (var i=0; i<formname.elements.length; i++) {
-        var id = formname.elements[i].id;
-        if (id != '') {
-            if (uploadstr.test(id)) {
-                if (formname.elements[i].type == 'file') {
-                    alluploads.push(id);
-                    if ((formname.elements[i].value != undefined) &&
-                        (formname.elements[i].value != '')) {
-                        uploaded.push(id);
-                    }
-                }
-            }
-        }
-    }
-    for (var i=0; i<alluploads.length; i++) {
-        var delstr = new RegExp("^"+alluploads[i]+"_\\\\d+_delete\$");
-        var delboxes = new Array();
-        for (var j=0; j<formname.elements.length; j++) {
-            var id = formname.elements[j].id;
-            if (id != '') {
-                if (delstr.test(id)) {
-                    if (formname.elements[j].type == 'checkbox') {
-                        delboxes.push(id);
-                    }
-                }
-            }
-        }
-        var overwrites = new Array();
-        if (delboxes.length > 0) {
-            if ((formname.elements[alluploads[i]].value != undefined) &&
-                (formname.elements[alluploads[i]].value != '')) {
-                var filepath = formname.elements[alluploads[i]].value;
-                var newfilename = fnametrim.exec(filepath);
-                if (newfilename != null) {
-                    var filename = String(newfilename);
-                    var nospaces = filename.replace(/\\s+/g,'_');
-                    var nospecials = nospaces.replace(/[^\\/\\w\\.\\-]/g,'');
-                    var cleanfilename = nospecials.replace(/\\.(\\d+\\.)/g,"_\$1");
-                    if (cleanfilename != '') {
-                        var fullpath = path+"/"+cleanfilename;
-                        if (multiresp == 1) {
-                            var partid = String(alluploads[i]);
-                            var subdir = partid.replace(/^HWFILE/,'');
-                            if (subdir != "" && subdir != undefined) {
-                                fullpath = path+"/"+subdir+"/"+cleanfilename;
-                            }
-                        }
-                        for (var k=0; k<delboxes.length; k++) {
-                            if (fullpath == formname.elements[delboxes[k]].value) {
-                                var id = formname.elements[delboxes[k]].id;
-                                if (id != '') {
-                                    if (!formname.elements[delboxes[k]].checked) {
-                                        overwrites.push(id);
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-            if (overwrites.length > 0) {
-                if (confirm("$overwritewarn")) {
-                    var delcheck = file_deletion_check(formname,alluploads[i]);
-                    if (delcheck == false) {
-                        return false;
-                    }
-                } else {
-                    for (var n=0; n<overwrites.length; n++) {
-                        formname.elements[overwrites[n]].value = "";
-                    }
-                    return false;
-                }
-            } else {
-                var delcheck = file_deletion_check(formname);
-                if (delcheck == false) {
-                    return false;
-                }
-            }
-        }
-    }
-    return true;
-}
-// ]]>
-</script>
-ENDSCRIPT
-}
-
 sub page_start {
     my ($target,$token,$tagstack,$parstack,$parser,$safeeval,$name,
 	$extra_head)=@_;
@@ -350,42 +198,61 @@ sub page_start {
     }
     my $is_task = ($env{'request.uri'} =~ /\.task$/);
     my $needs_upload;
+    my ($symb)= &Apache::lonnet::whichuser();
+    my ($map,$resid,$resurl)=&Apache::lonnet::decode_symb($symb);
     if ($is_task) {
-        $extra_head .= &file_delchk_js();
+        $extra_head .= &Apache::lonhtmlcommon::file_submissionchk_js();
     } else {
         if (&Apache::lonnet::EXT("resource.$Apache::inputtags::part.uploadedfiletypes") ne '') {
-            $needs_upload = 1;
+            unless ($env{'request.state'} eq 'construct') {
+                my $navmap = Apache::lonnavmaps::navmap->new();
+                if (ref($navmap)) {
+                    my $mapres = $navmap->getResourceByUrl($map);
+                    my $is_page;
+                    if (ref($mapres)) {
+                        $is_page = $mapres->is_page();
+                    }
+                    unless ($is_page) {
+                        $needs_upload = 1;
+                    }
+                }
+            }
         } else {
             unless ($env{'request.state'} eq 'construct') {
-                my ($symb)= &Apache::lonnet::whichuser();
                 my $navmap = Apache::lonnavmaps::navmap->new();
                 if (ref($navmap)) {
-                    my $res = $navmap->getBySymb($symb);
-                    if (ref($res)) {
-                        my $partlist = $res->parts();
-                        if (ref($partlist) eq 'ARRAY') {
-                            foreach my $part (@{$partlist}) {
-                                my @types = $res->responseType($part);
-                                my @ids = $res->responseIds($part);
-                                for (my $i=0; $i < scalar(@ids); $i++) {
-                                    if ($types[$i] eq 'essay') {
-                                        my $partid = $part.'_'.$ids[$i];
-                                        if (&Apache::lonnet::EXT("resource.$partid.uploadedfiletypes") ne '') {
-                                            $needs_upload = 1;
-                                            last;
+                    my $mapres = $navmap->getResourceByUrl($map);
+                    my $is_page;
+                    if (ref($mapres)) {
+                        $is_page = $mapres->is_page();
+                    }
+                    unless ($is_page) {
+                        my $res = $navmap->getBySymb($symb);
+                        if (ref($res)) {
+                            my $partlist = $res->parts();
+                            if (ref($partlist) eq 'ARRAY') {
+                                foreach my $part (@{$partlist}) {
+                                    my @types = $res->responseType($part);
+                                    my @ids = $res->responseIds($part);
+                                    for (my $i=0; $i < scalar(@ids); $i++) {
+                                        if ($types[$i] eq 'essay') {
+                                            my $partid = $part.'_'.$ids[$i];
+                                            if (&Apache::lonnet::EXT("resource.$partid.uploadedfiletypes") ne '') {
+                                                $needs_upload = 1;
+                                                last;
+                                            }
                                         }
                                     }
                                 }
-                            }
-                        } 
+                            } 
+                        }
                     }
                 }
             }
         }
-    }
-    if ($needs_upload) {
-        $extra_head .= &file_overwritechk_js()."\n".
-                       &file_delchk_js();
+        if ($needs_upload) {
+            $extra_head .= &Apache::lonhtmlcommon::file_submissionchk_js();
+        }
     }
 
     my %body_args;
@@ -481,10 +348,8 @@ sub page_start {
         my ($symb,$courseid,$udom,$uname)=&Apache::lonnet::whichuser();
         my ($path,$multiresp) = 
             &Apache::loncommon::get_turnedin_filepath($symb,$uname,$udom);
-        if ($is_task) {
-            $form_tag_start .= ' onsubmit="return file_deletion_check(this);"';
-        } elsif ($needs_upload) {
-            $form_tag_start .= ' onsubmit="return file_overwrite_check(this,'."'$path','$multiresp'".');"';
+        if (($is_task) || ($needs_upload)) {
+            $form_tag_start .= ' onsubmit="return file_submission_check(this,'."'$path','$multiresp'".');"';
         }
 	$form_tag_start.='>'."\n";