Merge pull request #644 from cdcabrera/issue-cleanup

Clean up for Non-breaking Changes

Author: jeff-phillips-18

src/canvas-view/canvas/canvas-viewmodel.js

@@ -51,7 +51,7 @@ var pfCanvas = {};
   pfCanvas.computeConnectorPos = function(node, connectorIndex, inputConnector) {
     return {
       x: node.x() + (inputConnector ? 0 : node.width ? node.width() : pfCanvas.defaultNodeWidth),
-      y: node.y() + pfCanvas.computeConnectorY(connectorIndex),
+      y: node.y() + pfCanvas.computeConnectorY(connectorIndex)
     };
   };
 
@@ -471,7 +471,7 @@ var pfCanvas = {};
     this.sourceCoord = function() {
       return {
         x: this.sourceCoordX(),
-        y: this.sourceCoordY(),
+        y: this.sourceCoordY()
       };
     };
 
@@ -494,7 +494,7 @@ var pfCanvas = {};
     this.destCoord = function() {
       return {
         x: this.destCoordX(),
-        y: this.destCoordY(),
+        y: this.destCoordY()
       };
     };
 
@@ -578,7 +578,7 @@ var pfCanvas = {};
   pfCanvas.computeConnectionSourceTangent = function(pt1, pt2) {
     return {
       x: pfCanvas.computeConnectionSourceTangentX(pt1, pt2),
-      y: pfCanvas.computeConnectionSourceTangentY(pt1, pt2),
+      y: pfCanvas.computeConnectionSourceTangentY(pt1, pt2)
     };
   };
 
@@ -602,7 +602,7 @@ var pfCanvas = {};
   pfCanvas.computeConnectionDestTangent = function(pt1, pt2) {
     return {
       x: pfCanvas.computeConnectionDestTangentX(pt1, pt2),
-      y: pfCanvas.computeConnectionDestTangentY(pt1, pt2),
+      y: pfCanvas.computeConnectionDestTangentY(pt1, pt2)
     };
   };
 
@@ -750,17 +750,17 @@ var pfCanvas = {};
 
       startNode = {
         nodeID: startNode.data.id,
-        connectorIndex: startConnectorIndex,
+        connectorIndex: startConnectorIndex
       };
 
       endNode = {
         nodeID: endNode.data.id,
-        connectorIndex: endConnectorIndex,
+        connectorIndex: endConnectorIndex
       };
 
       var connectionDataModel = {
         source: startConnectorType === 'output' ? startNode : endNode,
-        dest: startConnectorType === 'output' ? endNode : startNode,
+        dest: startConnectorType === 'output' ? endNode : startNode
       };
       connectionsDataModel.push(connectionDataModel);
 

src/card/basic/card.component.js

@@ -59,21 +59,21 @@
 
        $scope.data3 = {
          'used': '420',
-         'total': '500',
+         'total': '500'
        };
 
        $scope.title4 = 'Disk Usage';
        $scope.units4 = 'TB';
        $scope.data4 = {
          'used': '350',
-         'total': '500',
+         'total': '500'
        };
 
        $scope.title5 = 'Disk I/O';
        $scope.units5 = 'I/Ops';
        $scope.data5 = {
          'used': '450',
-         'total': '500',
+         'total': '500'
        };
 
        $scope.layoutInline = {

src/card/examples/card-timeframe.js

@@ -56,7 +56,7 @@
          'callBackFn': function () {
             alert("Footer Callback Fn Called");
           }
-       }
+       };
 
        $scope.filterConfigHeader = {
          'filters' : [{label:'Last 30 Days', value:'30'},
@@ -66,7 +66,7 @@
             alert("Header Filter Callback Fn Called for '" + f.label + "' value = " + f.value);
           },
         'position' : 'header'
-       }
+       };
 
        $scope.filterConfig = {
          'filters' : [{label:'Last 30 Days', value:'30'},
@@ -76,7 +76,7 @@
             alert("Filter Callback Fn Called for '" + f.label + "' value = " + f.value);
           },
         'defaultFilter' : '1'
-       }
+       };
      });
  </file>
  </example>

src/charts/c3/c3-chart.component.js

@@ -53,15 +53,15 @@
 
        $scope.getChart = function (chart) {
          $scope.chart = chart;
-       }
+       };
 
        $scope.focusUsed = function () {
          $scope.chart.focus("Used");
-       }
+       };
 
        $scope.updateAvailable = function (val) {
          $scope.available =  $scope.total - $scope.used;
-       }
+       };
 
        $scope.submitform = function (val) {
          console.log("submitform");

src/charts/topology/examples/topology-view.js

@@ -106,7 +106,7 @@
           "miq_id": 10000000000061,
           "status": "Unknown",
           "display_kind": "Service"
-        },
+        }
       },
       "relations": [
         {
@@ -186,7 +186,7 @@
           "icon": "",
           "fontfamily": "PatternFlyIcons-webfont"
         }
-      },
+      }
     });
 
     $rootScope.data = datasets[index];

src/charts/utilization-bar/utilization-bar-chart.component.js

@@ -107,21 +107,21 @@
 
     $scope.data3 = {
       'used': '420',
-      'total': '500',
+      'total': '500'
     };
 
     $scope.title4 = 'Disk Usage';
     $scope.units4 = 'TB';
     $scope.data4 = {
       'used': '350',
-      'total': '500',
+      'total': '500'
     };
 
     $scope.title5 = 'Disk I/O';
     $scope.units5 = 'I/Ops';
     $scope.data5 = {
       'used': '450',
-      'total': '500',
+      'total': '500'
     };
 
     $interval(function () {

src/form/examples/form-buttons.js

@@ -33,7 +33,7 @@
 
    <file name="script.js">
      angular.module( 'patternfly.form' ).controller( 'FormButtonCtrl', function( $scope, $timeout, $element ) {
-       $scope.status = 'Not yet Saved'
+       $scope.status = 'Not yet Saved';
        $scope.working = false;
 
        $scope.save = function (item) {

src/form/remaining-chars-count/remaining-chars-count.directive.js

@@ -24,7 +24,7 @@ angular.module('patternfly.form').directive('pfRemainingCharsCount', function ($
 
         // creating scope vars for unit testing
         $scope.remainingChars = remainingChars;
-        $scope.remainingCharsWarning = (remainingChars <= charsWarnRemaining ? true : false);
+        $scope.remainingCharsWarning = (remainingChars <= charsWarnRemaining);
 
         countRemainingFld.text(remainingChars);
         countRemainingFld.toggleClass('chars-warn-remaining-pf', remainingChars <= charsWarnRemaining);

src/modals/about-modal.component.js

@@ -44,10 +44,10 @@
          { name: 'User Role', value: 'Administrator' }];
        $scope.open = function () {
          $scope.isOpen = true;
-       }
+       };
        $scope.onClose = function() {
          $scope.isOpen = false;
-       }
+       };
      });
    </file>
  </example>

src/navigation/vertical-navigation.component.js

@@ -560,5 +560,5 @@ angular.module('patternfly.navigation').component('pfVerticalNavigation', {
         routeChangeListener();
       }
     };
-  },
+  }
 });