Opened 15 years ago

Last modified 15 years ago

#1401 closed defect

One-to-one associations should not be synchronized in the data layer classes — at Version 3

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: minor Milestone: BASE 2.14
Component: core Version:
Keywords: Cc:

Description (last modified by Nicklas Nordborg)

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 (3)

comment:1 by Nicklas Nordborg, 15 years ago

Owner: changed from everyone to Nicklas Nordborg
Status: newassigned

comment:2 by Nicklas Nordborg, 15 years ago

There is an exect copy of the same problem in HybridizationData.setArraySlide().

comment:3 by Nicklas Nordborg, 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
Note: See TracTickets for help on using tickets.