Opened 15 years ago
Closed 15 years ago
#1401 closed defect (fixed)
One-to-one associations should not be synchronized in the data layer classes
| Reported by: | Nicklas Nordborg | Owned by: | Nicklas Nordborg |
|---|---|---|---|
| Priority: | minor | Milestone: | BASE 2.14 |
| Component: | core | Version: | |
| Keywords: | Cc: |
Description (last modified by )
See source:/tags/2.13/src/core/net/sf/basedb/core/data/MeasuredBioMaterialData.java@HEAD#L163
The line calling bioWell.setBioMaterial() is made to make sure the one-to-one link is synchonized. The problem is that it effectively disables lazy loading of bio-wells since it initializes the proxy. The synchronization should be made at a higher level, eg. MeasuredBioMaterial.setBioWell().
This is slightly related to #1374, since a lot of annotations/experimental factors are probably associated with biomaterial items, and there is no need to initialize a lot of other things in this case.
Change History (4)
comment:1 by , 15 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 15 years ago
comment:3 by , 15 years ago
| Description: | modified (diff) |
|---|---|
| Summary: | MeasuredBioMaterialData.setBioWell() should not call BioWellData.setBioMaterial() → One-to-one associations should not be synchronized in the data layer classes |
comment:4 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
There is an exect copy of the same problem in
HybridizationData.setArraySlide().