> Hello,
>
> Currently Slide sets following properties into read-only mode (enforced by
> the PropPatchMethod.checkPropertyModification method):
>
> 1) Creationdate
> 2) Getcontentlength
> 3) Getetag
> 4) Getlastmodified
> 5) Lockdiscovery
>
> All other properties can be modified or deleted by a PropPatch command:
>
> 1) displayname
> 2) getcontentlanguage
> 3) getcontenttype
> 4) resourcetype
> 5) source
> 6) supportedlock
>
> In addition all these properties have the protectedProperty set to true in
> NodeProperty, independent of the read-only fact.
>
> The standard is pretty vague, what a read-only (can not be modified or
> deleted) property is, but the set of read-only properties is a subset of
> live properties.
>
> I want to suggest to add at least supportedlock to the set of read-only
> properties (more?) and enforce the read-only fact, by the
> NodeProperty.protectedProperty flag.
That looks very reasonable.
> The check code in PropPatch would look like:
>
>
> private boolean checkPropertyModification(Property property,
> NodeRevisionDescriptor rd) {
> boolean result = !rd.getProperty(property.name,
> property.namespace).isProtected();
> if (!result) property.status = WebdavStatus.SC_CONFLICT;
> return result;
>
> }
+1.
Remy
