Skip to content

Commit

Permalink
H#898 update error messages (#940)
Browse files Browse the repository at this point in the history 
* #898 Add new error messages and prevent empty rule string input

* #898 fix - can't add rule after warning toast
  • Loading branch information
@mh0203 @alchan-lee
mh0203 authored and alchan-lee committed Nov 29, 2018
1 parent e01b561 commit 1afab01
Show file tree
Hide file tree
Showing 4 changed files with 115 additions and 2 deletions.
14 changes: 14 additions & 0 deletions ...n/dataflow/dataflow-detail/component/edit-dataflow-rule/edit-dataflow-rule-2.component.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -583,6 +583,7 @@ export class EditDataflowRule2Component extends AbstractPopupComponent implement
public addRule() {

if( this._isExecAddRule ) {
this.editorUseFlag = false;
return;
}

Expand All @@ -598,6 +599,7 @@ export class EditDataflowRule2Component extends AbstractPopupComponent implement

if (isUndefined(this.ruleVO['command']) || '' === this.ruleVO['command']) {
Alert.warning(this.translateService.instant('msg.dp.alert.no.data'));
this._isExecAddRule = false;
return;
}

Expand All @@ -615,6 +617,7 @@ export class EditDataflowRule2Component extends AbstractPopupComponent implement
} else { // Using editor
if (this.inputRuleCmd === '') {
Alert.warning(this.translateService.instant('msg.dp.alert.editor.warn'));
this._isExecAddRule = false;
return;
}
rule = {
Expand All @@ -625,6 +628,17 @@ export class EditDataflowRule2Component extends AbstractPopupComponent implement
};
}
if (!isUndefined(rule)) {

let isErrorCommand : boolean = true;
for(var ind in this.commandList) {
if ( rule.ruleString.indexOf(this.commandList[ind].command) > -1 ) isErrorCommand = false;
}
if (isErrorCommand){
this._isExecAddRule = false;
Alert.error(this.translateService.instant('msg.dp.alert.command.error'));
return;
}

this.applyRule(rule);
}
} // function - addRule
Expand Down
51 changes: 50 additions & 1 deletion discovery-frontend/src/assets/i18n/en.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -855,9 +855,58 @@
"msg.dp.alert.check.ignore.char" : "Check ignore between characters",
"msg.dp.alert.check.insert.expression" : "Please enter the expression",
"msp.dp.alert.col.empty" : "Column name must not be empty",
"msg.dp.alert.teddy.parse.failed.by.delete.expression" : "Please check delete expression",
"msg.dp.alert.merge.col.error" : "Please check new column name",

