Bug #12393
closed
Overriding the parameter default value doent adds 1 to the 'Number of Values' count.
Added by Jitendra Yejare about 9 years ago.
Updated over 6 years ago.
Description
Description of problem:
Overriding the parameter default value doent adds 1 to the 'Number of Values' count in parameters table in Config -> Smart Class Parameter page.
Version-Release number of selected component (if applicable):
How reproducible:
Always
Steps to Reproduce:
1. Override smart class parameter, set the default value.
2. Don't check 'puppet default value' checkbox.
3. See the 'Number Of Values' column for that parameter in Config -> Smart Class Parameters.
Actual results:
The count doesn't adds 1 to the 'Number of Values' for that parameter.
Expected results:
The count for 'Number of Values' should be increased by 1.
- Status changed from New to Feedback
- Translation missing: en.field_release deleted (
71)
Please don't set the release field.
Setting the default value is not creating an override, that's the section at the bottom of the form with specific matchers. This appears to be correct behaviour to me.
Dominic Cleal wrote:
Please don't set the release field.
Setting the default value is not creating an override, that's the section at the bottom of the form with specific matchers. This appears to be correct behaviour to me.
But the column says the 'number of values' the parameter has and by giving default value which is atleast one value. And so I believe this needs to be fixed.
It will also help to identify that What are the parameters whose only default value has been overrided from table itself.
- Status changed from Feedback to New
- Assignee deleted (
Tomer Brisker)
- Priority changed from Normal to Low
It'd be better to call it the number of overrides since it's a counter, and if you need some way to indicate the presence of a default value, to indicate it another way.
- Status changed from New to Ready For Testing
- Assignee set to Tomer Brisker
- Pull request https://github.com/theforeman/foreman/pull/3179 added
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
- Translation missing: en.field_release set to 71
Also available in: Atom
PDF