[freeside-commits] branch FREESIDE_4_BRANCH updated. 56424799d5480be18cf21207d1e7f319d2719f35

[Mitch Jackson]

The branch, FREESIDE_4_BRANCH has been updated
       via  56424799d5480be18cf21207d1e7f319d2719f35 (commit)
      from  7f1bc260ca39c9a8ab53d24b51dc5610d893a76a (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 56424799d5480be18cf21207d1e7f319d2719f35
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 b0aa94af5..8cb922381 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(-)