Change in ovirt-engine[master]: userportal: NPE in OsTypeImage

Tomas Jelinek has submitted this change and it was merged. Change subject: userportal: NPE in OsTypeImage ...................................................................... userportal: NPE in OsTypeImage The OsTypeImage creates the name of the default OS type like: "otherOs" + nameUniquePart + IMAGE, where the IMAGE is "Image", the nameUniquePart is "Large" or "Small". Than takes this name and looks up the specific image from the ApplicationResourcesWithLookup. The probelm was, that the otherOsSmallImage was written with capital "O", which caused periodic NPE thrown on FE. Fixed by expecting the "OtherOs..." on OsTypeImage and adjusted also other names in ApplicationResourcesWithLookup. Change-Id: I2690cda0142f5c7849b3ad4bcc251906658244f1 Signed-off-by: Tomas Jelinek <tjelinek@redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/ApplicationResourcesWithLookup.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/widget/basic/OsTypeImage.java 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10209 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2690cda0142f5c7849b3ad4bcc251906658244f1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com>
participants (1)
-
tjelinek@redhat.com