fix submission bug w/chrome, RT#12638

diff --git a/httemplate/elements/form-file_upload.html b/httemplate/elements/form-file_upload.html
index 4ab70ad..45b6c97 100644 (file)
--- a/httemplate/elements/form-file_upload.html
+++ b/httemplate/elements/form-file_upload.html
@@ -72,7 +72,7 @@ Example:
       ACTION   = "<% $fsurl %>misc/file-upload.html"
       METHOD   = "POST"
       ENCTYPE  = "multipart/form-data"
       ACTION   = "<% $fsurl %>misc/file-upload.html"
       METHOD   = "POST"
       ENCTYPE  = "multipart/form-data"

-      onSubmit = "return doUpload(this, <% $opt{key} %>gotUploaded)"
+      onSubmit = "<% $opt{onsubmit} %>return doUpload(this, <% $opt{key} %>gotUploaded)"

 >
 
 <%init>
 >
 
 <%init>


@@ -90,4 +90,6 @@ my $msg_or_url = $opt{message}
                      }
                    : $opt{url};
 
                      }
                    : $opt{url};
 

+$opt{onsubmit} .= ';' if $opt{onsubmit} && $opt{onsubmit} !~ /;\s*$/;
+

 </%init>
 </%init>

diff --git a/httemplate/misc/cdr-import.html b/httemplate/misc/cdr-import.html
index 7af6c52..05d0bbc 100644 (file)
--- a/httemplate/misc/cdr-import.html
+++ b/httemplate/misc/cdr-import.html
@@ -1,14 +1,14 @@
 <% include("/elements/header.html",'Call Detail Record Import') %>
   
 <% include("/elements/header.html",'Call Detail Record Import') %>
   

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'CDRImportForm',
-              'action'    => 'process/cdr-import.html',
-              'num_files' => 1,
-              'fields'    => [ 'format', 'cdrbatch', ],
-              'message'   => 'CDR import successful',
-              'url'       => $p."search/cdr.html?cdrbatch=$cdrbatch",
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'CDRImportForm',
+     'action'    => 'process/cdr-import.html',
+     'num_files' => 1,
+     'fields'    => [ 'format', 'cdrbatch', ],
+     'message'   => 'CDR import successful',
+     'url'       => $p."search/cdr.html?cdrbatch=$cdrbatch",
+     'onsubmit'  => "document.CDRImportForm.submitButton.disabled=true;",
+&>

 
 Import a file containing Call Detail Records (CDRs).<BR><BR>
 
 
 Import a file containing Call Detail Records (CDRs).<BR><BR>
 


@@ -36,9 +36,9 @@ Import a file containing Call Detail Records (CDRs).<BR><BR>
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             ID      = "submitButton"
+             NAME    = "submitButton"

              VALUE   = "Import file"
              VALUE   = "Import file"

-             onClick = "document.InventoryItemImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/cust_main-import.cgi b/httemplate/misc/cust_main-import.cgi
index 028876c..6646510 100644 (file)
--- a/httemplate/misc/cust_main-import.cgi
+++ b/httemplate/misc/cust_main-import.cgi
@@ -3,15 +3,15 @@
 Import a file containing customer records.
 <BR><BR>
 
 Import a file containing customer records.
 <BR><BR>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'CustomerImportForm',
-              'action'    => 'process/cust_main-import.cgi',
-              'num_files' => 1,
-              'fields'    => [ 'agentnum', 'custbatch', 'format' ],
-              'message'   => 'Customer import successful',
-              'url'       => $p."search/cust_main.html?custbatch=$custbatch",
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'CustomerImportForm',
+     'action'    => 'process/cust_main-import.cgi',
+     'num_files' => 1,
+     'fields'    => [ 'agentnum', 'custbatch', 'format' ],
+     'message'   => 'Customer import successful',
+     'url'       => $p."search/cust_main.html?custbatch=$custbatch",
+     'onsubmit'  => "document.CustomerImportForm.submitButton.disabled=true;",
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -69,9 +69,9 @@ Import a file containing customer records.
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             NAME    = "submitButton"
+             ID      = "submitButton"

              VALUE   = "Import file"
              VALUE   = "Import file"

-             onClick = "document.CustomerImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/cust_pkg-import.html b/httemplate/misc/cust_pkg-import.html
index b29884d..f86d317 100644 (file)
--- a/httemplate/misc/cust_pkg-import.html
+++ b/httemplate/misc/cust_pkg-import.html
@@ -3,15 +3,15 @@
 Import a file containing package records.
 <BR><BR>
 
 Import a file containing package records.
 <BR><BR>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'PackageImportForm',
