[jboss-jira] [JBoss JIRA] (DROOLS-5060) [DMN Designer] Editor allows node with prefix/trailing space

[Matteo Mortari (Jira)]

    [ https://issues.redhat.com/browse/DROOLS-5060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13970781#comment-13970781 ] 

Matteo Mortari commented on DROOLS-5060:
----------------------------------------

no veto against having 2 separate jiras, as long as both UX of naming a node and importing a DMN model with that "defect" will be handled. Thanks1

> [DMN Designer] Editor allows node with prefix/trailing space
> ------------------------------------------------------------
>
>                 Key: DROOLS-5060
>                 URL: https://issues.redhat.com/browse/DROOLS-5060
>             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
> For your consideration, thanks.



--
This message was sent by Atlassian Jira
(v7.13.8#713008)