Re-add the PojoOptions.isGroup() logic.

[jamoore]

Somewhere between Omero2 and Omero3 we removed all the logic to filter results by group. That needs to be fixed now.

[jmoore]

r908 provides group functionality. findContainerHierarchies() is still failing. See #322.

[jmoore]

r909 fixes the findContainerHierarchies issue. It now seems that filtering is working. Though it should be noted that the enabled filters for each query (especially PojosFindHierarchiesQueryDefinition) need to be reviewed. (Affects performance)

Jean-Marie, can you give it a whirl?

[jburel]

The case when we don't specify the rootID is not handled properly.
follow error message

ome.conditions.ApiUsageException?: Set of ids for loadContainerHierarchy() may not be null if experimenter and group options are null.

at ome.logic.PojosImpl?.loadContainerHierarchy(PojosImpl?.java:113)
at ome.ro.ejb.PojosBean?.loadContainerHierarchy(PojosBean?.java:164)

[jmoore]

r911 adds a test for this. It's passing, however. Jean-Marie, can you update it to make it fail?

[jburel]

Just updated:
now error when trying to load the current user hierarchies.
Follow the error message:
ome.conditions.ApiUsageException?: Set of ids for loadContainerHierarchy() may not be null if experimenter and group options are null.

at ome.logic.PojosImpl?.loadContainerHierarchy(PojosImpl?.java:113)
at ome.ro.ejb.PojosBean?.loadContainerHierarchy(PojosBean?.java:164)

[jmoore]

Jean-Marie, your server will need to be updated. ome.ro.ejb.PojosBean doesn't exist anymore. I should have noticed that last time. Sorry.

[jburel]

server updated
rebuilt jars
but still same error message

[jburel]

oops sorry forgot to remove my comments ;-(

[jmoore]

Ok. I'm assuming this is working now. If there are particular issues, please make a new ticket.