-              'action'    => 'process/cust_pkg-import.html',
-              'num_files' => 1,
-              'fields'    => [ 'agentnum', 'pkgbatch', 'format' ],
-              'message'   => 'Package import successful',
-              'url'       => $p."search/cust_pkg.cgi?pkgbatch=$pkgbatch",
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'PackageImportForm',
+     'action'    => 'process/cust_pkg-import.html',
+     'num_files' => 1,
+     'fields'    => [ 'agentnum', 'pkgbatch', 'format' ],
+     'message'   => 'Package import successful',
+     'url'       => $p."search/cust_pkg.cgi?pkgbatch=$pkgbatch",
+     'onsubmit'  => "document.PackageImportForm.submitButton.disabled=true;"
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -49,9 +49,9 @@ Import a file containing package records.
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             NAME    = "submitButton"
+             ID      = "submitButton"

              VALUE   = "Import file"
              VALUE   = "Import file"

-             onClick = "document.PackageImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/inventory_item-import.html b/httemplate/misc/inventory_item-import.html
index d264baf..3789873 100644 (file)
--- a/httemplate/misc/inventory_item-import.html
+++ b/httemplate/misc/inventory_item-import.html
@@ -3,18 +3,18 @@
 Import a file containing <% PL($inventory_class->classname) %>, one per line.
 <BR><BR>
 
 Import a file containing <% PL($inventory_class->classname) %>, one per line.
 <BR><BR>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'InventoryItemImportForm',
-              'action'    => 'process/inventory_item-import.html',
-              'num_files' => 1,
-              #'fields'    => [ 'format', 'itembatch', 'classnum', 'agentnum' ],
-              'fields'    => [ 'format', 'classnum', 'agentnum', ],
-              'message'   => 'Inventory import successful',
-              #XXX redirect via $itembatch?  or just back to class browse?
-              #'url'       => $p."search/phone_avail.html?availbatch=$availbatch",
-              'url'       => $p."search/inventory_item.html?classnum=$classnum;avail=1",
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'InventoryImportForm',
+     'action'    => 'process/inventory_item-import.html',
+     'num_files' => 1,
+     #'fields'    => [ 'format', 'itembatch', 'classnum', 'agentnum' ],
+     'fields'    => [ 'format', 'classnum', 'agentnum', ],
+     'message'   => 'Inventory import successful',
+     #XXX redirect via $itembatch?  or just back to class browse?
+     #'url'       => $p."search/phone_avail.html?availbatch=$availbatch",
+     'url'       => $p."search/inventory_item.html?classnum=$classnum;avail=1",
+     'onsubmit'  => "document.InventoryImportForm.submitButton.disabled=true;",
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -37,10 +37,10 @@ Import a file containing <% PL($inventory_class->classname) %>, one per line.
 
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
 
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">

-      <INPUT TYPE    = "submit"
-             ID      = "submit"
-             VALUE   = "Import file"
-             onClick = "document.InventoryItemImportForm.submit.disabled=true;"
+      <INPUT TYPE  = "submit"
+             NAME  = "submitButton"
+             ID    = "submitButton"
+             VALUE = "Import file"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/part_device-import.html b/httemplate/misc/part_device-import.html
index 7bd6404..7cf8167 100644 (file)
--- a/httemplate/misc/part_device-import.html
+++ b/httemplate/misc/part_device-import.html
@@ -3,15 +3,15 @@
 Import a file containing phone device types, one per line.
 <BR><BR>
 
 Import a file containing phone device types, one per line.
 <BR><BR>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'PartDeviceImportForm',
-              'action'    => 'process/part_device-import.html',
-              'num_files' => 1,
-              'fields'    => [ 'format', ], 
-              'message'   => 'Device type import successful',
-              'url'       => $p.'browse/part_device.html',
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'PartDeviceImportForm',
+     'action'    => 'process/part_device-import.html',
+     'num_files' => 1,
+     'fields'    => [ 'format', ], 
+     'message'   => 'Device type import successful',
+     'url'       => $p.'browse/part_device.html',
+     'onsubmit'  => "document.PartDeviceImportForm.submitButton.disabled=true;",
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -26,9 +26,9 @@ Import a file containing phone device types, one per line.
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             ID      = "submitButton"
+             NAME    = "submitButton"

              VALUE   = "Import file"
              VALUE   = "Import file"

-             onClick = "document.PartDeviceImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/phone_avail-import.html b/httemplate/misc/phone_avail-import.html
index b6fea7e..0febfbc 100644 (file)
--- a/httemplate/misc/phone_avail-import.html
+++ b/httemplate/misc/phone_avail-import.html
@@ -3,15 +3,16 @@
 Import a file containing phone numbers (DIDs).
 <BR><BR>
 
 Import a file containing phone numbers (DIDs).
 <BR><BR>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'PhonenumImportForm',
-              'action'    => 'process/phone_avail-import.html',
-              'num_files' => 1,
-              'fields'    => [ 'format', 'availbatch', 'exportnum', 'countrycode', 'ordernum', 'confirmed', 'vendor_order_id' ],
-              'message'   => 'DID import successful',
-              'url'       => $p."search/phone_avail.html?availbatch=$availbatch",
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'PhonenumImportForm',
+     'action'    => 'process/phone_avail-import.html',
+     'num_files' => 1,
+     'fields'    => [ 'format', 'availbatch', 'exportnum', 'countrycode',
+                      'ordernum', 'confirmed', 'vendor_order_id' ],
+     'message'   => 'DID import successful',
+     'url'       => $p."search/phone_avail.html?availbatch=$availbatch",
+     'onsubmit'  => "document.PhonenumImportForm.submitButton.disabled=true;",
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -86,9 +87,9 @@ Import a file containing phone numbers (DIDs).
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             ID      = "submitButton"
+             NAME    = "submitButton"

              VALUE   = "Import file"
              VALUE   = "Import file"

