Context Navigation

Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #8990 (closed)

Opened 7 years ago

Closed 7 years ago

Bug: userId not taken into account in RoiOptions

Reported by:	sbesson	Owned by:	jamoore
Priority:	major	Milestone:	OMERO-4.4
Component:	General	Version:	n.a.
Keywords:	n.a.	Cc:	jburel, wmoore, jamoore
Resources:	n.a.	Referenced By:	n.a.
References:	n.a.	Remaining Time:	0.0d
Sprint:	n.a.

Description

Specifying userId in omero.api.options does not filter rois by owner.

import omero
from omero.gateway import BlitzGateway
import omero.model
import omero.api

client = omero.client("localhost")
client.createSession("test","omero")
conn = BlitzGateway(client_obj=client)
images = list(conn.getObjects("Image",[506]))

roiOptions = omero.api.RoiOptions()
roiOptions.userId = omero.rtypes.rlong(conn._userid)
r = conn.getRoiService().findByImage(images[0].getPixelsId(), roiOptions)
print r.rois[0].getId().getValue()

roiOptions.userId = omero.rtypes.rlong(0)
r =conn.getRoiService().findByImage(images[0].getPixelsId(), roiOptions)
print r.rois[0].getId().getValue()

r =conn.getRoiService().findByImage(images[0].getPixelsId(), None)
print r.rois[0].getId().getValue()

References

Change History (3)

comment:1 Changed 7 years ago by jmoore

Owner set to jmoore

comment:2 Changed 7 years ago by jmoore

Remaining Time set to 0.25
Status changed from new to accepted

comment:3 Changed 7 years ago by jmoore

Remaining Time changed from 0.25 to 0
Resolution set to fixed
Status changed from accepted to closed

Fix pushed for findByImage (with and without a namspace), findByRoi, findByPlane.

*   5fc7514 - (HEAD, will/sprint17-bugfixes, sprint17-bugfixes) Merge branch '8990-roioptions' into sprint17-bugfixes (57 seconds ago) <jmoore>
|\  
| * cce8bd3 - (8990-roioptions) Update 8890 test for other RoiOptions-based methods (See #8890) (3 minutes ago) <jmoore>
| * 1dbe644 - Apply RoiOptions to other methods (See #8990) (3 minutes ago) <jmoore>
| * 8ece4ff - (josh/8990-roioptions) Apply RoiOptions filter to findByImage (Fix #8990) (3 hours ago) <jmoore>
| * aa583ce - Convert Sebastien's example to a test (See #8990) (3 hours ago) <jmoore>

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

Download in other formats: