Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sprint5 server bugs #369

Merged
merged 6 commits into from
Sep 24, 2012
Merged

Sprint5 server bugs #369

merged 6 commits into from
Sep 24, 2012

Conversation

joshmoore
Copy link
Member

These are issues found during the last round of testing of the server.

  • 9632 - data in the system group loaded by root caused an exception. (integration test now passes)
  • 9635 - the canEdit flag on objects was being corrupted. Non-deterministic behavior in the web UI should now be gone.

chris-allan and others added 6 commits September 19, 2012 09:27
Data that was 1) in the system group and 2) queried via omero.group=-1
was causing an internal exception. The short-circuit block at the top of
allowLoad was failing to cache permissions for core groups.
Neither of these types will have a group so
short-circuiting the if/else will prevent a
WARN statement.
@chris-allan
Copy link
Member

All the fixes surrounding #9632 seem to be working nicely. The last two commits (2482a7f, 3cc848b) were not in the build today, right @joshmoore?

@joshmoore
Copy link
Member Author

@chris-allan, correct. Adding comment to description now.

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@joshmoore, @chris-allan: Will check difference between both clients

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@chris-allan, @joshmoore: tested in both clients with various permissions level and user's roles. No problem noticed now. Updating testing setup sheet

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@joshmoore, @chris-allan: found a problem

  • Log in as user-5
  • Add read-write group to display
  • View data of user-4
  • Cannot edit name of user-4's data. In theory I should. Screenshot added to #9635

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@joshmoore: similar problem in web

  • Read-write group logged in as user-5
  • Cannot edit data of user-4. Lock icon

Problem seems only to be for read-write, read-annotate seems fine.

@joshmoore
Copy link
Member Author

Do you mean "read-write-1"? It's currently listed as rwra:

omero@gretzky:~$ OMERO-CURRENT/bin/omero group list
Using session 64b4f491-a37a-4c91-8241-347dd3d3e94a (user-5@localhost:4064). Idle timeout: 10.0 min. Current group: read-write-1
 id  | name                                 | perms  | # of owners | # of members 
-----+--------------------------------------+--------+-------------+--------------
 0   | system                               | rw---- | 1           | 2            
 1   | user                                 | rwr-r- | 0           | 29           
 2   | guest                                | rw---- | 0           | 1            
 3   | read-write-1                         | rwra-- | 2           | 5            
 4   | read-annotate-1                      | rwra-- | 2           | 5    

@joshmoore
Copy link
Member Author

(and user-5 is only a member, not an owner of that group)

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@joshmoore: that will explain, another problem in the setup script. It should be rwrw--. Other tests/reports are potentially not valid, need to check

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@joshmoore: doc updated ome/omero-documentation#92
testing setup too.

@jburel
Copy link
Member

jburel commented Sep 20, 2012

@joshmoore: I have set correctly the permissions of read-write. Now it works as expected.

joshmoore added a commit that referenced this pull request Sep 24, 2012
@joshmoore joshmoore merged commit c82fb07 into ome:develop Sep 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants