Merge pull request #547 from jeff-phillips-18/wizard

Update wizard button components to fix minify/mangle issue

Author: dtaylor113

src/wizard/wizard-buttons.js

@@ -1,48 +1,112 @@
 (function () {
   'use strict';
-  function pfWizardButtonComponent (action) {
-    angular.module('patternfly.wizard')
-      .component(action, {
-        bindings: {
-          callback: "=?"
-        },
-        controller: function ($element, $scope) {
-          var ctrl = this;
-
-          var findWizard = function (scope) {
-            var wizard;
-
-            if (scope) {
-              if (angular.isDefined(scope.wizard)) {
-                wizard = scope.wizard;
-              } else {
-                wizard = findWizard(scope.$parent);
-              }
-            }
-
-            return wizard;
-          };
-
-          ctrl.$onInit = function () {
-            $scope.wizard = findWizard($scope);
-          };
-
-          ctrl.$postLink = function () {
-            $element.on("click", function (e) {
-              e.preventDefault();
-              $scope.$apply(function () {
-                // scope apply in button module
-                $scope.wizard[action.replace("pfWiz", "").toLowerCase()](ctrl.callback);
-              });
-            });
-          };
-        }
+
+  var findWizard = function (scope) {
+    var wizard;
+
+    if (scope) {
+      if (angular.isDefined(scope.wizard)) {
+        wizard = scope.wizard;
+      } else {
+        wizard = findWizard(scope.$parent);
+      }
+    }
+
+    return wizard;
+  };
+
+  var setupCallback = function (scope, button, fnName, callback) {
+    button.on("click", function (e) {
+      e.preventDefault();
+      scope.$apply(function () {
+        // scope apply in button module
+        scope.wizard[fnName](callback);
       });
-  }
+    });
+  };
+
+  angular.module('patternfly.wizard').component('pfWizNext', {
+    bindings: {
+      callback: "=?"
+    },
+    controller: function ($element, $scope) {
+      var ctrl = this;
+
+      ctrl.$onInit = function () {
+        $scope.wizard = findWizard($scope);
+      };
+
+      ctrl.$postLink = function () {
+        setupCallback($scope, $element, 'next', ctrl.callback);
+      };
+    }
+  });
+
+  angular.module('patternfly.wizard').component('pfWizPrevious', {
+    bindings: {
+      callback: "=?"
+    },
+    controller: function ($element, $scope) {
+      var ctrl = this;
+
+      ctrl.$onInit = function () {
+        $scope.wizard = findWizard($scope);
+      };
+
+      ctrl.$postLink = function () {
+        setupCallback($scope, $element, 'previous', ctrl.callback);
+      };
+    }
+  });
+
+  angular.module('patternfly.wizard').component('pfWizFinish', {
+    bindings: {
+      callback: "=?"
+    },
+    controller: function ($element, $scope) {
+      var ctrl = this;
+
+      ctrl.$onInit = function () {
+        $scope.wizard = findWizard($scope);
+      };
+
+      ctrl.$postLink = function () {
+        setupCallback($scope, $element, 'finish', ctrl.callback);
+      };
+    }
+  });
+
+  angular.module('patternfly.wizard').component('pfWizCancel', {
+    bindings: {
+      callback: "=?"
+    },
+    controller: function ($element, $scope) {
+      var ctrl = this;
+
+      ctrl.$onInit = function () {
+        $scope.wizard = findWizard($scope);
+      };
+
+      ctrl.$postLink = function () {
+        setupCallback($scope, $element, 'cancel', ctrl.callback);
+      };
+    }
+  });
+
+  angular.module('patternfly.wizard').component('pfWizReset', {
+    bindings: {
+      callback: "=?"
+    },
+    controller: function ($element, $scope) {
+      var ctrl = this;
+
+      ctrl.$onInit = function () {
+        $scope.wizard = findWizard($scope);
+      };
 
-  pfWizardButtonComponent('pfWizNext');
-  pfWizardButtonComponent('pfWizPrevious');
-  pfWizardButtonComponent('pfWizFinish');
-  pfWizardButtonComponent('pfWizCancel');
-  pfWizardButtonComponent('pfWizReset');
+      ctrl.$postLink = function () {
+        setupCallback($scope, $element, 'reset', ctrl.callback);
+      };
+    }
+  });
 })();