"msg.dp.alert.teddy.parse.failed" : "Rule parsing failed.",
"msg.dp.alert.teddy.parse.failed.by.rulename": "Unsupported rule name.",
"msg.dp.alert.teddy.parse.failed.by.move.before.and.after": "Move rule parsing failed. Check the reference column field.",
"msg.dp.alert.teddy.parse.failed.by.move.expression": "Move rule parsing failed.",
"msg.dp.alert.teddy.parse.failed.by.keep.expression": "Keep rule parsing failed. Check the condition expression.",
"msg.dp.alert.teddy.parse.failed.by.delete.expression": "Delete rule parsing failed. Check the condition expression.",
"msg.dp.alert.teddy.parse.failed.by.flatten.col": "Flatten rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.header.rownum": "Header rule parsing failed. Check the target row number.",
"msg.dp.alert.teddy.parse.failed.by.rename.col": "Rename rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.rename.to": "Rename rule parsing failed. Check the new name.",
"msg.dp.alert.teddy.parse.failed.by.sort.type": "Sort rule parsing failed. Check the sort type.",
"msg.dp.alert.teddy.parse.failed.by.sort.order": "Sort rule parsing failed. Check the sort order.",
"msg.dp.alert.teddy.parse.failed.by.drop.col": "Drop rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.replace.col": "Replace rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.replace.on": "Replace rule parsing failed. Check the pattern value.",
"msg.dp.alert.teddy.parse.failed.by.settype.col": "Settype rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.settype.type": "Settype rule parsing failed. Check the new type value.",
"msg.dp.alert.teddy.parse.failed.by.setformat.col": "Setformat rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.setformat.format": "Setformat rule parsing failed. Check the format value.",
"msg.dp.alert.teddy.parse.failed.by.set.col": "Set rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.set.value": "Set rule parsing failed. Check the expression value.",
"msg.dp.alert.teddy.parse.failed.by.countpattern.col": "Countpattern rule parsing failed. Check the target column value.",
"msg.dp.alert.teddy.parse.failed.by.countpattern.on": "Countpattern rule parsing failed. Check the pattern value.",
"msg.dp.alert.teddy.parse.failed.by.derive.as": "Derive rule parsing failed. Check the new column name.",
"msg.dp.alert.teddy.parse.failed.by.derive.value": "Derive rule parsing failed. Check the expression value.",
"msg.dp.alert.teddy.parse.failed.by.merge.as": "Merge rule parsing failed. Check the new column name.",
"msg.dp.alert.teddy.parse.failed.by.merge.col": "Merge rule parsing failed. Check the target column names.",
"msg.dp.alert.teddy.parse.failed.by.merge.with": "Merge rule parsing failed. Check the delimiter.",
"msg.dp.alert.teddy.parse.failed.by.unnest.idx": "Unnest rule parsing failed. Check the idx value.",
"msg.dp.alert.teddy.parse.failed.by.unnest.into": "Unnest rule parsing failed. Check the into value.",
"msg.dp.alert.teddy.parse.failed.by.extract.col": "Extract rule parsing failed. Check the target column name.",
"msg.dp.alert.teddy.parse.failed.by.extract.limit": "Extract rule parsing failed. Check the limit value.",
"msg.dp.alert.teddy.parse.failed.by.extract.on": "Extract rule parsing failed. Check the pattern value.",
"msg.dp.alert.teddy.parse.failed.by.aggregate.group": "Aggregate rule parsing failed. Check the group value",
"msg.dp.alert.teddy.parse.failed.by.aggregate.value": "Aggregate rule parsing failed. Check the expression(value).",
"msg.dp.alert.teddy.parse.failed.by.split.col": "Split rule parsing failed. Check the target column.",
"msg.dp.alert.teddy.parse.failed.by.split.limit": "Split rule parsing failed. Check the number of times(limit).",
"msg.dp.alert.teddy.parse.failed.by.split.on": "Split rule parsing failed. Check the pattern value.",
"msg.dp.alert.teddy.parse.failed.by.nest.as": "Nest rule parsing failed. Check the new column name.",
"msg.dp.alert.teddy.parse.failed.by.nest.col": "Nest rule parsing failed. Check the target columns.",
"msg.dp.alert.teddy.parse.failed.by.nest.into": "Nest rule parsing failed. Check the Type(into) value.",
"msg.dp.alert.teddy.parse.failed.by.pivot.col": "Pivot rule parsing failed. Check the target columns.",
"msg.dp.alert.teddy.parse.failed.by.pivot.group": "Pivot rule parsing failed. Check the group values.",
"msg.dp.alert.teddy.parse.failed.by.pivot.value": "Pivot rule parsing failed. Check the expression values.",
"msg.dp.alert.teddy.parse.failed.by.unpivot.col": "Unpivot rule parsing failed. Check the target column.",
"msg.dp.alert.teddy.parse.failed.by.unpivot.groupevery": "Unpivot rule parsing failed. Check the group-every value.",
"msg.dp.alert.teddy.parse.failed.by.join.dataset2": "Join rule parsing failed. Check the second dataset.",
"msg.dp.alert.teddy.parse.failed.by.join.jointype": "Join rule parsing failed. Check the join type value.",
"msg.dp.alert.teddy.parse.failed.by.window.value": "Check expression value of window rule",

"msg.dp.ui.no.snapshot.history" : "No snapshot create history",
"msg.dp.ui.snapshot.cancel.confirm" : "Sure to cancel snapshot creation?",
"msg.dp.ui.creating.snapshot" : "Creating a snapshot",
Expand Down
51 changes: 50 additions & 1 deletion discovery-frontend/src/assets/i18n/ko.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -854,9 +854,58 @@
"msg.dp.alert.check.ignore.char" : "다음 문자 사이 무시를 확인하세요",
"msg.dp.alert.check.insert.expression" : "수식을 입력하십시요",
"msp.dp.alert.col.empty" : "컬럼 이름을 반드시 입력해야합니다",
"msg.dp.alert.teddy.parse.failed.by.delete.expression" : "Delete 수식을 확인하세요",
"msg.dp.alert.merge.col.error" : "새로운 컬럼 이름을 확인하세요",

