Bug 537191 - Id field of Configuration not serialized when using reflective serialization
Summary: Id field of Configuration not serialized when using reflective serialization
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core Server
Version: 1.4
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
: ---
Assignee: Simeon Pinder
QA Contact: Corey Welton
URL:
Whiteboard:
Depends On:
Blocks: JON231
TreeView+ depends on / blocked
 
Reported: 2009-11-12 18:37 UTC by John Sanda
Modified: 2010-08-12 16:50 UTC (History)
1 user (show)

Fixed In Version: 2.4
Clone Of:
Environment:
Last Closed: 2010-08-12 16:50:04 UTC
Embargoed:


Attachments (Terms of Use)

Description John Sanda 2009-11-12 18:37:31 UTC
Description of problem:
The Configuration class delegates to EntitySerializer to handle the reflective serialization strategy. Configuration.id does not specify the @Column annotation, only the @Id annotation. EntitySerializer does not look for the @Id annotation, and therefore, Configuration.id is not properly serialized with the reflective strategy which I believe is used with the CLI.

I have already modified EntitySerializer to look for the @Id annotation in addition to the other entity annotations it looks for. This fixes Configuration.id as well as any others that declare @Id without also declaring @Column.

The hash for the commit is 68534e1f50d3c054e2dbddfd8e622818db45139b and has been pushed to the raw-config branch.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Simeon Pinder 2010-01-18 16:08:36 UTC
Reproduce instructions:

1: Login to CLI as rhqadmin and locate resource id for the root platform(Ex. Linux Platform ... 10001)
2: var platform = ProxyFactory.getResource(10001);
3: Exectue platform.viewProcessList()
* Note the id field of Configuration is always [0] and not serialized before fix.

Error:
Invoking operation viewProcessList
Configuration [0] - null
  processList [249] {

Correct:
Invoking operation viewProcessList
Configuration [12077] - null
  processList [249] {


Fix:
Svn CP commit revision 5275

Comment 2 Corey Welton 2010-01-22 23:58:56 UTC
QA Verified, the Process list is correctly returned.

Comment 3 Corey Welton 2010-08-12 16:50:04 UTC
Mass-closure of verified bugs against JON.


Note You need to log in before you can comment on or make changes to this bug.