Fix codingjoe#309 -- handle submit buttons form* attributes

.gitignore

@@ -59,3 +59,4 @@ target/
 node_modules
 
 *.min.js
+.python-version

s3file/static/s3file/js/s3file.js

@@ -114,7 +114,7 @@
     form.appendChild(submitInput)
   }
 
-  function uploadS3Inputs (form) {
+  function uploadS3Inputs (form, submitter) {
     window.uploading = 0
     form.loaded = 0
     form.total = 0
@@ -136,6 +136,15 @@
       window.uploading += 1
       uploadFiles(form, input, input.name)
     })
+    // override form attributes with submit button attributes
+    if (submitter) {
+       form.action = submitter.getAttribute('formaction') || form.action
+       form.method = submitter.getAttribute('formmethod') || form.method
+       form.enctype = submitter.getAttribute('formEnctype') || form.enctype
+       form.formnovalidate = submitter.getAttribute('formnovalidate') || form.novalidate
+       form.target = submitter.getAttribute('formtarget') || form.target
+    }
+
     waitForAllFiles(form)
   }
 
@@ -147,7 +156,8 @@
     forms.forEach(function (form) {
       form.addEventListener('submit', function (e) {
         e.preventDefault()
-        uploadS3Inputs(e.target)
+        console.log(e.submitter)
+        uploadS3Inputs(e.target, e.submitter)
       })
       var submitButtons = form.querySelectorAll('input[type=submit], button[type=submit]')
       Array.from(submitButtons).forEach(function (submitButton) {

tests/test_forms.py

@@ -187,6 +187,7 @@ def test_file_insert_submit_value(
         with wait_for_page_load(driver, timeout=10):
             save_button.click()
         assert "save" in driver.page_source
+        assert "formaction" not in driver.page_source
 
         driver.get(live_server + self.create_url)
         file_input = driver.find_element(By.XPATH, "//input[@name='file']")
@@ -197,6 +198,7 @@ def test_file_insert_submit_value(
             save_button.click()
         assert "save_continue" in driver.page_source
         assert "continue_value" in driver.page_source
+        assert "formaction" in driver.page_source
 
     @pytest.mark.selenium
     def test_progress(self, driver, live_server, upload_file, freeze_upload_folder):

tests/testapp/templates/form.html

@@ -14,8 +14,8 @@
 <form method="post">
     {% csrf_token %}
     {{ form }}
-    <input type="submit" name="save" value="Save"/>
-    <button type="submit" name="save_continue" value="continue_value">Save &amp; continue</button>
+    <input type="submit" name="save" value="Save" />
+    <button type="submit" name="save_continue" value="continue_value" formaction="?formaction=1">Save &amp; continue</button>
     <div class="progress">
         <div class="progress-bar" role="progressbar" style="width: 0%;" aria-valuenow="0" aria-valuemin="0"
              aria-valuemax="100">0%

tests/testapp/views.py

@@ -26,6 +26,7 @@ def form_valid(self, form):
                     "file": self.request.FILES.getlist("file"),
                     "other_file": self.request.FILES.getlist("other_file"),
                 },
+                "QUERY_STRING": self.request.GET,
             },
             status=201,
             encoder=FileEncoder,
@@ -45,6 +46,7 @@ def form_valid(self, form):
                     "file": self.request.FILES.getlist("file"),
                     "other_file": self.request.FILES.getlist("other_file"),
                 },
+                "QUERY_STRING": self.request.GET,
             },
             status=201,
             encoder=FileEncoder,