Update builds controller to use getPreferredVersion

Author: benjaminapetersen

app/scripts/controllers/builds.js

@@ -8,7 +8,17 @@
  * Controller of the openshiftConsole
  */
 angular.module('openshiftConsole')
-  .controller('BuildsController', function ($routeParams, $scope, DataService, $filter, LabelFilter, Logger, $location, BuildsService, ProjectsService) {
+  .controller('BuildsController', function (
+    $routeParams,
+    $scope,
+    APIService,
+    DataService,
+    $filter,
+    LabelFilter,
+    Logger,
+    $location,
+    BuildsService,
+    ProjectsService) {
     $scope.projectName = $routeParams.project;
     $scope.builds = {};
     $scope.unfilteredBuildConfigs = {};
@@ -20,6 +30,9 @@ angular.module('openshiftConsole')
 
     var buildConfigForBuild = $filter('buildConfigForBuild');
 
+    $scope.buildsVersion = APIService.getPreferredVersion('builds');
+    $scope.buildConfigsVersion = APIService.getPreferredVersion('buildconfigs');
+
     var watches = [];
 
     ProjectsService
@@ -29,7 +42,7 @@ angular.module('openshiftConsole')
 
         var isPipeline = $filter('isJenkinsPipelineStrategy');
 
-        watches.push(DataService.watch("builds", context, function(builds) {
+        watches.push(DataService.watch($scope.buildsVersion, context, function(builds) {
           // Filter out pipeline builds, which have a separate page.
           $scope.builds = _.omitBy(builds.by("metadata.name"), isPipeline);
           $scope.emptyMessage = "No builds to show";
@@ -39,7 +52,7 @@ angular.module('openshiftConsole')
           Logger.log("builds (subscribe)", $scope.builds);
         }));
 
-        watches.push(DataService.watch("buildconfigs", context, function(buildConfigs) {
+        watches.push(DataService.watch($scope.buildConfigsVersion, context, function(buildConfigs) {
           // Filter out pipeline builds, which have a separate page.
           $scope.unfilteredBuildConfigs = _.omitBy(buildConfigs.by("metadata.name"), isPipeline);
           LabelFilter.addLabelSuggestionsFromResources($scope.unfilteredBuildConfigs, $scope.labelSuggestions);

dist/scripts/scripts.js

@@ -5139,45 +5139,47 @@ a.toggle.roles = !a.toggle.roles, a.toggle.roles ? a.filteredRoles = t : (a.filt
 });
 });
 }));
-} ]), angular.module("openshiftConsole").controller("BuildsController", [ "$routeParams", "$scope", "DataService", "$filter", "LabelFilter", "Logger", "$location", "BuildsService", "ProjectsService", function(e, t, n, a, r, o, i, s, c) {
+} ]), angular.module("openshiftConsole").controller("BuildsController", [ "$routeParams", "$scope", "APIService", "DataService", "$filter", "LabelFilter", "Logger", "$location", "BuildsService", "ProjectsService", function(e, t, n, a, r, o, i, s, c, l) {
 t.projectName = e.project, t.builds = {}, t.unfilteredBuildConfigs = {}, t.buildConfigs = void 0, t.labelSuggestions = {}, t.alerts = t.alerts || {}, t.emptyMessage = "Loading...", t.latestByConfig = {};
-var l = a("buildConfigForBuild"), u = [];
-c.get(e.project).then(_.spread(function(e, i) {
-function c(e) {
-var n = r.getLabelSelector();
+var u = r("buildConfigForBuild");
+t.buildsVersion = n.getPreferredVersion("builds"), t.buildConfigsVersion = n.getPreferredVersion("buildconfigs");
+var d = [];
+l.get(e.project).then(_.spread(function(e, n) {
+function s(e) {
+var n = o.getLabelSelector();
 if (n.isEmpty()) return !0;
-var a = l(e) || "";
+var a = u(e) || "";
 return a && t.unfilteredBuildConfigs[a] ? !!t.buildConfigs[a] : n.matches(e);
 }
-function d(e) {
-if (l(e)) return !1;
-var t = r.getLabelSelector();
+function l(e) {
+if (u(e)) return !1;
+var t = o.getLabelSelector();
 return !!t.isEmpty() || t.matches(e);
 }
 function p() {
-t.latestByConfig = s.latestBuildByConfig(t.builds, c), t.buildsNoConfig = _.pickBy(t.builds, d), angular.forEach(t.buildConfigs, function(e, n) {
+t.latestByConfig = c.latestBuildByConfig(t.builds, s), t.buildsNoConfig = _.pickBy(t.builds, l), angular.forEach(t.buildConfigs, function(e, n) {
 t.latestByConfig[n] = t.latestByConfig[n] || null;
 });
 }
 function m() {
 var e = _.omitBy(t.latestByConfig, _.isNull);
-!r.getLabelSelector().isEmpty() && _.isEmpty(t.buildConfigs) && _.isEmpty(e) ? t.alerts.builds = {
+!o.getLabelSelector().isEmpty() && _.isEmpty(t.buildConfigs) && _.isEmpty(e) ? t.alerts.builds = {
 type: "warning",
 details: "The active filters are hiding all builds."
 } : delete t.alerts.builds;
 }
 t.project = e;
-var f = a("isJenkinsPipelineStrategy");
-u.push(n.watch("builds", i, function(e) {
-t.builds = _.omitBy(e.by("metadata.name"), f), t.emptyMessage = "No builds to show", p(), r.addLabelSuggestionsFromResources(t.builds, t.labelSuggestions), o.log("builds (subscribe)", t.builds);
-})), u.push(n.watch("buildconfigs", i, function(e) {
-t.unfilteredBuildConfigs = _.omitBy(e.by("metadata.name"), f), r.addLabelSuggestionsFromResources(t.unfilteredBuildConfigs, t.labelSuggestions), r.setLabelSuggestions(t.labelSuggestions), t.buildConfigs = r.getLabelSelector().select(t.unfilteredBuildConfigs), p(), m(), o.log("buildconfigs (subscribe)", t.buildConfigs);
-})), r.onActiveFiltersChanged(function(e) {
+var f = r("isJenkinsPipelineStrategy");
+d.push(a.watch(t.buildsVersion, n, function(e) {
+t.builds = _.omitBy(e.by("metadata.name"), f), t.emptyMessage = "No builds to show", p(), o.addLabelSuggestionsFromResources(t.builds, t.labelSuggestions), i.log("builds (subscribe)", t.builds);
+})), d.push(a.watch(t.buildConfigsVersion, n, function(e) {
+t.unfilteredBuildConfigs = _.omitBy(e.by("metadata.name"), f), o.addLabelSuggestionsFromResources(t.unfilteredBuildConfigs, t.labelSuggestions), o.setLabelSuggestions(t.labelSuggestions), t.buildConfigs = o.getLabelSelector().select(t.unfilteredBuildConfigs), p(), m(), i.log("buildconfigs (subscribe)", t.buildConfigs);
+})), o.onActiveFiltersChanged(function(e) {
 t.$apply(function() {
 t.buildConfigs = e.select(t.unfilteredBuildConfigs), p(), m();
 });
 }), t.$on("$destroy", function() {
-n.unwatchAll(u);
+a.unwatchAll(d);
 });
 }));
 } ]), angular.module("openshiftConsole").controller("PipelinesController", [ "$filter", "$routeParams", "$scope", "Constants", "Navigate", "BuildsService", "DataService", "Logger", "ProjectsService", function(e, t, n, a, r, o, i, s, c) {