Merge pull request #76 from digital11/master

IE8 Script error in segmentation - Kuddos @digital11
matomo-org · Jun 2, 2013 · 500e29f · 500e29f
2 parents 2bf8b29 + e405e21
commit 500e29f
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/plugins/SegmentEditor/templates/Segmentation.js b/plugins/SegmentEditor/templates/Segmentation.js
@@ -185,9 +185,9 @@ Segmentation = (function($) {
 
         var getSegmentFromId = function (id) {
             if(self.availableSegments.length > 0) {
-                for(var key in self.availableSegments)
+                for(var i = 0; i < self.availableSegments.length; i++)
                 {
-                    segment = self.availableSegments[key];
+                    segment = self.availableSegments[i];
                     if(segment.idsegment == id) {
                         return segment;
                     }
@@ -206,9 +206,9 @@ Segmentation = (function($) {
                             + ' ' + self.translations['General_DefaultAppended']
                             + '</span></li> ';
             if(self.availableSegments.length > 0) {
-                for(var key in self.availableSegments)
+                for(var i = 0; i < self.availableSegments.length; i++)
                 {
-                    segment = self.availableSegments[key];
+                    segment = self.availableSegments[i];
                     injClass = "";
                     if( segment.definition == self.currentSegmentStr){
                         injClass = 'class="segmentSelected"';
@@ -247,9 +247,9 @@ Segmentation = (function($) {
                         + '">' + self.translations['SegmentEditor_AddNewSegment']
                         + '</option>';
             segmentsDropdown.append(newOption);
-            for(var key in self.availableSegments)
+            for(var i = 0; i < self.availableSegments.length; i++)
             {
-                segment = self.availableSegments[key];
+                segment = self.availableSegments[i];
                 newOption = '<option data-idsegment="'+segment.idsegment+'" data-definition="'+(segment.definition).replace(/"/g, '&quot;')+'" title="'+segment.name+'">'+self.shortenSegmentName(segment.name)+'</option>';
                 segmentsDropdown.append(newOption);
             }