<!-- 
RSS generated by JIRA (9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b) at Mon Feb 12 03:25:38 CET 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>Magnolia - Issue tracker</title>
    <link>https://jira.magnolia-cms.com</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-uk</language>    <build-info>
        <version>9.4.2</version>
        <build-number>940002</build-number>
        <build-date>19-01-2023</build-date>
    </build-info>


<item>
            <title>[MAGNOLIA-1304] Proper handling of PropertyType.NAME in MgnlNode.setProperty</title>
                <link>https://jira.magnolia-cms.com/browse/MAGNOLIA-1304</link>
                <project id="10000" key="MAGNOLIA">Magnolia</project>
                    <description>&lt;p&gt;For some unknown reason, the JcrBeanCoder uses PropertyType.NAME when storing Null values - I&apos;ve only seen this happening with the &quot;openwfe.org.jcr.beancoder.JcrBeanCoder__instance_class&quot; property. As a result, the related beans can&apos;t be deserialized.&lt;/p&gt;

&lt;p&gt;I&apos;ve applied to following fix for now, but it&apos;s far from elegant:&lt;br/&gt;
if (type == PropertyType.NAME) {&lt;br/&gt;
        log.warn(&quot;setProperty(&quot; + propertyName + &quot;, &quot; + valueStr + &quot; with type PropertyType.NAME, will switch to PropertyType.STRING ...&quot;);&lt;br/&gt;
        type = PropertyType.STRING;&lt;br/&gt;
}&lt;/p&gt;

&lt;p&gt;We should:&lt;br/&gt;
1) understand why/how PropertyType.NAME was used in the first place&lt;br/&gt;
2) if PropertyType.NAME is a valid choice, make it so that the bean can be deserialized anyhow. if not, then fix JcrBeanCoder.&lt;/p&gt;


&lt;p&gt;This was working before &lt;a href=&quot;https://jira.magnolia-cms.com/browse/MAGNOLIA-1303&quot; title=&quot;Handle different property types in MgnlNode.setProperty&quot; class=&quot;issue-link&quot; data-issue-key=&quot;MAGNOLIA-1303&quot;&gt;&lt;del&gt;MAGNOLIA-1303&lt;/del&gt;&lt;/a&gt; because the type parameter was ignored and all properties stored as Strings.&lt;/p&gt;</description>
                <environment></environment>
        <key id="12333">MAGNOLIA-1304</key>
            <summary>Proper handling of PropertyType.NAME in MgnlNode.setProperty</summary>
                <type id="3" iconUrl="https://jira.magnolia-cms.com/secure/viewavatar?size=xsmall&amp;avatarId=10898&amp;avatarType=issuetype">Task</type>
                                            <priority id="3" iconUrl="https://jira.magnolia-cms.com/images/icons/priorities/major.svg">Major</priority>
                        <status id="6" iconUrl="https://jira.magnolia-cms.com/images/icons/statuses/closed.png" description="The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.">Closed</status>
                    <statusCategory id="3" key="done" colorName="success"/>
                                    <resolution id="10000">Won&apos;t Do</resolution>
                                        <assignee username="gjoseph">Magnolia International</assignee>
                                    <reporter username="gjoseph">Magnolia International</reporter>
                        <labels>
                    </labels>
                <created>Tue, 16 Jan 2007 21:27:24 +0100</created>
                <updated>Mon, 19 Dec 2016 15:34:27 +0100</updated>
                            <resolved>Wed, 4 Nov 2015 15:42:37 +0100</resolved>
                                    <version>3.0.1</version>
                                                    <component>core</component>
                    <component>workflow</component>
                        <due></due>
                            <votes>0</votes>
                                    <watches>1</watches>
                                                                                                                <comments>
                            <comment id="13185" author="jmettraux" created="Wed, 17 Jan 2007 06:46:51 +0100"  >&lt;p&gt;1). We used NAME and not STRING to explicitely state that we are storing a null value and not &quot;null&quot; the string.&lt;br/&gt;
2). We&apos;d be glad to fix that but we need more info.&lt;/p&gt;

&lt;p&gt;Best regards,&lt;/p&gt;

&lt;p&gt;John&lt;/p&gt;</comment>
                            <comment id="13191" author="gjoseph" created="Wed, 17 Jan 2007 12:27:58 +0100"  >&lt;p&gt;John,&lt;/p&gt;

&lt;p&gt;I had specific problems with JcrBeanCoder, around line 204:&lt;br/&gt;
    protected int currentType ()&lt;br/&gt;
    // ...&lt;br/&gt;
        final String className = currentClassName();&lt;/p&gt;

&lt;p&gt;        if (className == null)&lt;br/&gt;
        //... throws exception&lt;/p&gt;

