Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No events for new role players #316

Closed
GoogleCodeExporter opened this issue Mar 16, 2015 · 3 comments
Closed

No events for new role players #316

GoogleCodeExporter opened this issue Mar 16, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

The interface TopicMapListenerIF provides no separate callbacks for added or 
modified associations. I therefore expect that whenever a topic t is added as a 
role player to an association, then the callback objectModified(snapshot) will 
be called with snapshot.getObjectId() == t.getObjectId().

However, the current implementation calls the callback only for the players 
that were already attached to the association.

I will attach a test case that demonstrates the issue. The XTM file needs to be 
placed in test-data/various/alumni.xtm.

The failure of testRolePlayerEvent1() shows that there is no event at all when 
the first role player is attached to the association.

The failure of testRolePlayerEvent2() shows the same for the second role 
player, but it also shows that there is an event for the first role player when 
the second player is attached. (Because only the second assert fails.)

Original issue reported on code.google.com by lud...@fh-worms.de on 17 Oct 2010 at 5:02

Attachments:

@GoogleCodeExporter
Copy link
Author

Excellent issue report, and extra bonus points for including a test case.

Original comment by lar...@gmail.com on 18 Oct 2010 at 10:04

  • Changed state: Accepted
  • Added labels: Component-Engine

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision r1471.

Original comment by indiapaleale@gmail.com on 5 Nov 2010 at 1:54

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Revision r1472 clean verifies that the fix works with the rdbms backend.

Original comment by indiapaleale@gmail.com on 5 Nov 2010 at 2:09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant