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 <tjeli...@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 <tjeli...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com> Gerrit-Reviewer: Martin Betak <mbe...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