&lt;p&gt;        if (className.equals(&quot;Null&quot;))&lt;/p&gt;

&lt;p&gt;The currentClassName() method would return null - which is probably what&apos;s expected - but then the 1st if would go against that and throw an exception stating the class property is missing - while if we used the Propertype.STRING type, we jump into the 2nd if block.&lt;/p&gt;

&lt;p&gt;I can&apos;t find any clear/concise/explicit reference as to what PropertyType.NAME is supposed to reflect - can you point me to one ?&lt;/p&gt;</comment>
                            <comment id="13193" author="jmettraux" created="Wed, 17 Jan 2007 13:07:01 +0100"  >&lt;p&gt;Hi Greg,&lt;/p&gt;

&lt;p&gt;PropertyType.NAME is some JCR stuff, the document is the JCR spec.&lt;/p&gt;


&lt;p&gt;I&apos;ll put add more comments into the JcrBeanCoder around line 204 to make it clearer.&lt;/p&gt;


&lt;p&gt;An exception stack trace would help illustrate the problem.&lt;/p&gt;


&lt;p&gt;Best regards,&lt;/p&gt;

&lt;p&gt;John&lt;/p&gt;</comment>
                            <comment id="13194" author="gjoseph" created="Wed, 17 Jan 2007 13:15:47 +0100"  >&lt;p&gt;The exception I mentionned is the one thrown by &lt;br/&gt;
            throw new JcrException&lt;br/&gt;
                (&quot;property &apos;class&apos; is missing &quot;+&lt;br/&gt;
                 &quot;(&quot;&lt;ins&gt;this.currentItem.getPath()&lt;/ins&gt;&quot;)&quot;);&lt;br/&gt;
.. no copy of the stacktrace handy at the moment, sorry.&lt;/p&gt;</comment>
                            <comment id="13608" author="gjoseph" created="Fri, 23 Feb 2007 19:50:19 +0100"  >&lt;p&gt;I&apos;m tempted to postpone further work on this to 3.1, maybe along with a complete review of the bean encoding mechanism. (need better understanding..)&lt;/p&gt;</comment>
                            <comment id="13660" author="gjoseph" created="Thu, 1 Mar 2007 20:06:29 +0100"  >&lt;p&gt;postponing.&lt;/p&gt;</comment>
                            <comment id="115653" author="mmuehlebach" created="Wed, 4 Nov 2015 15:42:37 +0100"  >&lt;p&gt;Given the thousands of other issues we have open that are more highly requested, we won&apos;t be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes.&lt;br/&gt;
