[freeside-commits] branch master updated. 9a9eb2c2fcda78be903c418239b9a51b18c51d33

[Mitch Jackson]

The branch, master has been updated
       via  9a9eb2c2fcda78be903c418239b9a51b18c51d33 (commit)
      from  a2403695de3d9802bb5ce10bf64f64d8358141a9 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9a9eb2c2fcda78be903c418239b9a51b18c51d33
Author: Mitch Jackson <mitch at freeside.biz>
Date:   Fri Nov 9 15:09:37 2018 -0500

    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 

-----------------------------------------------------------------------

Summary of changes:
 httemplate/elements/tr-select-reason.html |  1 +
 httemplate/elements/xmlhttp.html          | 11 +++++++++--
 2 files changed, 10 insertions(+), 2 deletions(-)