Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Null check before model cleanup ...................................................................... webadmin: Null check before model cleanup Patch gerrit.ovirt.org/63708 introduced improved model cleanup yet in cases where the model is already null due to a missing null check a UI exception will be thrown (for instance when closing a dialog with ESC) Bug-Url: https://bugzilla.redhat.com/1379312 Bug-Url: https://bugzilla.redhat.com/1389998 Change-Id: I89b94e175352a25019bfcabf1af70ea9a615af83 Signed-off-by: Tal Nisan <tnisan(a)redhat.com&gt; Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com&gt; --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractModelBoundPopupPresenterWidget.java 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Greg Sheremeta: Verified -- To view, visit https://gerrit.ovirt.org/65917 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89b94e175352a25019bfcabf1af70ea9a615af83 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com&gt; Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com&gt; Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;