Context Navigation

← Previous Change
Ticket History
Next Change →

Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.

Changes between Initial Version and Version 2 of Ticket #8955

Timestamp:: 05/25/12 15:42:38 (12 years ago)
Author:: atarkowska
Comment:

Legend:

: Unmodified
: Added
: Removed
: Modified

Ticket #8955
- Property Priority changed from minor to critical
- Property Component changed from General to Services
- Property Summary changed from BUG: omero.sys.Parameters vs _ctx to BUG: omero.sys.Parameters vs Ice.Current

Ticket #8955 – Description

-                      initial
+                      v2
+With the latest idea of passing current context via omero.group omero.sys.Parameters().filter.groupId should be redundant as mislead and is not taken to the account any more.
+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.