[ http://jira.magnolia.info/browse/MAGNOLIA-625?page=all ]
     
Philipp Bracher resolved MAGNOLIA-625:
--------------------------------------

    Fix Version: 2.2
     Resolution: Fixed

fixed
- use StringUtils.isEmpty()
- save the paragraph type only if not yet set

> Saving without nodeCollection fails
> -----------------------------------
>
>          Key: MAGNOLIA-625
>          URL: http://jira.magnolia.info/browse/MAGNOLIA-625
>      Project: magnolia wcm
>         Type: Bug

>   Components: core
>     Versions: 2.2
>  Environment: Win XP
>     Reporter: Ralf Hirning
>     Assignee: Philipp Bracher
>      Fix For: 2.2

>
> Original Estimate: 30 minutes
>         Remaining: 30 minutes
>
> In info.magnolia.cms.gui.control.Save.getSaveNode(HierarchyManager hm, 
> Content rootNode) is a check for the nodeCollectionName. The 
> nodeCollectionName is a String which is initialized as StringUtils.EMPTY, but 
> it is checked against null. If you want to save something directly in a page 
> a org.apache.jackrabbit.name.MalformedPathException is thrown.
> Could you please change the code snippet in getSaveNode(HierarchyManager hm, 
> Content rootNode)
>         // get or create nodeCollection
>         Content nodeCollection = null;
>         if (this.getNodeCollectionName() != null) {
> to something like
>         // get or create nodeCollection
>         Content nodeCollection = null;
>         if (this.getNodeCollectionName() != null && ! 
> this.getNodeCollectionName().equals(StringUtils.EMPTY)) {
> thanks
> Ralf

-- 
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
----------------------------------------------------------------

Reply via email to