BUG: omero.sys.Parameters vs Ice.Current

[atarkowska]

The latest changes that allows passing context groupId via Ice.Current to overwrite eventContext.groupId change the usage of omero.sys.Parameters().filter.groupId.

Competitions:

| omero.sys.Parameters.filter.groupId |     Ice.Current    |    groupId                |
|======================================================================================|
|              null / >=0             |       null         | EventContext.groupId      |
|              null / >=0             |  omero.group >= 0  | Ice.Current               |
|              null                   |  omero.group = -1  | cross group call          |
|              >=0                    |  omero.group = -1  | Parameters.filter.groupId |

Example of overwriting:

p = omero.sys.Parameters()
p.map = {}
f = omero.sys.Filter()
f.limit = rint(100000)
f.groupId = rlong(group_id)
f.ownerId = rlong(experimenter_id)

sql = "select i from Image i join fetch i.details.owner join fetch i.details.group"
conn.getQueryService().findAllByQuery(sql, p, {'omero.group': "-1"})

As discussed, would be nice to invoke wrapper to make sure services' methods are forced to use default value if Ice.Current is not set , like omero.group = -1. That is especially important in ISearch and ITimiline, as those services are expected to make cross group calls by default and results could be easily narrow down by Parameters then.

[jburel]

Moved from sprint 2012-06-05 (16)

[jmoore]

Ola, does everything work as intended for the moment or is OmeroWeb/OmeroPy? waiting on this wrapper? (i.e. is this a "Bug:" or a "RFE:"?)

[atarkowska]

Wrapper implemented in #9115.

tm.getEventLogsByPeriod(rtime(start), rtime(end), p, ServiceOptsDict({'omero.group': -1}))

Problem only resolved in python gateway. No server side implementation yet.

[jmoore]

Thanks, Ola. Opened an RFE #9137 to do this more globally.