DOC: Delete annotation with delete object

[jburel]

Deleting annotation using the deleteObject method from {{{IUpdate}} fails.

[jmoore]

Jean-Marie: do you remember about what was happening here? Was the annotation attached to something else and therefore the link kept it alive? If so, I suggest we close this and just deprecate IUpdate.deleteObject (which we want to do anyway).

Note: pushing this passing test to my branch:

+class TestTickets5000(lib.ITest):
+
+    def test4341(self):
+        """
+        Delete annotation with IUpdate
+        """
+        tag = omero.model.TagAnnotationI()
+        tag = self.update.saveAndReturnObject(tag)
+        self.update.deleteObject(tag)
+

[jburel]

I will review the tests I had.

[jburel]

deleteObject can be useful if we want to just break links. The ones we need to deprecate first are IDelete.deleteImage, IDelete.deletePlate

[jburel]

The problem was related to the deletion of file annotation and original file. The test is in UpdateServiceTest.

@Test(enabled = false, groups = {"ticket:2705"})
public void testDeleteAnnotation() 

[jburel]

see #7774

[jburel]

The delete method from {{{IUpdate}} "fails" when trying to delete a file annotation. The file annotation is deleted but the original file is not.
The original file should be deleted.
See deleteAnnotation in components/tools/OmeroJava/UpdateServiceTest

Passing back to Josh for consideration, if we mark the method has deprecated that's fine.

[jmoore]

That's certainly the expected behavior for IUpdate.deleteObject: only remove a single row from the database. I'm not sure we want to get into modifying the logic of IUpdate, and would prefer to just deprecate it. Should I do that as part of this ticket, or hold off for a larger round of deprecation?

[jburel]

I will probably hold off. If it is not deprecated we need to indicate not to use that it i.e. a documentation ticket

[jmoore]

In discussion with Jean-Marie, adding to IUpdate.deleteObject documentation the limitations of this method (one row at a time) and pointing them to the new delete service.

[jmoore]

Pushing minor documentation fix to sprint6-server-backlog. We still need to merge the javadocs from ome.api with those from omero.api (#2101).

[jmoore <josh@…>]

(In [a7c9f6ee9623b949b3f72a039a016dc47b985803/ome.git] on branch develop) Passing IUpdate delete test (See #4341)

[jmoore <josh@…>]

(In [a4905400b00d468fb9d177d48e2e9af116aaf15f/ome.git] on branch develop) IUpdate.deleteObject doc update (Fix #4341)