Bug 300588 - Validate that an Interface param's readonly attribute is inherited by Module implementers
Summary: Validate that an Interface param's readonly attribute is inherited by Module ...
Status: RESOLVED WORKSFORME
Alias: None
Product: RTSC (ARCHIVED)
Classification: Technology
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Dave Russo CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2010-01-22 20:29 EST by Chris Ring CLA
Modified: 2011-11-25 20:15 EST (History)
1 user (show)

See Also:


Attachments
packages that illustrate readonly behavior (20.00 KB, application/octet-stream)
2010-01-22 21:10 EST, Dave Russo CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Ring CLA 2010-01-22 20:29:00 EST
When an Interface declares a config param to be 'readonly', that property should be inherited by Modules that implement that interface.

Then, a Module implementer need not explicitly restate this 'readonly' attribute if they override the default setting.  As a result, if the Interface restrictions are lifted (e.g., the Interface removes the 'readonly' attribute making the field writeable), Module producers would not be required to rebuild against the new interface.

Some initial quick testing (with the Interface and Module in the same package) seem to indicate this is supported.  So this enhancement is perhaps simply a request for a test to validate the use case is supported.
Comment 1 Dave Russo CLA 2010-01-22 21:08:34 EST
The current implementation allows the module writer to override the config parameter without the readonly attribute _but_ the readonly property is "lost".

However, this is arguably correct: the inheriting module is declaring that it will allow both gets and sets whereas the underlying interface simply requires gets.

The attached tar contains a set of package that allows experimentation with this behavior.
Comment 2 Dave Russo CLA 2010-01-22 21:10:45 EST
Created attachment 157026 [details]
packages that illustrate readonly behavior
Comment 3 Dave Russo CLA 2011-11-25 20:15:00 EST
nothing to be done (I think)