Eric
Thanks for fixing this properly, and so quickly, my patch was just a
hack to get out of a hole by comparison.
I've had a look at your commit and I can't see any problems in this
area.
Thanks
Ian
Sent from my iPhone
On 3 Jul 2010, at 00:33, "Eric Norman (JIRA)" <[email protected]> wrote:
[ https://issues.apache.org/jira/browse/SLING-1583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12884857#action_12884857
]
Eric Norman commented on SLING-1583:
------------------------------------
Sorry about that. I just resolved the remaining part of SLING-1172
which now removes the need for that "name" property in the JSON.
The unit tests pass locally, but please review to ensure your use
case still functions properly.
Recent changes to content loader swallow "name" property in nodes,
which was previously used only for Princiapls, breaks existing code.
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---
---------------------------------------------------------------------
Key: SLING-1583
URL: https://issues.apache.org/jira/browse/SLING-1583
Project: Sling
Issue Type: Bug
Components: JCR
Affects Versions: JCR ContentLoader 2.0.8
Reporter: Ian Boston
Assignee: Ian Boston
Fix For: JCR ContentLoader 2.0.8
Name should be changed to something that wont be used in a node
property, eg :name and added to the ignore list for node properties.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
