Bug 1313851

Summary:	Unckecking override checkbox in smart class parameters effects other checkboxes
Product:	Red Hat Satellite	Reporter:	Tomer Brisker <tbrisker>
Component:	Provisioning	Assignee:	orabin
Status:	CLOSED ERRATA	QA Contact:	Jitendra Yejare <jyejare>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.0.4	CC:	bbuckingham, cwelton, jyejare
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/13976
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-27 09:03:56 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Tomer Brisker 2016-03-02 12:54:36 UTC

When unchecking the override checkbox the matchers and validators are saved so when it's checked again they are all there.
All the checkboxes on the form (use puppet default, hidden value, required) are unchecked (because they are disabled and don't send their value).
This should be consistent - we either save the state of the parameter or we remove everything.

Comment 1 Tomer Brisker 2016-03-02 12:54:39 UTC

Created from redmine issue http://projects.theforeman.org/issues/13976

Comment 2 Tomer Brisker 2016-03-02 12:54:43 UTC

Upstream bug assigned to orabin

Comment 3 orabin 2016-03-02 13:01:48 UTC

upstream bug is merged

Comment 4 Bryan Kearney 2016-03-02 13:11:43 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/13976 has been closed
-------------
Ori Rabin
Applied in changeset commit:dba5afe0f39748ea8dafad88a7da046a6aab5c9a.

Comment 7 Jitendra Yejare 2016-04-15 11:06:26 UTC

Verified this in Sat 6.2 snap 8.1

I see that on unchecking the override parameter, all other entities(checkboxes, values) state is saved.

I am agree with this update and its one of the solution suggested in Description of bug.

So moving this to Verifed.

Comment 9 errata-xmlrpc 2016-07-27 09:03:56 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:1500