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 1 of Ticket #12781, comment 5


Ignore:
Timestamp:
03/19/15 09:11:17 (9 years ago)
Author:
sbesson
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #12781, comment 5

    initial v1  
    11Agreed, https://github.com/openmicroscopy/openmicroscopy/blob/v5.1.0-m4/components/blitz/src/ome/formats/OMEROMetadataStoreClient.java#L1958 contains the logic to switch group in the store when looking for the target across groups. However, looking at the import log logging the store used server-side uses the initial context. 
    22 
    3 Interestingly a related bug is that the following also launches a stack trace. 
     3A someway related bug is that the following call (non-existing dataset) also fails with a non-intuitive stack trace: 
    44 
    55{{{ 
     
    77}}} 
    88 
    9 My proposal here is to unify the CLI behavior with the one of Insight, i.e. force the correct context to be used in the initial command. That would likely mean trying to retrieve the dataset/screen at the CLI import level and returning a meaningful error message early. 
     9My proposal here is to unify the CLI behavior with the one of Insight and fix both bugs (target in a different group/non-existing) by forcing the correct context to be used in the initial command. That will likely mean trying to retrieve the dataset/screen at the CLI import level and returning a meaningful error message. 

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

We're Hiring!