[
https://issues.redhat.com/browse/DROOLS-5062?page=com.atlassian.jira.plug...
]
Michael Anstis updated DROOLS-5062:
-----------------------------------
Description:
I understand [DROOLS-5017] has been re-purposed to allow Info and Warn level messages be
displayed on Stunner Validation dialog. That is fine.
But, imho, an UX point is not being considered, where is necessary instead.
By the DMN spec, a node should not have a prefix/trailing space, as space insensitivity
would mandate 1 trailing or 2 trailing spaces are simply ignored etc.
The engine is lenient to avoid this crashing the evaluation, but still is a bad designed
model to persist the node name (DRGElement name and its corresponding variable name) with
those extra spaces IMHO.
Further, I consider is just the Analyst which slipped a space.
In my perspective, the editor should just strip away the prefix/trailing space:
* While Importing a DMN model from file Upload
* -While editing a DRGElement and hitting OK to save its name- (See
[
DROOLS-5060|https://issues.redhat.com/browse/DROOLS-5060])
For your consideration, thanks.
was:
I understand [DROOLS-5017] has been re-purposed to allow Info and Warn level messages be
displayed on Stunner Validation dialog. That is fine.
But, imho, an UX point is not being considered, where is necessary instead.
By the DMN spec, a node should not have a prefix/trailing space, as space insensitivity
would mandate 1 trailing or 2 trailing spaces are simply ignored etc.
The engine is lenient to avoid this crashing the evaluation, but still is a bad designed
model to persist the node name (DRGElement name and its corresponding variable name) with
those extra spaces IMHO.
Further, I consider is just the Analyst which slipped a space.
In my perspective, the editor should just strip away the prefix/trailing space:
* while Importing a DMN model from file Upload
* while editing a DRGElement and hitting OK to save its name
For your consideration, thanks.
[DMN Designer] Automatically correct node prefix/trailing space on
import
-------------------------------------------------------------------------
Key: DROOLS-5062
URL:
https://issues.redhat.com/browse/DROOLS-5062
Project: Drools
Issue Type: Bug
Components: DMN Editor
Reporter: Matteo Mortari
Assignee: Michael Anstis
Priority: Major
I understand [DROOLS-5017] has been re-purposed to allow Info and Warn level messages be
displayed on Stunner Validation dialog. That is fine.
But, imho, an UX point is not being considered, where is necessary instead.
By the DMN spec, a node should not have a prefix/trailing space, as space insensitivity
would mandate 1 trailing or 2 trailing spaces are simply ignored etc.
The engine is lenient to avoid this crashing the evaluation, but still is a bad designed
model to persist the node name (DRGElement name and its corresponding variable name) with
those extra spaces IMHO.
Further, I consider is just the Analyst which slipped a space.
In my perspective, the editor should just strip away the prefix/trailing space:
* While Importing a DMN model from file Upload
* -While editing a DRGElement and hitting OK to save its name- (See
[
DROOLS-5060|https://issues.redhat.com/browse/DROOLS-5060])
For your consideration, thanks.
--
This message was sent by Atlassian Jira
(v7.13.8#713008)