On 24 Sep 2010, at 10:35, Ian Boston wrote:

> 
> On 24 Sep 2010, at 10:15, Ian Boston wrote:
>> 
>> Bug or expected behaviour ?
> 
> 
> The registered bundle content contains no uninstall-paths so IIUC, nothing 
> should have been uninstalled and yet the folders go awal
> 
> x43543-2:contenttest ieb$ curl 
> http://localhost:8080/var/sling/bundle-content.tidy.3.json
> ....
> 
>  "org.sakaiproject.nakamura.contenttest1": {
>    "content-loaded-by": "25cced8c-72a4-4c80-9611-1b63f071c0d5",
>    "content-load-time": "Fri Sep 24 2010 10:05:59 GMT+0100",
>    "jcr:mixinTypes": ["mix:lockable"],
>    "content-loaded": true,
>    "jcr:primaryType": "nt:unstructured"
>  },
>  "org.sakaiproject.nakamura.contenttest2": {
>    "content-loaded-by": "25cced8c-72a4-4c80-9611-1b63f071c0d5",
>    "content-load-time": "Fri Sep 24 2010 10:05:41 GMT+0100",
>    "jcr:mixinTypes": ["mix:lockable"],
>    "content-loaded": true,
>    "jcr:primaryType": "nt:unstructured"
>  }
> 
> Ian


DefaultContentCreator:252
-            // if node already exists but should be overwritten, delete it
-            if (!this.ignoreOverwriteFlag && this.configuration.isOverwrite() 
&& parentNode.hasNode(name)) {
+            // if the parent node already exists and should be overwritten, 
delete it but only if uninstall is set to false.
+            if (!this.ignoreOverwriteFlag && this.configuration.isOverwrite() 
&& !this.configuration.isUninstall() && parentNode.hasNode(name)) {
                parentNode.getNode(name).remove();
            }


configuration.isUninstall() does not exists at the moment.

WDYT ?

or do we need a new flag update:=true ?

Ian


> 
> 
>> Ian
> 

Reply via email to