Github user ahgittin commented on a diff in the pull request:
https://github.com/apache/brooklyn-ui/pull/72#discussion_r219855211
--- Diff:
ui-modules/blueprint-composer/app/components/providers/blueprint-service.provider.js
---
@@ -287,8 +297,28 @@ function BlueprintService($log, $q, $sce, paletteApi,
iconGenerator, dslService)
}
break;
case 'regex':
- if (!entity.config.has(config.name) ||
!angular.isDefined(entity.config.get(config.name)) || !(new
RegExp(constraint).test(entity.config.get(config.name)))) {
- message =
`<samp>${config.name}</samp> does not match the required format:
<samp>${config.constraints.regex}</samp>`;
+ if (entity.config.has(config.name) &&
angular.isDefined(entity.config.get(config.name)) && !(new
RegExp(args).test(entity.config.get(config.name)))) {
+ message =
`<samp>${config.name}</samp> does not match the required format:
<samp>${args}</samp>`;
+ }
+ break;
+ case 'forbiddenIf':
+ if (entity.config.get(config.name) &&
entity.config.get(args)) {
--- End diff --
yes, these probably should be `isSet(key)` where we introduce `isSet` to
mean `config.has(key) && isDefined(config.get(key))` (as used in regex)
---
