Merge pull request #357 from WazeDev/Beta

Beta
WazeDev · Dec 30, 2018 · 626d438 · 626d438
2 parents c9697b0 + c2f62b4
commit 626d438
Showing 1 changed file with 33 additions and 34 deletions.
diff --git a/WME-Place-Harmonizer.js b/WME-Place-Harmonizer.js
@@ -1,7 +1,7 @@
 // ==UserScript==
 // @name        WME Place Harmonizer
 // @namespace   WazeUSA
-// @version     1.3.134
+// @version     1.3.135
 // @description Harmonizes, formats, and locks a selected place
 // @author      WMEPH Development Group
 // @include     /^https:\/\/(www|beta)\.waze\.com\/(?!user\/)(.{2,6}\/)?editor\/?.*$/
@@ -418,25 +418,24 @@
 
         function buildSearchUrl(searchName, address) {
             searchName = searchName
-                .replace(/&/g, '%26')
-                .replace(/[ \/]/g, '%20')
+                .replace(/[\/]/g, ' ')
                 .trim();
             address = address
                 .replace(/No street, /, '')
                 .replace(/No address/, '')
-                .replace(/ /g, '%20')
-                .replace(/CR-/g, 'County%20Rd%20')
-                .replace(/SR-/g, 'State%20Hwy%20')
-                .replace(/US-/g, 'US%20Hwy%20')
-                .replace(/ CR /g, '%20County%20Rd%20')
-                .replace(/ SR /g, '%20State%20Hwy%20')
-                .replace(/ US /g, '%20US%20Hwy%20')
-                .replace(/$CR /g, 'County%20Rd%20')
-                .replace(/$SR /g, 'State%20Hwy%20')
-                .replace(/$US /g, 'US%20Hwy%20')
+                .replace(/CR-/g, 'County Rd ')
+                .replace(/SR-/g, 'State Hwy ')
+                .replace(/US-/g, 'US Hwy ')
+                .replace(/ CR /g, ' County Rd ')
+                .replace(/ SR /g, ' State Hwy ')
+                .replace(/ US /g, ' US Hwy ')
+                .replace(/$CR /g, 'County Rd ')
+                .replace(/$SR /g, 'State Hwy ')
+                .replace(/$US /g, 'US Hwy ')
                 .trim();
 
-            return 'http://www.google.com/search?q=' + searchName + (address.length > 0 ? ',%20' + address: '');
+            searchName = encodeURIComponent(searchName + (address.length > 0 ? ', ' + address: ''));
+            return 'http://www.google.com/search?q=' + searchName;
         }
 
         function openWebSearch() {
@@ -3390,7 +3389,8 @@
                     // no field HL
                 }
             }
-            if (item.is2D()) {
+            // NOTE: do not use is2D() function. It doesn't seem to be 100% reliable.
+            if (!item.isPoint()) {
                 bannButt.pointNotArea = new Flag.PointNotArea();
             }
         } else if (item.isParkingLot() || (newName && newName.trim().length > 0)) {  // for non-residential places
@@ -4093,7 +4093,8 @@
 
 
             var isPoint = item.isPoint();
-            var isArea = item.is2D();
+            // NOTE: do not use is2D() function. It doesn't seem to be 100% reliable.
+            var isArea = !isPoint;
             var maxPointSeverity = 0;
             var maxAreaSeverity = 3;
             var highestCategoryLock = -1;
@@ -6326,30 +6327,28 @@
     // Build a Google search url based on place name and address
     function buildGLink(searchName,addr,HN) {
         var searchHN = '', searchStreet = '', searchCity = '';
-        searchName = searchName.replace(/&/g, '%26');
-        searchName = searchName.replace(/[ \/]/g, '%20');
+        searchName = searchName.replace(/\//g, ' ');
         if ('string' === typeof addr.street.name && addr.street.name !== null && addr.street.name !== '') {
-            searchStreet = addr.street.name + ',%20';
-        }
-        searchStreet = searchStreet.replace(/ /g, '%20');
-        searchStreet = searchStreet.replace(/CR-/g, 'County%20Rd%20');
-        searchStreet = searchStreet.replace(/SR-/g, 'State%20Hwy%20');
-        searchStreet = searchStreet.replace(/US-/g, 'US%20Hwy%20');
-        searchStreet = searchStreet.replace(/ CR /g, '%20County%20Rd%20');
-        searchStreet = searchStreet.replace(/ SR /g, '%20State%20Hwy%20');
-        searchStreet = searchStreet.replace(/ US /g, '%20US%20Hwy%20');
-        searchStreet = searchStreet.replace(/$CR /g, 'County%20Rd%20');
-        searchStreet = searchStreet.replace(/$SR /g, 'State%20Hwy%20');
-        searchStreet = searchStreet.replace(/$US /g, 'US%20Hwy%20');
+            searchStreet = addr.street.name + ', ';
+        }
+        searchStreet = searchStreet.replace(/CR-/g, 'County Rd ');
+        searchStreet = searchStreet.replace(/SR-/g, 'State Hwy ');
+        searchStreet = searchStreet.replace(/US-/g, 'US Hwy ');
+        searchStreet = searchStreet.replace(/ CR /g, ' County Rd ');
+        searchStreet = searchStreet.replace(/ SR /g, ' State Hwy ');
+        searchStreet = searchStreet.replace(/ US /g, ' US Hwy ');
+        searchStreet = searchStreet.replace(/$CR /g, 'County Rd ');
+        searchStreet = searchStreet.replace(/$SR /g, 'State Hwy ');
+        searchStreet = searchStreet.replace(/$US /g, 'US Hwy ');
         if ('string' === typeof HN && searchStreet !== '') {
-            searchHN = HN + '%20';
+            searchHN = HN + ' ';
         }
         if ('string' === typeof addr.city.attributes.name && addr.city.attributes.name !== '') {
-            searchCity = addr.city.attributes.name + ',%20';
+            searchCity = addr.city.attributes.name + ', ';
         }
-        searchCity = searchCity.replace(/ /g, '%20');
 
-        return 'http://www.google.com/search?q=' + searchName + (searchName ? ',%20' : '') + searchHN + searchStreet + searchCity + addr.state.name;
+        searchName = searchName + (searchName ? ', ' : '') + searchHN + searchStreet + searchCity + addr.state.name;
+        return 'http://www.google.com/search?q=' + encodeURIComponent(searchName);
     } // END buildGLink function
 
     // WME Category translation from Natural language to object language  (Bank / Financial --> BANK_FINANCIAL)