[ mx4j-Bugs-893581 ] RelationService: possible method call param sequence error

"SourceForge.net" <[email protected]>
Newsgroups gmane.comp.java.mx4j.devel
Message-ID <[email protected]>
Bugs item #893581, was opened at 2004-02-09 18:15
Message generated for change (Comment added) made by shadow12
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450647&aid=893581&group_id=47745

Category: JMX implementation
Group: Release 2.0.0
>Status: Closed
Resolution: None
Priority: 5
Submitted By: sinclair bain (slbain)
Assigned to: Bronwen Cassidy (shadow12)
Summary: RelationService: possible method call param sequence error

Initial Comment:
Release: mx4j 2.0 Beta 1 source

Class: javax.management.relation.RelationService
Method: public void sendRoleUpdateNotification(...) 
Location: approx line #954

In this method there is a call to the 
javax.management.relation.RelationNotification 
constructor (the constructor used for updating)
i.e.
public RelationNotification(String updateType,
                                 Object source, 
                                 long sequenceNumber, 
                                 long timestamp, 
                                 String message,
                                 String relationId, 
                                 String relationTypeName,
                               ObjectName relationObjectName,
                                 String roleName, 
                                 List newRoleValues, 
                                 List oldRoleValues) throws ...

however the call switches the last two arguments to 
the constructor call 

i.e. the <newRoleValues> and the <oldRoleValues>.

So the notification new values are the old values and 
vice versa.

Looks like these need to be switched according to the 
api documentation.


sinclair bain
panacya inc.


----------------------------------------------------------------------

>Comment By: Bronwen Cassidy (shadow12)
Date: 2004-02-14 02:55

Message:
Logged In: YES 
user_id=378939

Thanks again Ben it is now fixed in cvs

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450647&aid=893581&group_id=47745


-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.