@spinnaker/core 0.22.2 → 0.23.1

package/src/widgets/accountRegionClusterSelector.component.js

@@ -2,6 +2,7 @@
 
 import { module } from 'angular';
 import _ from 'lodash';
+import { $log } from 'ngimport';
 
 import { AccountService } from '../account/AccountService';
 import { AppListExtractor } from '../application/listExtractor/AppListExtractor';
@@ -27,104 +28,106 @@ module(CORE_WIDGETS_ACCOUNTREGIONCLUSTERSELECTOR_COMPONENT, []).directive('accou
     templateUrl: require('./accountRegionClusterSelector.component.html'),
     controllerAs: 'vm',
     controller: function controller() {
-      this.clusterField = this.clusterField || 'cluster';
+      this.$onInit = () => {
+        this.clusterField = this.clusterField || 'cluster';
 
-      const vm = this;
+        const vm = this;
 
-      if (vm.showClusterSelect === undefined) {
-        vm.showClusterSelect = true;
-      }
-
-      const showAllRegions = vm.showAllRegions || false;
-
-      let isTextInputForClusterFiled;
-
-      let regions;
-
-      const setRegionList = () => {
-        const accountFilter = (cluster) => (cluster ? cluster.account === vm.component.credentials : true);
-        const regionList = AppListExtractor.getRegions([vm.application], accountFilter);
-        vm.regions = showAllRegions ? regions : regionList.length ? regionList : regions;
-        (vm.regions || []).sort();
-      };
-
-      const setClusterList = () => {
-        const regionField = this.singleRegion ? vm.component.region : vm.component.regions;
-        const clusterFilter = AppListExtractor.clusterFilterForCredentialsAndRegion(
-          vm.component.credentials,
-          regionField,
-        );
-        vm.clusterList = AppListExtractor.getClusters([vm.application], clusterFilter);
-      };
-
-      vm.regionChanged = () => {
-        setClusterList();
-        if (!isTextInputForClusterFiled && !_.includes(vm.clusterList, vm.component[this.clusterField])) {
-          vm.component[this.clusterField] = undefined;
+        if (vm.showClusterSelect === undefined) {
+          vm.showClusterSelect = true;
         }
-      };
 
-      const setToggledState = () => {
-        vm.regions = regions;
-        isTextInputForClusterFiled = true;
-      };
-
-      const setUnToggledState = () => {
-        vm.component[this.clusterField] = undefined;
-        isTextInputForClusterFiled = false;
-        setRegionList();
-      };
-
-      vm.clusterSelectInputToggled = (isToggled) => {
-        isToggled ? setToggledState() : setUnToggledState();
-      };
-
-      vm.clusterChanged = (clusterName) => {
-        const filterByCluster = AppListExtractor.monikerClusterNameFilter(clusterName);
-        const clusterMoniker = _.first(_.uniq(AppListExtractor.getMonikers([vm.application], filterByCluster)));
-        if (_.isNil(clusterMoniker)) {
-          //remove the moniker from the stage if one doesn't exist.
-          vm.component.moniker = undefined;
-        } else {
-          //clusters don't contain sequences, so null it out.
-          clusterMoniker.sequence = null;
-          vm.component.moniker = clusterMoniker;
-        }
-      };
-
-      vm.accountUpdated = () => {
-        vm.component[this.clusterField] = undefined;
-        setRegionList();
-        setClusterList();
-        if (vm.onAccountUpdate) {
-          vm.onAccountUpdate();
-        }
-      };
-
-      const init = () => {
-        AccountService.getUniqueAttributeForAllAccounts(vm.component.cloudProviderType, 'regions')
-          .then((allRegions) => {
-            regions = allRegions;
-
-            // TODO(duftler): Remove this once we finish deprecating the old style regions/zones in clouddriver GCE credentials.
-            const regionObjs = _.filter(regions, (region) => _.isObject(region));
-            if (regionObjs.length) {
-              const oldStyleRegions = _.chain(regionObjs)
-                .map((regionObj) => _.keys(regionObj))
-                .flatten()
-                .value();
-              regions = _.chain(regions).difference(regionObjs).union(oldStyleRegions).value();
-            }
-            return regions.sort();
-          })
-          .then((allRegions) => {
-            setRegionList();
-            setClusterList();
-            vm.regions = _.includes(vm.clusterList, vm.component[this.clusterField]) ? vm.regions : allRegions;
-          });
+        const showAllRegions = vm.showAllRegions || false;
+
+        let isTextInputForClusterFiled;
+
+        let regions;
+
+        const setRegionList = () => {
+          const accountFilter = (cluster) => (cluster ? cluster.account === vm.component.credentials : true);
+          const regionList = AppListExtractor.getRegions([vm.application], accountFilter);
+          vm.regions = showAllRegions ? regions : regionList.length ? regionList : regions;
+          (vm.regions || []).sort();
+        };
+
+        const setClusterList = () => {
+          const regionField = this.singleRegion ? vm.component.region : vm.component.regions;
+          const clusterFilter = AppListExtractor.clusterFilterForCredentialsAndRegion(
+            vm.component.credentials,
+            regionField,
+          );
+          vm.clusterList = AppListExtractor.getClusters([vm.application], clusterFilter);
+        };
+
+        vm.regionChanged = () => {
+          setClusterList();
+          if (!isTextInputForClusterFiled && !_.includes(vm.clusterList, vm.component[this.clusterField])) {
+            vm.component[this.clusterField] = undefined;
+          }
+        };
+
+        const setToggledState = () => {
+          vm.regions = regions;
+          isTextInputForClusterFiled = true;
+        };
+
+        const setUnToggledState = () => {
+          vm.component[this.clusterField] = undefined;
+          isTextInputForClusterFiled = false;
+          setRegionList();
+        };
+
+        vm.clusterSelectInputToggled = (isToggled) => {
+          isToggled ? setToggledState() : setUnToggledState();
+        };
+
+        vm.clusterChanged = (clusterName) => {
+          const filterByCluster = AppListExtractor.monikerClusterNameFilter(clusterName);
+          const clusterMoniker = _.first(_.uniq(AppListExtractor.getMonikers([vm.application], filterByCluster)));
+          if (_.isNil(clusterMoniker)) {
+            //remove the moniker from the stage if one doesn't exist.
+            vm.component.moniker = undefined;
+          } else {
+            //clusters don't contain sequences, so null it out.
+            clusterMoniker.sequence = null;
+            vm.component.moniker = clusterMoniker;
+          }
+        };
+
+        vm.accountUpdated = () => {
+          vm.component[this.clusterField] = undefined;
+          setRegionList();
+          setClusterList();
+          if (vm.onAccountUpdate) {
+            vm.onAccountUpdate();
+          }
+        };
+
+        const init = () => {
+          AccountService.getUniqueAttributeForAllAccounts(vm.component.cloudProviderType, 'regions')
+            .then((allRegions) => {
+              regions = allRegions;
+
+              // TODO(duftler): Remove this once we finish deprecating the old style regions/zones in clouddriver GCE credentials.
+              const regionObjs = _.filter(regions, (region) => _.isObject(region));
+              if (regionObjs.length) {
+                const oldStyleRegions = _.chain(regionObjs)
+                  .map((regionObj) => _.keys(regionObj))
+                  .flatten()
+                  .value();
+                regions = _.chain(regions).difference(regionObjs).union(oldStyleRegions).value();
+              }
+              return regions.sort();
+            })
+            .then((allRegions) => {
+              setRegionList();
+              setClusterList();
+              vm.regions = _.includes(vm.clusterList, vm.component[this.clusterField]) ? vm.regions : allRegions;
+            });
+        };
+
+        init();
       };
-
-      init();
     },
   };
 });