From: Mitch Jackson <mitch@freeside.biz>
Date: Fri, 9 Nov 2018 20:09:37 +0000 (-0500)
Subject: RT# 81712 Reduce unnecessary xmlhttp calls
X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=commitdiff_plain;h=9a9eb2c2fcda78be903c418239b9a51b18c51d33

RT# 81712 Reduce unnecessary xmlhttp calls
---

diff --git a/httemplate/elements/tr-select-reason.html b/httemplate/elements/tr-select-reason.html
index a4e0c033e..64648ba54 100755
--- a/httemplate/elements/tr-select-reason.html
+++ b/httemplate/elements/tr-select-reason.html
@@ -43,6 +43,7 @@ Example:
 <& /elements/xmlhttp.html,
   url => $p . 'misc/xmlhttp-reason-hint.html',
   subs => [ 'get_hint' ],
+  skip_empty => 1,
 &>
 <SCRIPT TYPE="text/javascript">
   function <% $id %>_changed() {
diff --git a/httemplate/elements/xmlhttp.html b/httemplate/elements/xmlhttp.html
index e70871169..b2f265afb 100644
--- a/httemplate/elements/xmlhttp.html
+++ b/httemplate/elements/xmlhttp.html
@@ -8,8 +8,10 @@ Example:
     'subs' => [ 'subroutine' ],
 
     # optional
-    'method' => 'GET', #defaults to GET, could specify POST
-    'key'    => 'unique', #unique key
+    'method'     => 'GET',    # defaults to GET, could specify POST
+    'key'        => 'unique', # unique key
+    'skip_empty' => '1',      # When the given key value is null or = 0,
+                              #   skip making a useless http request
 
   );
 
@@ -43,6 +45,11 @@ my %initialized = ();#won't work if component is "preloaded"... so don't do that
             args = a;
             len = args.length - 1;
         }
+% if ( $opt{skip_empty} ) {
+        if ( args[0] == 0 || !args[0] ) {
+          return;
+        }
+% }
         for (var i = 0; i < len; i++) 
             content = content + "&arg=" + encodeURIComponent(args[i]);
         content = content.replace( /[+]/g, '%2B'); // fix unescaped plus signs