[freeside-commits] branch master updated. e2ee874843b19f6c5221f5ce0af90979dba34e57

[Mitch Jackson]

The branch, master has been updated
       via  e2ee874843b19f6c5221f5ce0af90979dba34e57 (commit)
      from  04c717c096f556a9ba0e8c4a137e867029c78a1b (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 e2ee874843b19f6c5221f5ce0af90979dba34e57
Author: Mitch Jackson <mitch at freeside.biz>
Date:   Fri Jun 29 13:04:33 2018 -0500

    RT# 30783 Clean up json code for free_addrs

diff --git a/FS/FS/svc_IP_Mixin.pm b/FS/FS/svc_IP_Mixin.pm
index 0fce86c1d..b6af0a67b 100644
--- a/FS/FS/svc_IP_Mixin.pm
+++ b/FS/FS/svc_IP_Mixin.pm
@@ -132,7 +132,7 @@ sub _used_addresses {
   #       parameter to bypass FS::Record objects creation and just
   #       return hashrefs from DBI.  200,000 hashrefs are many seconds faster
   #       than 200,000 FS::Record objects
-  my %qsearch = (
+  my %qsearch_param = (
       table     => $class->table,
       select    => $ip_field,
       hashref   => \%qsearch,
@@ -140,7 +140,8 @@ sub _used_addresses {
   );
   if ( $octets ) {
     my $block_str = join('.', (split(/\D/, $block_na->first))[0..$octets-1]);
-    $qsearch{extra_sql} .= " AND $ip_field LIKE ".dbh->quote("${block_str}.%");
+    $qsearch_param{extra_sql}
+      .= " AND $ip_field LIKE ".dbh->quote("${block_str}.%");
   }
 
   if ( $block->ip_netmask % 8 ) {
@@ -154,7 +155,7 @@ sub _used_addresses {
 
   return
     map { $_->$ip_field }
-    qsearch( \%qsearch );
+    qsearch( \%qsearch_param );
 }
 
 sub _is_used {
diff --git a/httemplate/elements/tr-select-router_block_ip.html b/httemplate/elements/tr-select-router_block_ip.html
index 535e953c4..eac41cfad 100644
--- a/httemplate/elements/tr-select-router_block_ip.html
+++ b/httemplate/elements/tr-select-router_block_ip.html
@@ -71,7 +71,7 @@ function populate_ip_select() {
 % }
   if ( blocknum && $.isNumeric(blocknum) && ! e.is(':hidden')) {
     $.getJSON(
-      '<% $p %>json/free_addresses_in_block.json.html',
+      '<% $p %>misc/xmlhttp-free_addresses_in_block.json.html',
       {blocknum: blocknum},
       function(ip_json) {
         $.each( ip_json, function(idx, val) {
diff --git a/httemplate/json/free_addresses_in_block.json.html b/httemplate/misc/xmlhttp-free_addresses_in_block.json.html
similarity index 94%
rename from httemplate/json/free_addresses_in_block.json.html
rename to httemplate/misc/xmlhttp-free_addresses_in_block.json.html
index 6785aac6b..801718d35 100644
--- a/httemplate/json/free_addresses_in_block.json.html
+++ b/httemplate/misc/xmlhttp-free_addresses_in_block.json.html
@@ -3,7 +3,7 @@
   Unless block is larger than /24 - Does somebody really want to populate
   65k addresses into a HTML selectbox?
 </%doc>
-<% encode_rest($json) %>\
+<% encode_json($json) %>\
 <%init>
 
 my $json = [];

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

Summary of changes:
 FS/FS/svc_IP_Mixin.pm                                              | 7 ++++---
 httemplate/elements/tr-select-router_block_ip.html                 | 2 +-
 .../xmlhttp-free_addresses_in_block.json.html}                     | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)
 rename httemplate/{json/free_addresses_in_block.json.html => misc/xmlhttp-free_addresses_in_block.json.html} (94%)