Change in ovirt-engine[master]: webadmin: fixing NPE in Reports (#849623)

alkaplan at redhat.com alkaplan at redhat.com
Tue Aug 21 14:10:49 UTC 2012


Alona Kaplan has submitted this change and it was merged.

Change subject: webadmin: fixing NPE in Reports (#849623)
......................................................................


webadmin: fixing NPE in Reports (#849623)

https://bugzilla.redhat.com/849623

There was a change in behavior, when fetching Reports.xml,
in the past, the jboss used to return null, and now it returns
something, so added a try/catch clause to protect the call.

Change-Id: I4e9f9e47d258b9def47a050bb7eeaafcbdd2d342
Signed-off-by: Alona Kaplan <alkaplan at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/ReportInit.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/ReportParser.java
2 files changed, 20 insertions(+), 13 deletions(-)

Approvals:
  Alona Kaplan: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4e9f9e47d258b9def47a050bb7eeaafcbdd2d342
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list