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