[ 
http://jira.magnolia-cms.com/browse/MAGNOLIA-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jan Haderka updated MAGNOLIA-1304:
----------------------------------

    Fix Version/s: 4.3.x
                       (was: 4.3.2)

> Proper handling of PropertyType.NAME in MgnlNode.setProperty
> ------------------------------------------------------------
>
>                 Key: MAGNOLIA-1304
>                 URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-1304
>             Project: Magnolia
>          Issue Type: Task
>          Components: core, workflow
>    Affects Versions: 3.0.1
>            Reporter: Grégory Joseph
>            Assignee: Grégory Joseph
>             Fix For: 4.3.x
>
>
> For some unknown reason, the JcrBeanCoder uses PropertyType.NAME when storing 
> Null values - I've only seen this happening with the 
> "openwfe.org.jcr.beancoder.JcrBeanCoder__instance_class" property. As a 
> result, the related beans can't be deserialized.
> I've applied to following fix for now, but it's far from elegant:
> if (type == PropertyType.NAME) {
>         log.warn("setProperty(" + propertyName + ", " + valueStr + " with 
> type PropertyType.NAME, will switch to PropertyType.STRING ...");
>         type = PropertyType.STRING;
> }
> We should:
> 1) understand why/how PropertyType.NAME was used in the first place
> 2) if PropertyType.NAME is a valid choice, make it so that the bean can be 
> deserialized anyhow. if not, then fix JcrBeanCoder.
> This was working before MAGNOLIA-1303 because the type parameter was ignored 
> and all properties stored as Strings.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira



----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com>
----------------------------------------------------------------

Reply via email to