Variables wrong value fixes

hannu · Mar 26, 2015 · 1014333 · 1014333
1 parent e3f01ba
commit 1014333
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 16 deletions.
diff --git a/lib/app/js/app.js b/lib/app/js/app.js
@@ -138,10 +138,16 @@ angular.module('sgApp', [
       if (!str) {
         return '';
       }
-      angular.forEach(variables, function(variable) {
-        if (str.match(new RegExp('\\$' + variable.name + ';', 'g'))) {
-          str = str.replace(new RegExp('\\$' + variable.name, 'g'), variable.value);
-        }
+
+      var sortedVariables;
+      if (variables) {
+        sortedVariables = variables.slice().sort(function(a, b) {
+          return b.name.length - a.name.length;
+        });
+      }
+
+      angular.forEach(sortedVariables, function(variable) {
+        str = str.replace(new RegExp('\\$' + variable.name, 'g'), variable.value);
       });
       return str;
     };

diff --git a/test/angular/unit/app.test.js b/test/angular/unit/app.test.js
@@ -59,22 +59,12 @@ describe('sgApp module', function() {
     });
 
     it('should set variables correctly if the first vairbale name is seconds substring', function() {
-      var input = 'background: $bgColor; color: $bgColor;',
+      var input = 'background: $bgColor; color: $bgColor-light;',
         variables = [
           {name: 'bgColor', value: '#FF0000'},
           {name: 'bgColor-light', value: '#00FF00'}
         ],
-        result = 'background: #FF0000; color: #FF0000;';
-      expect(setVariables(input, variables)).to.eql(result);
-    });
-
-    it('should set variables correctly if the second vairbale name has first vairbale name as substring', function() {
-      var input = 'background: $bgColor-light; color: $bgColor-light;',
-        variables = [
-          {name: 'bgColor', value: '#FF0000'},
-          {name: 'bgColor-light', value: '#00FF00'}
-        ],
-        result = 'background: #00FF00; color: #00FF00;';
+        result = 'background: #FF0000; color: #00FF00;';
       expect(setVariables(input, variables)).to.eql(result);
     });