Bug 754419 - Configuration objects required to specifically contain required properties even though those are required to have default values
Summary: Configuration objects required to specifically contain required properties ev...
Keywords:
Status: NEW
Alias: None
Product: RHQ Project
Classification: Other
Component: Core Server
Version: 4.3
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Nobody
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 734866
TreeView+ depends on / blocked
 
Reported: 2011-11-16 12:49 UTC by Lukas Krejci
Modified: 2022-03-31 04:28 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description Lukas Krejci 2011-11-16 12:49:53 UTC
Description of problem:
Throughout our codebase, we have this basic assumption - if a configuration property is required, we assume it exists in the supplied configuration object. 

Now from a usability point of view, there should be no need to supply those unless their values differ from the default value specified in the configuration definition. 

This is of course not a problem when accessing RHQ through GUI, because GUI takes care of that, but there is a number of remote API methods that either directly or indirectly use a Configuration class in their parameters. A remote user shouldn't have to be required to slavishly repeat the default values on required properties.

The "impact surface" of this bug is quite large - basically the whole remote API needs to be checked if the arguments of all methods don't specify Configuration (or objects having configuration as one of their fields, recursively).

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


How reproducible:
always

Steps to Reproduce:
See bug https://bugzilla.redhat.com/show_bug.cgi?id=734866
  
Actual results:
NPEs and other unexpected effects

Expected results:
our impls should be clever enough to replace the missing properties with their default values

Additional info:

Comment 1 Lukas Krejci 2011-11-16 13:17:12 UTC
Just a note that this is sort of an umbrella bug for the individual issues yet to be discovered in the individual remote APIs. The bug 734866 mentioned in the repro steps is merely the first BZ raised that falls under this umbrella.


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