"msg.dp.alert.teddy.parse.failed" : "Rule 해석에 실패하였습니다.",
"msg.dp.alert.teddy.parse.failed.by.rulename": "지원하지 않는 Rule입니다.",
"msg.dp.alert.teddy.parse.failed.by.move.before.and.after" : "Move rule 해석에 실패하였습니다. Reference column 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.move.expression": "Move rule 해석에 실패하였습니다.",
"msg.dp.alert.teddy.parse.failed.by.keep.expression": "Keep rule 해석에 실패하였습니다. 조건문을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.delete.expression": "Delete rule 해석에 실패하였습니다. 조건문을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.flatten.col": "Flatten rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.header.rownum": "Header rule 해석에 실패하였습니다. 대상 행을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.rename.col": "Rename rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.rename.to": "Rename rule 해석에 실패하였습니다. 새 컬럼 이름을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.sort.type": "Sort rule 해석에 실패하였습니다. 정렬 type 값을 확인하세.",
"msg.dp.alert.teddy.parse.failed.by.sort.order": "Sort rule 해석에 실패하였습니다. Order 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.drop.col":"Drop rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.replace.col": "Replace rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.replace.on": "Replace rule 해석에 실패하였습니다. Pattern 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.settype.col": "Settype rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.settype.type": "Settype rule 해석에 실패하였습니다. 새 Type을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.setformat.col": "Setformat rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.setformat.format": "Setformat rule 해석에 실패하였습니다. Format 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.set.col": "Set rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.set.value": "Set rule 해석에 실패하였습니다. Expression 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.countpattern.col": "Countpattern rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.countpattern.on": "Countpattern rule 해석에 실패하였습니다. Pattern 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.derive.as": "Derive rule 해석에 실패하였습니다. 새 컬럼 이름을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.derive.value": "Derive rule 해석에 실패하였습니다. Expression 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.merge.as": "Merge rule 해석에 실패하였습니다. 새 컬럼 이름을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.merge.col": "Merge rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.merge.with": "Merge rule 해석에 실패하였습니다. 구분자(Delimiter)를 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.unnest.idx": "Unnest rule 해석에 실패하였습니다. idx 값을 확인하세요",
"msg.dp.alert.teddy.parse.failed.by.unnest.into": "Unnest rule 해석에 실패하였습니다. into 값을 확인하세.",
"msg.dp.alert.teddy.parse.failed.by.extract.col": "Extract rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.extract.limit": "Extract rule 해석에 실패하였습니다. 추출 횟수(limit)를 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.extract.on": "Extract rule 해석에 실패하였습니다. Pattern 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.aggregate.group": "Aggregate rule 해석에 실패하였습니다. 그룹 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.aggregate.value": "Aggregate rule 해석에 실패하였습니다. Expression(value) 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.split.col": "Split rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.split.limit": "Split rule 해석에 실패하였습니다. 분할 횟수(limit)를 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.split.on": "Split rule 해석에 실패하였습니다. Pattern 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.nest.as": "Nest rule 해석에 실패하였습니다. 새 컬럼 이름을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.nest.col": "Nest rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.nest.into": "Nest rule 해석에 실패하였습니다. Type 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.pivot.col": "Pivot rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.pivot.group": "Pivot rule 해석에 실패하였습니다. 그룹 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.pivot.value": "Pivot rule 해석에 실패하였습니다. Expression(value) 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.unpivot.col": "Unpivot rule 해석에 실패하였습니다. 대상 컬럼을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.unpivot.groupevery": "Unpivot rule 해석에 실패하였습니다. group-every 값을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.join.dataset2": "Join rule 해석에 실패하였습니다. 두번째 Dataset을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.join.jointype": "Join rule 해석에 실패하였습니다. Join 타입을 확인하세요.",
"msg.dp.alert.teddy.parse.failed.by.window.value": "Window Rule의 Expression 문법을 확인하세요",

"msg.dp.ui.no.snapshot.history" : "스냅 샷 생성 기록 없음",
"msg.dp.ui.snapshot.cancel.confirm" : "스냅 샷 생성을 취소 하시겠습니까?",
"msg.dp.ui.creating.snapshot" : "스냅샷을 생성하고있습니다",
Expand Down
1 change: 1 addition & 0 deletions ...erver/src/main/java/app/metatron/discovery/domain/dataprep/exceptions/PrepMessageKey.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ public enum PrepMessageKey {
MSG_DP_ALERT_TEDDY_PARSE_FAILED_BY_UNPIVOT_GROUPEVERY( "msg.dp.alert.teddy.parse.failed.by.unpivot.groupevery"),
MSG_DP_ALERT_TEDDY_PARSE_FAILED_BY_JOIN_DATASET2( "msg.dp.alert.teddy.parse.failed.by.join.dataset2"),
MSG_DP_ALERT_TEDDY_PARSE_FAILED_BY_JOIN_JOINTYPE( "msg.dp.alert.teddy.parse.failed.by.join.jointype"),
MSG_DP_ALERT_TEDDY_PARSE_FAILED_BY_WINDOW_VALUE( "msg.dp.alert.teddy.parse.failed.by.window.value"),
MSG_DP_ALERT_TEDDY_COLUMN_NOT_FOUND( "msg.dp.alert.teddy.column.not.found"),
MSG_DP_ALERT_TEDDY_NOT_SUPPORTED_TYPE( "msg.dp.alert.teddy.not.supported.type"),
MSG_DP_ALERT_TEDDY_QUERY_FAILED( "msg.dp.alert.teddy.query.failed"),
Expand Down

0 comments on commit 1afab01

Please sign in to comment.