-             onClick = "document.PhonenumImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/rate-import.html b/httemplate/misc/rate-import.html
index ae8ee69..538418d 100644 (file)
--- a/httemplate/misc/rate-import.html
+++ b/httemplate/misc/rate-import.html
@@ -1,14 +1,14 @@
 <% include("/elements/header.html",'Import Rate Plan') %>
 
 <% include("/elements/header.html",'Import Rate Plan') %>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'RateImportForm',
-              'action '   => 'process/rate-import.html',
-              'num_files' => 1,
-              'fields'    => [ 'ratename' ],
-              'message'   => 'Rate plan import successful',
-#              'url'       => $p."browse/rate_detail.cgi?ratenum=$ratenum", #XXX how?
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'RateImportForm',
+     'action '   => 'process/rate-import.html',
+     'num_files' => 1,
+     'fields'    => [ 'ratename' ],
+     'message'   => 'Rate plan import successful',
+     #'url'       => $p."browse/rate_detail.cgi?ratenum=$ratenum", #XXX how?
+     'onsubmit'  => "document.RateImportForm.submitButton.disabled=true;",
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -28,9 +28,9 @@
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             ID      = "submitButton"
+             NAME    = "submitButton"

              VALUE   = "Import rate plan"
              VALUE   = "Import rate plan"

-             onClick = "document.RateImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/rate_edit_excel.html b/httemplate/misc/rate_edit_excel.html
index 442d83a..e55c676 100644 (file)
--- a/httemplate/misc/rate_edit_excel.html
+++ b/httemplate/misc/rate_edit_excel.html
@@ -4,15 +4,15 @@
   <FONT COLOR="#FF0000">WARNING: This functionality does not yet preserve connection charges.</FONT><BR><BR>
 % }
 
   <FONT COLOR="#FF0000">WARNING: This functionality does not yet preserve connection charges.</FONT><BR><BR>
 % }
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'RateImportForm',
-              'action'    => 'process/rate_edit_excel.html',
-              'num_files' => 1,
-              'fields'    => [ 'format' ],
-              'message'   => 'Rate edit successful',
-              'url'       => $p."browse/rate_region.html",
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'RateImportForm',
+     'action'    => 'process/rate_edit_excel.html',
+     'num_files' => 1,
+     'fields'    => [ 'format' ],
+     'message'   => 'Rate edit successful',
+     'url'       => $p."browse/rate_region.html",
+     'onsubmit'  => "document.RateImportForm.submitButton.disabled=true;"
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -46,9 +46,9 @@
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
       <INPUT TYPE    = "submit"

-             ID      = "submit"
+             ID      = "submitButton"
+             NAME    = "submitButton"

              VALUE   = "Upload"
              VALUE   = "Upload"

-             onClick = "document.RateImportForm.submit.disabled=true;"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>

diff --git a/httemplate/misc/tax-import.cgi b/httemplate/misc/tax-import.cgi
index ceb7464..25414b2 100644 (file)
--- a/httemplate/misc/tax-import.cgi
+++ b/httemplate/misc/tax-import.cgi
@@ -3,14 +3,14 @@
 Import a CSV file set containing tax rate records.
 <BR><BR>
 
 Import a CSV file set containing tax rate records.
 <BR><BR>
 

-<% include( '/elements/form-file_upload.html',
-              'name'      => 'TaxRateUpload',
-              'action'    => 'process/tax-import.cgi', 
-              'num_files' => 6,
-              'fields'    => [ 'format', 'reload' ],
-              'message'   => 'Tax rates imported',
-          )
-%>
+<& /elements/form-file_upload.html,
+     'name'      => 'TaxRateUpload',
+     'action'    => 'process/tax-import.cgi', 
+     'num_files' => 6,
+     'fields'    => [ 'format', 'reload' ],
+     'message'   => 'Tax rates imported',
+     'onsubmit'  => "document.TaxRateUpload.submitButton.disabled=true;",
+&>

 
 <% &ntable("#cccccc", 2) %>
 
 
 <% &ntable("#cccccc", 2) %>
 


@@ -54,9 +54,10 @@ Import a CSV file set containing tax rate records.
 
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">
 
   <TR>
     <TD COLSPAN=2 ALIGN="center" STYLE="padding-top:6px">

-      <INPUT TYPE    = "submit"
-             VALUE   = "Import CSV files"
-             onClick = "document.TaxRateUpload.submit.disabled=true;"
+      <INPUT TYPE  = "submit"
+             NAME  = "submitButton"
+             ID    = "submitButton"
+             VALUE = "Import CSV files"

       >
     </TD>
   </TR>
       >
     </TD>
   </TR>