[ http://jira.magnolia.info/browse/MAGNOLIA-1304?page=comments#action_13193 ] John Mettraux commented on MAGNOLIA-1304: -----------------------------------------
Hi Greg, PropertyType.NAME is some JCR stuff, the document is the JCR spec. I'll put add more comments into the JcrBeanCoder around line 204 to make it clearer. An exception stack trace would help illustrate the problem. Best regards, John > Proper handling of PropertyType.NAME in MgnlNode.setProperty > ------------------------------------------------------------ > > Key: MAGNOLIA-1304 > URL: http://jira.magnolia.info/browse/MAGNOLIA-1304 > Project: Magnolia > Issue Type: Task > Components: core, workflow > Affects Versions: 3.0.2 > Reporter: Grégory Joseph > Assigned To: Sameer Charles > Fix For: 3.0.2 > > > 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.info/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ---------------------------------------------------------------- for list details see http://www.magnolia.info/en/magnolia/developer.html ----------------------------------------------------------------
