ome / omero-blitz

Gradle project containing Ice remoting code for OMERO
https://www.openmicroscopy.org/omero
GNU General Public License v2.0
0 stars 15 forks source link

Fix import-as by group owners. #94

Closed mtbc closed 4 years ago

mtbc commented 4 years ago

This PR corrects the OMERO 5.4.6 regression that fails to permit import for group members by group owners who are not administrators.

Fixes https://github.com/ome/omero-server/issues/63 and can be tested with https://github.com/ome/openmicroscopy/pull/6221.

The adjustment to persistLogFile's return value could be considered a minor breaking change.

pwalczysko commented 4 years ago

Tested with release 5.5.11 insight connecting to merge-ci server. All works as expected, a group owner can import into P,RO and RA groups on behalf of others successfully.

pwalczysko commented 4 years ago

Also tested a 5.6.1 release CLI against the merge-ci. Went fine too

imagesc-bot commented 4 years ago

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/release-of-omero-server-5-6-2-omero-web-5-7-0-and-omero-iviewer-0-10-0/40463/1