Change in ovirt-engine[master]: core: Fix GWT RPC serialization issue in RepoImage

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix GWT RPC serialization issue in RepoImage ...................................................................... core: Fix GWT RPC serialization issue in RepoImage Since no UI code references RepoImage constructor, both constructor (RepoImage method) and instance initializer ($init method) are marked as dead code by GWT compiler. In consequence, instance initializer which is responsible for adding fields such as "private X field = Y;" won't be part of resulting JavaScript and corresponding GWT RPC serialization policy. In other words, if UI code doesn't reference business entity constructor and we still want to use this entity in UI code, the entity must not contain fields such as: private X field = Y; instead, the entity must contain fields such as: private X field; and do "field = Y;" assignment within its constructor. Even though constructor won't execute in UI code, all fields will still be part of resulting JavaScript. Change-Id: I7085762877afc8f76090cf18f38bd303c5db2864 Signed-off-by: Vojtech Szocs <vszocs@redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/RepoImage.java 1 file changed, 8 insertions(+), 4 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17352 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7085762877afc8f76090cf18f38bd303c5db2864 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: Alexander Wels <awels@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server
participants (1)
-
amureini@redhat.com