Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316891 - ChangeTracking weaving does not work with virtual access
Summary: ChangeTracking weaving does not work with virtual access
Status: NEW
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-15 08:47 EDT by Guy Pelletier CLA
Modified: 2022-06-09 10:05 EDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Guy Pelletier CLA 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 CLA 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 CLA 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 CLA 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 CLA 2022-06-09 10:05:19 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink