Task #4901 (closed)
Opened 13 years ago
Closed 13 years ago
BlitzObjectWrapper.__getattr__()
Reported by: | wmoore | Owned by: | wmoore |
---|---|---|---|
Priority: | critical | Milestone: | OMERO-Beta4.3 |
Component: | OmeroPy | Version: | n.a. |
Keywords: | n.a. | Cc: | python-team@… |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | 2011-05-19 (12) |
Description
Today's discussion http://www.openmicroscopy.org.uk/site/community/minutes/minigroup/api-unification-meetings/2011.04.07 decided that Blitz Object Wrapper methods (on the underlying ._obj) should only return wrapped objects.
Should not get omero.model objects or omero.rtypes, E.g.
imageWrapper.getPrimaryPixels() # get omero.model.PixelsI imageWrapper.format # get omero.model.FormatI imageWrapper.getFormat() # get omero.model.FormatI imageWrapper.acquisitionDate # get Long - OK imageWrapper.getAcquisitionDate() # get omero.rtypes.RTimeI - should get Long imageWrapper.getInstrument() # get InstrumentWrapper - OK imageWrapper.instrument # get omero.model.InstrumentI - should get InstrumentWrapper
Change History (6)
comment:1 Changed 13 years ago by Will Moore <will@…>
comment:2 Changed 13 years ago by wmoore
- Owner changed from cneves-x to wmoore
comment:3 Changed 13 years ago by wmoore
- Cc python-team@… added
- Remaining Time set to 1
- Sprint set to 2011-05-19 (12)
comment:4 Changed 13 years ago by wmoore
- Status changed from new to accepted
comment:5 Changed 13 years ago by Will Moore <will@…>
(In [d5f6d5506c72543d24c02898cc342005320e3dc1/ome.git] on branch develop) BlitzObjectWrapper? 'get' methods will wrap omero.model.ObjectI. See #4901
comment:6 Changed 13 years ago by Will Moore <will@…>
- Remaining Time changed from 1 to 0
- Resolution set to fixed
- Status changed from accepted to closed
(In [fa427409529be4a359889bc74d0394e7d83ae57d/ome.git] on branch develop) gatewaytest for 'get' methods wrapping omero.model.ObjectI. Closes #4901
(In [58ba1cabddabebb12cab15c2709433fd8c2ed1a5/ome.git] on branch develop) Adding testAllObjectsWrapped() to gatewaytest. See #4901