Change in ovirt-engine[master]: webadmin: NPE while login if there are some running tasks

[tjelinek at redhat.com]

Tomas Jelinek has submitted this change and it was merged.

Change subject: webadmin: NPE while login if there are some running tasks
......................................................................


webadmin: NPE while login if there are some running tasks

The new SimpleActionTable() in the AlertsEventsFooterView's constructor
triggers the TaskModelProvider.updateDataProvider() which in case there are at
least one started job, triggers the
taskHandler.onRunningTasksCountChange(count). This in turn triggers the call to
AlertsEventsFooterView.setAlertCount() which expects the buttons to be already
inited. But this fields are inited at the end of the constructor which causes
NPE.

Fixed by calling the updateButtonResources() which inits this buttons sooner.

This bug happens in development mode only.

Change-Id: I3315802e33d0bc21ae1810802d1466464e8321b4
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/footer/AlertsEventsFooterView.java
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Tomas Jelinek: Verified
  Alexander Wels: Looks good to me, approved
  Greg Sheremeta: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3315802e33d0bc21ae1810802d1466464e8321b4
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server