Skip to content

Commit e564ac8

Browse files
jeff-phillips-18jeff-phillips-18
committed
Fix bugzilla 1507822 - Update plan info on service instance update
1 parent 685f44d commit e564ac8

File tree

2 files changed

+63
-37
lines changed

2 files changed

+63
-37
lines changed

‎app/scripts/controllers/serviceInstance.js

+38-18
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ angular.module('openshiftConsole')
5151
var watches = [];
5252
var secretWatchers = [];
5353
var serviceClassPromise;
54+
var servicePlansPromise;
5455

5556
var serviceInstanceDisplayName = $filter('serviceInstanceDisplayName');
5657
var serviceInstanceReady = $filter('isServiceInstanceReady');
@@ -128,33 +129,52 @@ angular.module('openshiftConsole')
128129
updateParameterData();
129130
};
130131

131-
var updateServiceClass = function() {
132-
// If we've previously loaded the service class or a request is in flight, don't do anything.
133-
if (!$scope.serviceInstance || $scope.serviceClass || serviceClassPromise) {
132+
var updateServicePlan = function() {
133+
var servicePlanName = _.get($scope.serviceInstance, 'spec.clusterServicePlanRef.name');
134+
$scope.plan = _.find($scope.servicePlans, function(plan) {
135+
return _.get(plan, 'metadata.name') === servicePlanName;
136+
});
137+
updateParameterSchema();
138+
updateEditable();
139+
};
140+
141+
var updateServicePlans = function() {
142+
if (!$scope.serviceClass || servicePlansPromise) {
134143
return;
135144
}
136145

137-
serviceClassPromise = ServiceInstancesService.fetchServiceClassForInstance($scope.serviceInstance).then(function (serviceClass) {
138-
$scope.serviceClass = serviceClass;
139-
$scope.displayName = serviceInstanceDisplayName($scope.serviceInstance, $scope.serviceClass);
140-
141-
updateBreadcrumbs();
142-
serviceClassPromise = null;
143-
144-
Catalog.getServicePlansForServiceClass($scope.serviceClass).then(function (plans) {
145-
plans = plans.by('metadata.name');
146-
146+
if (!$scope.servicePlans) {
147+
servicePlansPromise = Catalog.getServicePlansForServiceClass($scope.serviceClass).then(function (plans) {
147148
var servicePlanName = _.get($scope.serviceInstance, 'spec.clusterServicePlanRef.name');
148-
$scope.servicePlans = _.reject(plans, function(plan) {
149+
$scope.servicePlans = _.reject(plans.by('metadata.name'), function(plan) {
149150
return _.get(plan, 'status.removedFromBrokerCatalog') && (plan.metadata.name !== servicePlanName);
150151
});
151152

152-
$scope.plan = plans[servicePlanName];
153+
updateServicePlan();
154+
servicePlansPromise = null;
155+
});
156+
} else {
157+
updateServicePlan();
158+
}
159+
};
160+
161+
var updateServiceClass = function() {
162+
if (!$scope.serviceInstance || serviceClassPromise) {
163+
return;
164+
}
165+
166+
if ($scope.serviceClass) {
167+
updateServicePlans();
168+
} else {
169+
serviceClassPromise = ServiceInstancesService.fetchServiceClassForInstance($scope.serviceInstance).then(function (serviceClass) {
170+
$scope.serviceClass = serviceClass;
171+
$scope.displayName = serviceInstanceDisplayName($scope.serviceInstance, $scope.serviceClass);
153172

154-
updateParameterSchema();
155-
updateEditable();
173+
updateBreadcrumbs();
174+
serviceClassPromise = null;
175+
updateServicePlans();
156176
});
157-
});
177+
}
158178
};
159179

160180
var serviceResolved = function(serviceInstance, action) {

‎dist/scripts/scripts.js

+25-19
Original file line numberDiff line numberDiff line change
@@ -6356,15 +6356,15 @@ e.showParameterValues = !e.showParameterValues;
63566356
}, e.closeEditDialog = function() {
63576357
e.editDialogShown = !1;
63586358
};
6359-
var m, p = [], f = [], g = t("serviceInstanceDisplayName"), v = t("isServiceInstanceReady"), h = a.getPreferredVersion("servicebindings");
6359+
var m, p, f = [], g = [], v = t("serviceInstanceDisplayName"), h = t("isServiceInstanceReady"), y = a.getPreferredVersion("servicebindings");
63606360
e.serviceInstancesVersion = a.getPreferredVersion("serviceinstances");
6361-
var y = function() {
6361+
var b = function() {
63626362
e.breadcrumbs.push({
63636363
title: e.displayName
63646364
});
6365-
}, b = function() {
6365+
}, C = function() {
63666366
if (e.serviceInstance && e.parameterSchema) {
6367-
s.unwatchAll(f), f = [], e.allowParametersReveal = o.canI("secrets", "get", e.projectName), e.parameterData = {}, e.opaqueParameterKeys = [];
6367+
s.unwatchAll(g), g = [], e.allowParametersReveal = o.canI("secrets", "get", e.projectName), e.parameterData = {}, e.opaqueParameterKeys = [];
63686368
var t = e.allowParametersReveal ? "" : "*****";
63696369
_.each(_.keys(_.get(e.parameterSchema, "properties")), function(n) {
63706370
e.parameterData[n] = t;
@@ -6373,7 +6373,7 @@ var n = _.get(e.serviceInstance, "status.externalProperties.parameters", {});
63736373
_.each(_.keys(n), function(t) {
63746374
"<redacted>" === n[t] ? e.parameterData[t] = "*****" : (e.parameterData[t] = n[t], e.opaqueParameterKeys.push(t));
63756375
}), e.allowParametersReveal && _.each(_.get(e.serviceInstance, "spec.parametersFrom"), function(t) {
6376-
f.push(s.watchObject("secrets", _.get(t, "secretKeyRef.name"), e.projectContext, function(n) {
6376+
g.push(s.watchObject("secrets", _.get(t, "secretKeyRef.name"), e.projectContext, function(n) {
63776377
try {
63786378
var a = JSON.parse(u.decodeSecretData(n.data)[t.secretKeyRef.key]);
63796379
_.extend(e.parameterData, a);
@@ -6383,34 +6383,40 @@ c.warn("Unable to load parameters from secret " + _.get(t, "secretKeyRef.name"),
63836383
}));
63846384
});
63856385
}
6386-
}, C = function() {
6386+
}, S = function() {
63876387
if (e.plan && e.serviceClass && e.serviceInstance) {
63886388
var t = _.get(e.plan, "spec.instanceUpdateParameterSchema"), n = _.size(_.get(t, "properties")) > 0 || _.get(e.serviceClass, "spec.planUpdatable") && _.size(e.servicePlans) > 1;
6389-
e.editAvailable = n && v(e.serviceInstance) && !_.get(e.serviceInstance, "metadata.deletionTimestamp");
6389+
e.editAvailable = n && h(e.serviceInstance) && !_.get(e.serviceInstance, "metadata.deletionTimestamp");
63906390
}
6391-
}, S = function() {
6392-
e.parameterFormDefinition = angular.copy(_.get(e.plan, "spec.externalMetadata.schemas.service_instance.update.openshift_form_definition")), e.parameterSchema = _.get(e.plan, "spec.instanceCreateParameterSchema"), b();
63936391
}, w = function() {
6394-
!e.serviceInstance || e.serviceClass || m || (m = d.fetchServiceClassForInstance(e.serviceInstance).then(function(t) {
6395-
e.serviceClass = t, e.displayName = g(e.serviceInstance, e.serviceClass), y(), m = null, i.getServicePlansForServiceClass(e.serviceClass).then(function(t) {
6396-
t = t.by("metadata.name");
6392+
e.parameterFormDefinition = angular.copy(_.get(e.plan, "spec.externalMetadata.schemas.service_instance.update.openshift_form_definition")), e.parameterSchema = _.get(e.plan, "spec.instanceCreateParameterSchema"), C();
6393+
}, k = function() {
6394+
var t = _.get(e.serviceInstance, "spec.clusterServicePlanRef.name");
6395+
e.plan = _.find(e.servicePlans, function(e) {
6396+
return _.get(e, "metadata.name") === t;
6397+
}), w(), S();
6398+
}, P = function() {
6399+
e.serviceClass && !p && (e.servicePlans ? k() : p = i.getServicePlansForServiceClass(e.serviceClass).then(function(t) {
63976400
var n = _.get(e.serviceInstance, "spec.clusterServicePlanRef.name");
6398-
e.servicePlans = _.reject(t, function(e) {
6401+
e.servicePlans = _.reject(t.by("metadata.name"), function(e) {
63996402
return _.get(e, "status.removedFromBrokerCatalog") && e.metadata.name !== n;
6400-
}), e.plan = t[n], S(), C();
6401-
});
6403+
}), k(), p = null;
64026404
}));
6403-
}, k = function(t, n) {
6405+
}, j = function() {
6406+
e.serviceInstance && !m && (e.serviceClass ? P() : m = d.fetchServiceClassForInstance(e.serviceInstance).then(function(t) {
6407+
e.serviceClass = t, e.displayName = v(e.serviceInstance, e.serviceClass), b(), m = null, P();
6408+
}));
6409+
}, R = function(t, n) {
64046410
e.loaded = !0, e.serviceInstance = t, "DELETED" === n && (e.alerts.deleted = {
64056411
type: "warning",
64066412
message: "This provisioned service has been deleted."
6407-
}), w(), b(), C();
6413+
}), j(), C(), S();
64086414
};
64096415
l.get(n.project).then(_.spread(function(a, o) {
64106416
e.project = a, e.projectContext = o, s.get(e.serviceInstancesVersion, n.instance, o, {
64116417
errorNotification: !1
64126418
}).then(function(t) {
6413-
k(t), p.push(s.watchObject(e.serviceInstancesVersion, n.instance, o, k)), p.push(s.watch(h, o, function(n) {
6419+
R(t), f.push(s.watchObject(e.serviceInstancesVersion, n.instance, o, R)), f.push(s.watch(y, o, function(n) {
64146420
var a = n.by("metadata.name");
64156421
e.bindings = r.getBindingsForResource(a, t);
64166422
}));
@@ -6428,7 +6434,7 @@ message: "The service details could not be loaded.",
64286434
details: t("getErrorDetails")(n)
64296435
};
64306436
})), e.$on("$destroy", function() {
6431-
s.unwatchAll(p), s.unwatchAll(f);
6437+
s.unwatchAll(f), s.unwatchAll(g);
64326438
});
64336439
} ]), angular.module("openshiftConsole").controller("SecretsController", [ "$routeParams", "$scope", "DataService", "ProjectsService", function(e, t, n, a) {
64346440
t.projectName = e.project, t.secretsByType = {}, a.get(e.project).then(_.spread(function(e, a) {

0 commit comments

Comments
 (0)