Tomas Jelinek has submitted this change and it was merged.
Change subject: userportal: Edit VM on UP fails on NPE
......................................................................
userportal: Edit VM on UP fails on NPE
The regression has been introduced by the SerialNumber feature:
72fcf57dc92d4e5469b1911cb5b3b42e433bd784
which adds the possibility to edit the serial number also to the UP but the
query which is supposed to return, if the serial number is supported for the
specific version is not a user query leading to a NPE in AbstractVmPopupWidget
where the isSerialNumberPolicySupported was being converted to boolean. And
since it was null, this conversion failed on NPE.
Change-Id: Ifdd13ded70e3b4abb17c114ebbe86b11a3ef31ea
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Tomas Jelinek: Verified
Martin Betak: Looks good to me, but someone else must approve
Gilad Chaplik: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/26524
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifdd13ded70e3b4abb17c114ebbe86b11a3ef31ea
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server