Thanks for taking the time to raise this issue. As you are no doubt aware this issue has been on our backlog for some time now with very little movement.&lt;br/&gt;
I&apos;m going to close this to set expectations so the issue doesn&apos;t stay open for years with few updates. If the issue is still relevant please feel free to reopen it or create a new issue.&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10010">
                    <name>relation</name>
                                            <outwardlinks description="is related to">
                                        <issuelink>
            <issuekey id="12332">MAGNOLIA-1303</issuekey>
        </issuelink>
                            </outwardlinks>
                                                                <inwardlinks description="is related to">
                                        <issuelink>
            <issuekey id="12527">MAGNOLIA-1396</issuekey>
        </issuelink>
                            </inwardlinks>
                                    </issuelinktype>
                    </issuelinks>
                <attachments>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                            <customfield id="customfield_14166" key="com.okapya.jira.checklist:checklist">
                        <customfieldname>Acceptance criteria</customfieldname>
                        <customfieldvalues>
                            
        <checklist>
        <![CDATA[
                            




                
                                    <div class="o-completion" style="display: flex; flex-shrink: 0;"><span  class="aui-lozenge aui-lozenge-complete" style="font-size: 12px; font-weight: normal; display: flex; flex-direction: row; align-items: center;" ><span style="padding-right: 4px; vertical-align: middle;"><svg width="15" height="15" viewBox="0 0 15 15" xmlns="http://www.w3.org/2000/svg" fill="white"><path clip-rule="evenodd" d="m10.41037,3.42544l-7.86501,0c-0.72395,0 -1.31084,0.58688 -1.31084,1.31084l0,7.86508c0,0.7239 0.58689,1.3108 1.31084,1.3108l7.86501,0c0.724,0 1.3109,-0.5869 1.3109,-1.3108l0,-7.86508c0,-0.72396 -0.5869,-1.31084 -1.3109,-1.31084zm-7.86501,-0.65542c-1.08593,0 -1.96626,0.88032 -1.96626,1.96626l0,7.86508c0,1.0859 0.88033,1.9662 1.96626,1.9662l7.86501,0c1.086,0 1.9663,-0.8803 1.9663,-1.9662l0,-7.86508c0,-1.08594 -0.8803,-1.96626 -1.9663,-1.96626l-7.86501,0z" fill-rule="evenodd"/><path d="m5.09049,10.18526l-1.82767,-1.82766l-0.78479,0.78479l2.61246,2.61246l5.38758,-5.38754l-0.78483,-0.78479l-4.60275,4.60274z"/></svg></span><span>Empty</span></span></div>
                        ]]>
    </checklist>


                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                <customfield id="customfield_10111" key="com.atlassian.jira.toolkit:reporterdomain">
                        <customfieldname>Company</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>magnolia-cms.com</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                <customfield id="customfield_10031" key="com.atlassian.jira.ext.charting:firstresponsedate">
                        <customfieldname>Date of First Response</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>Wed, 17 Jan 2007 06:46:51 +0100</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_12730" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_14151" key="com.atlassian.jira.toolkit:message">
                        <customfieldname>Docu info</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            <customfield id="customfield_10061" key="com.atlassian.jira.toolkit:lastusercommented">
                        <customfieldname>Last comm is not jira-dev</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>true</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10071" key="com.atlassian.jira.toolkit:lastupdaterorcommenter">
                        <customfieldname>Last participant</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>mmuehlebach</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_13136" key="com.atlassian.jira.toolkit:LastCommentDate">
                        <customfieldname>Last public comment date</customfieldname>
                        <customfieldvalues>
                            8 years, 15 weeks, 4 days ago
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                            <customfield id="customfield_10020" key="com.atlassian.jira.toolkit:attachments">
                        <customfieldname>Number of attachments</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0.0</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10150" key="com.atlassian.jira.toolkit:comments">
                        <customfieldname>Number of comments</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>7.0</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        <customfield id="customfield_10011" key="com.atlassian.jira.toolkit:participants">
                        <customfieldname>Participants</customfieldname>
                        <customfieldvalues>
                                        <customfieldvalue>jmettraux</customfieldvalue>
            <customfieldvalue>gjoseph</customfieldvalue>
            <customfieldvalue>mmuehlebach</customfieldvalue>
    
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                <customfield id="customfield_10833" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0|i017pr:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10244" key="com.pyxis.greenhopper.jira:gh-global-rank">
                        <customfieldname>Rank (Obsolete)</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>7132</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                <customfield id="customfield_14167" key="com.okapya.jira.checklist:checklist">
                        <customfieldname>Task DoR</customfieldname>
                        <customfieldvalues>
                            
        <checklist>
        <![CDATA[
                            




                
                                    <div class="o-completion" style="display: flex; flex-shrink: 0;"><span  class="aui-lozenge aui-lozenge-complete" style="font-size: 12px; font-weight: normal; display: flex; flex-direction: row; align-items: center;" ><span style="padding-right: 4px; vertical-align: middle;"><svg width="15" height="15" viewBox="0 0 15 15" xmlns="http://www.w3.org/2000/svg" fill="white"><path clip-rule="evenodd" d="m10.41037,3.42544l-7.86501,0c-0.72395,0 -1.31084,0.58688 -1.31084,1.31084l0,7.86508c0,0.7239 0.58689,1.3108 1.31084,1.3108l7.86501,0c0.724,0 1.3109,-0.5869 1.3109,-1.3108l0,-7.86508c0,-0.72396 -0.5869,-1.31084 -1.3109,-1.31084zm-7.86501,-0.65542c-1.08593,0 -1.96626,0.88032 -1.96626,1.96626l0,7.86508c0,1.0859 0.88033,1.9662 1.96626,1.9662l7.86501,0c1.086,0 1.9663,-0.8803 1.9663,-1.9662l0,-7.86508c0,-1.08594 -0.8803,-1.96626 -1.9663,-1.96626l-7.86501,0z" fill-rule="evenodd"/><path d="m5.09049,10.18526l-1.82767,-1.82766l-0.78479,0.78479l2.61246,2.61246l5.38758,-5.38754l-0.78483,-0.78479l-4.60275,4.60274z"/></svg></span><span>Empty</span></span></div>
                        ]]>
    </checklist>


                        </customfieldvalues>
                    </customfield>
                                                                                                                                                    <customfield id="customfield_14145" key="com.intenso.jira.issue-templates:issue-templates-customfield">
                        <customfieldname>Template</customfieldname>
                        <customfieldvalues>
                            


                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                            <customfield id="customfield_15131" key="com.onresolve.jira.groovy.groovyrunner:scripted-field">
                        <customfieldname>Time in Discovery</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10032" key="com.atlassian.jira.ext.charting:timeinstatus">
                        <customfieldname>Time in Status</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        </customfields>
    </item>
</channel>
</rss>