[jboss-jira] [JBoss JIRA] (WFCORE-2670) list-add doesn't work for nested list of child attribute

[Tomaz Cerar (JIRA)]

    [ https://issues.jboss.org/browse/WFCORE-2670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13393560#comment-13393560 ] 

Tomaz Cerar commented on WFCORE-2670:
-------------------------------------

Error handling should be more forgiving here.
It should first check for result as it does now, but if it doesn't find it, it should be more pragmatic and check if expression as such is valid, which can be done by checking if attribute that is not present in result is even present in attribute description.

Looking at the code, it looks like this kind of regression as we have now was introduced when we moved to always all :read-resource before writing back changed attribute. and :read-attribute has different mechanics, where error reporting like this kind a makes sense, but when doing write-attribute you don't care if "middle" parts of expression are not yet defined in the model, they should just be added.

> list-add doesn't work for nested list of child attribute
> --------------------------------------------------------
>
>                 Key: WFCORE-2670
>                 URL: https://issues.jboss.org/browse/WFCORE-2670
>             Project: WildFly Core
>          Issue Type: Bug
>          Components: Domain Management
>            Reporter: Claudio Miranda
>            Assignee: Brian Stansberry
>            Priority: Critical
>
> The list-add doesn't work for a child of an attribute as in the example below
> Add a dir-context first
> {code}
> /profile=full/subsystem=elytron/dir-context=dir2:add(url="ldap://test")
> {code}
> Add a ldap-realm
> {code}
> /profile=full/subsystem=elytron/ldap-realm=foobar2:add(dir-context=dir2,identity-mapping={rdn-identifier=test})
> {
>     "outcome" => "success",
>     "result" => undefined,
>     "server-groups" => {"main-server-group" => {"host" => {"master" => {"server-one" => {"response" => {
>         "outcome" => "success",
>         "response-headers" => {"process-state" => "reload-required"}
>     }}}}}}
> }
> {code}
> This is the command to add the item to the list
> {code}
> /profile=full/subsystem=elytron/ldap-realm=foobar2:list-add(name=identity-mapping.new-identity-attributes,value={name=key2,value=["val1","val2"]})
> {
>     "outcome" => "failed",
>     "result" => undefined,
>     "server-groups" => undefined,
>     "rolled-back" => true
> }
> {code}



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)