Skip to content
This repository was archived by the owner on May 14, 2025. It is now read-only.

Commit e10e1d1

Browse files
BoykoAlexghillert
authored andcommitted
Exclude destination and explicit tap from support of short and long name alias to fix regression
1 parent e988396 commit e10e1d1

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

ui/app/scripts/stream/controllers/properties-dialog.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,8 @@ define([], function () {
6767
}
6868

6969

70-
var propertiesSchemaPromise = cell.attr('metadata').get('properties');
70+
var metadata = cell.attr('metadata');
71+
var propertiesSchemaPromise = metadata.get('properties');
7172
$scope.cgbusy = propertiesSchemaPromise;
7273
propertiesSchemaPromise.then(function (schemaProperties) {
7374
var properties = {};
@@ -81,7 +82,9 @@ define([], function () {
8182
// If the user specifies a name in the DSL then that 'alias' should be
8283
// used when converting the properties back to text. By default the
8384
// short-name will be used if the user hasn't specified anything.
84-
var nameInUse = schema.name;
85+
// The alias is applicable to sources, processors and sinks.
86+
// Exclusions are: explicit taps and destinations. The entire other group.
87+
var nameInUse = metadata.group === 'other' ? schema.id : schema.name;
8588
var props = cell.attr('props');
8689
if (props.hasOwnProperty(key)) { // long-name, eg. 'trigger.cron'
8790
specifiedValue = props[key];

0 commit comments

Comments
 (0)