Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 316891

Summary:	ChangeTracking weaving does not work with virtual access
Product:	z_Archived	Reporter:	Guy Pelletier <guy.pelletier>
Component:	Eclipselink	Assignee:	Project Inbox <eclipselink.orm-inbox>
Status:	NEW ---	QA Contact:
Severity:	normal
Priority:	P3	CC:	douglas.clarke, eencircled, gordon.yorke, tom.ware
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Guy Pelletier

2010-06-15 08:47:44 EDT

We should not weave for change tracking when any one mapping acccessor on a class uses VIRTUAL access (has a VirtualAttributeAccessor)

Current testing model (using mock virtual classes) used in the extended jpa test suite:

org.eclipse.persistence.testing.models.jpa.xml.advanced.Shovel*

Explicit @ChangeTracking(DEFERRED) added to Shovel to ensure the testMockDynamicClassCRUD test from org.eclipse.persistence.testing.tests.jpa.xml.advanced.EntityMappingsAdvancedJUnitTestCase requires this to ensure the test passes (i.e. correctly detects an update)

Comment 1 Gordon Yorke

2010-06-15 08:51:52 EDT

We should still weave these classes.  We have enough information to weave correctly we just need to add the support.

Comment 2 Doug Clarke

2010-06-18 15:17:49 EDT

My hope is that VIRTUAL access becomes an additional means by which users can build extensible models. Currently they can add a element-collection to store additional values in a secondary table but adding VIRTUAL attributes with additional columns added to the table also is of interest to some.

I'll build up an example and attach but I believe 2.2 is a reasonable plan for this assuming the bug gets some votes.

Comment 3 Vlad Ks

2016-04-19 09:39:45 EDT

Hello,
any progress here?
By the way, this bug can be fixed just by calling "_persistence_propertyChange" method with corresponding params.

Thanks
Vlad

Comment 4 Eclipse Webmaster

2022-06-09 10:05:19 EDT

The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink