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.
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 #6893 (closed)

Opened 13 years ago

Closed 12 years ago

changeActiveGroup vs. setGroupForSession

Reported by: atarkowska Owned by:
Priority: critical Milestone: n.a.
Component: OmeroPy Version: n.a.
Keywords: n.a. Cc: python-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

There is only local implementation of changing active group in webclient_gateway.py. There should be one single method on the omero.gateway to do it.
In stead of pushing changeActiveGroup we could possibly use setGroupForSession. But current implementation does not work as expected. setGroupForSession updates omero session, but BlitzGateway? fields. It cause mismatch between those two objects, while you call BlitzGateway?.getExentContext(), getUser(), etc. again.

Change History (2)

comment:1 Changed 13 years ago by cneves

Ola, please add a unit test that describes what you expect from that method and pass the ticket to me.

comment:2 Changed 12 years ago by atarkowska

  • Resolution set to duplicate
  • Status changed from new to closed

duplicated in 2339

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

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.66065 sec.)

We're Hiring!