Opened 12 months ago

Closed 12 months ago

#2303 closed enhancement (fixed)

The Annotation import should only request ANNOTATE permission instead of WRITE permission

Reported by: Nicklas Nordborg Owned by: everyone
Priority: minor Milestone: BASE 3.19.9
Component: coreplugins Version:
Keywords: Cc:

Description

I noticed (in #2302) that the Annotation importer request WRITE permission on file items. It should be enough with ANNOTATE permission.

Change History (1)

comment:1 by Nicklas Nordborg, 12 months ago

Resolution: → fixed
Status: new → closed

In 8146:

Fixes #2303: The Annotation importer should only request ANNOTATE permission instead of WRITE permission

Actually, there is no ANNOTATE permission, but the RESTRICTED_WRITE permission is used for this. Needed a look at #2034 to refresh my mind.

The Annotation importer now requests the RESTRICTED_WRITE permission. The Item.File also needed to be updated so that the GUI enables this option (the permisssion is called Annotated in the GUI).

The Annotation batcher implementation also needed an upate since it checked for WRITE permission (as noted in #2033 but never corrected).

Note: See TracTickets for help on using tickets.