Ticket #315 (closed enhancement: fixed)

Opened 2 years ago

Last modified 5 months ago

Administrative modifications to annotations.

Reported by: amit Assigned to: jsuttor
Priority: high Milestone:
Component: ambra Version: 0.8
Keywords: annotations Cc:
Blocking: Blocked By:

Description

As part of the problem associated with changing of the underlying XML, annotations positions could potentially be wrong. We thought a quick way to fix this is to allow admins to edit the Xpointer of the annotations to reposition them correctly. However Steve found out that: 

There may be a slight problem with this. I'm doing a new annotation
that supercedes an existing annotation.  Thing is, the user is derived
from the logged-in CAS user.  I don't have a way to impersonate the
original author of the annotation, it will always be from the admin
user.

So what exactly should the semantics be? Admin operation on behalf of user? Do we rewrite the annotation without regard to revision history?

Dependency Graph

Change History

(in reply to: ↑ description ; follow-up: ↓ 3 ) 03/23/07 17:21:26 changed by pradeep

Replying to amit:

As part of the problem associated with changing of the underlying XML, annotations positions could potentially be wrong. We thought a quick way to fix this is to allow admins to edit the Xpointer of the annotations to reposition them correctly.

How is this done? via a script or manually? If you are using the annotaion-ws, yes since we don't have an update semantics other than supersede you would run into the issue where all annotations are now authored by admin. So may be bypassing and fixing the xpointer in mulgara directly via a script is a better option.

03/26/07 10:44:46 changed by stevec

The plan was to do it manually since it is not clear how many spaces things will shift after removing the formatting. Given the relatively small number of annotations, it seemed like it would be easier to have it done manually. It would be difficult for a script (actually hard for a person as well) to know what the originally intended boundaries were.

(in reply to: ↑ 1 ) 03/26/07 11:44:41 changed by amit

Replying to pradeep:

Replying to amit:

As part of the problem associated with changing of the underlying XML, annotations positions could potentially be wrong. We thought a quick way to fix this is to allow admins to edit the Xpointer of the annotations to reposition them correctly.

How is this done? via a script or manually? If you are using the annotaion-ws, yes since we don't have an update semantics other than supersede you would run into the issue where all annotations are now authored by admin. So may be bypassing and fixing the xpointer in mulgara directly via a script is a better option.

Agreed, but we need a long term fix for this too. Problems like this will keep arising and we need to set a precedent with PLoS ONE so that new application developers have a model to think about.

(follow-up: ↓ 5 ) 03/27/07 07:43:41 changed by amit

  • owner changed from somebody to pradeep.

(in reply to: ↑ 4 ) 03/27/07 11:03:56 changed by pradeep

Replying to amit: When is this needed? If this needs to be done via the current annotation-ws, may be adding an updateAnnotation() method would solve it. Otherwise with the OTM we can take care of it.

04/18/07 09:33:23 changed by amit

  • owner changed from pradeep to stevec.
  • priority changed from high to medium.

I guess we will wait to migrate to new architecture before tacking this one. It will be easier for PLoS ONE to provide this feature. Transferring back to Steve and reducing priority.

05/29/07 17:43:05 changed by amit

  • owner changed from stevec to jsuttor.
  • version set to 0.8.
  • milestone changed from TBD to 0.8.

06/19/07 17:20:05 changed by amit

  • priority changed from medium to high.

This needs to be thought about when we merge the annotation administration for all annotations types.

06/20/07 12:41:14 changed by jsuttor

  • keywords set to annotations.
  • status changed from new to assigned.
  • component changed from topaz to plos-one.

08/03/07 20:06:55 changed by jsuttor

  • status changed from assigned to closed.
  • resolution set to fixed.

see r3321. interface is very basic, you get to edit the raw Annotation.context, e.g. the xpointer. must be admin.

07/16/08 11:00:34 changed by

  • milestone deleted.

Milestone 0.8 deleted