Daniel Erez has submitted this change and it was merged.

Change subject: core: removed final from VM._diskList
......................................................................


core: removed final from VM._diskList

As the GWT serializer considers final fields as transient, no fields,
which will be sent to client should be marked with the final keyword.

In this case the _diskList of the VM entity has been marked as final,
and the consequence was that the ResourcesView's setStorageGridData()
method faild on NPE because of the getDiskList on the VM retuned null
(the UserPortal project)

The fix was only to remove the final modifier.

Change-Id: I7e09999fabf947db4ce2f3c0fb49431b820fcaad
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Daniel Erez: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7e09999fabf947db4ce2f3c0fb49431b820fcaad
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Danny Rankevich <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to