[node-patches] Change in ovirt-node[master]: Rework the inputs on the RHN page a little

fabiand at redhat.com fabiand at redhat.com
Thu Aug 14 11:19:35 UTC 2014


Fabian Deutsch has submitted this change and it was merged.

Change subject: Rework the inputs on the RHN page a little
......................................................................


Rework the inputs on the RHN page a little

Don't blindly set the text on input fields. Instead, we should
disable input fields and keep their value, passing it back in
if it's re-enabled. Additionally, this prevents getting into a
situation where bad input in rhn.url or rhn.ca can be marked as
valid by changing the type from Satellite to SAM, as well as
improperly marking the page as invalid if invalid input has been
entered into fields which are no longer active.

Change-Id: I977ac638914db789bc367a91cdbfda3851d2e4eb
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1064836
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/setup/rhn/rhn_page.py
1 file changed, 13 insertions(+), 7 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/31473
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I977ac638914db789bc367a91cdbfda3851d2e4eb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list