Merge pull request #50 from qSebastiaNp/master

Fixes issue #45, maybe also #42
johnitvn · Feb 16, 2016 · 5100474 · 5100474
2 parents c15bdd0 + 9e603d0
commit 5100474
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 13 deletions.
diff --git a/src/assets/ModalRemote.js b/src/assets/ModalRemote.js
@@ -315,11 +315,13 @@ function ModalRemote(modalId) {
                 // Test if browser supports FormData which handles uploads
                 if (window.FormData) {
                     data = new FormData($('#ModalRemoteConfirmForm')[0]);
-                    data.append('pks', selectedIds.join());
+                    if (typeof selectedIds !== 'undefined' && selectedIds)
+                        data.append('pks', selectedIds.join());
                 } else {
                     // Fallback to serialize
                     data = $('#ModalRemoteConfirmForm');
-                    data.pks = selectedIds;
+                    if (typeof selectedIds !== 'undefined' && selectedIds)
+                        data.pks = selectedIds;
                     data = data.serializeArray();
                 }
 
@@ -340,15 +342,6 @@ function ModalRemote(modalId) {
             }
         );
 
-        // Close the user input form
-        $(this.footer).append('</form>');
-
-        if ($(this.content).find("form")[0] !== undefined) {
-            this.setupFormSubmit(
-                $(this.content).find("form")[0],
-                $(this.footer).find('[type="submit"]')[0]
-            );
-        }
     }
 
     /**
@@ -393,4 +386,4 @@ function ModalRemote(modalId) {
             );
         }
     }
-} // End of Object
+} // End of Object
diff --git a/src/assets/ModalRemote.min.js b/src/assets/ModalRemote.min.js