[SLING-1627] import operation does not respect :replace=true for properties

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Servlets Post 2.1.0, Launchpad Testing 6, JCR ContentLoader 2.1.0
• Component/s: JCR
• Assignee: Eric Norman
• Reporter: Zach A. Thomas
• Created: 03/Aug/10 17:33
• Updated: 20/Sep/19 11:34
• Resolved: 06/Aug/10 22:29

Description

If I want to import some JSON and then update it with a subsequent POST (using the :replace=true flag), the nodes are updated, but not the properties.

curl -u admin:admin -F":operation=import" -F":contentType=json" -F':content={"columns":

{"one":"value"}

, "layout": "dev"}' http://localhost:8080/dashboard

If I do a subsequent POST with the :replace parameter:

curl -u admin:admin -F":operation=import" -F":contentType=json" -F':content={"columns":

{"one":"love"}

, "layout": "dev"}' -F":replace=true" http://localhost:8080/dashboard

The columns object is updated, but the layout property is not. I tracked this down to Sling's DefaultContentCreator, whose createProperty method expressly won't overwrite a property if it already exists.