jsedding commented on code in PR #9:
URL: 
https://github.com/apache/sling-org-apache-sling-installer-provider-jcr/pull/9#discussion_r1608319420


##########
src/main/java/org/apache/sling/installer/provider/jcr/impl/JcrInstaller.java:
##########
@@ -707,6 +716,7 @@ private UpdateResult handleUpdate(final String resourceType,
             dataNode.setProperty(PROP_MODIFIED, Calendar.getInstance());
             dataNode.setProperty(PROP_ENC, ENCODING);
             dataNode.setProperty(PROP_MIME, MIME_TXT);
+            dataNode.setProperty(ORIGINAL_PID, id);

Review Comment:
   Does this work, seeting a property on the `jcr:content` node? I think the 
`nt:resource` node type is quite restrictive. IIRC Oak uses a different 
node-type for a file's `jcr:content` node. Maybe that one is more lenient. 
Alternatively, a mixin nodetype could be added to allow for the extra property.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@sling.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to