algairim commented on code in PR #328:
URL: https://github.com/apache/brooklyn-ui/pull/328#discussion_r886934742
##########
ui-modules/blueprint-composer/app/components/catalog-saver/catalog-saver.modal.template.html:
##########
@@ -60,7 +60,7 @@ <h3 class="modal-title">{{title}}</h3>
<label class="control-label">Bundle ID</label>
<div class="input-group">
<span class="input-group-addon">catalog-bom-</span>
- <input ng-model="config.bundle" ng-disabled="state.saving"
class="form-control" name="bundle" ng-pattern="state.pattern" autofocus
placeholder="{{ defaultBundle || 'E.g. my-bundle-id' }}"/>
+ <input ng-model="config.bundle" ng-disabled="isUpdate() ||
state.saving" class="form-control" name="bundle" ng-pattern="state.pattern"
autofocus placeholder="{{ defaultBundle || 'E.g. my-bundle-id' }}"/>
Review Comment:
@ahgittin, instead of suppressing duplicate version check if ID is changed,
I suggest to block bundle ID change all together when template update launched
from the catalog
Changing bundle ID on template update creates a mess with versions in the
catalog, where a "new" bundle holds information about previous versions that do
not exist this *new* bundle, and vice versa.
I did not see how such a mess is possible to create via CLI.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@brooklyn.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
