[Engine-patches] Change in ovirt-engine[master]: core: fix conflicting getter definitions

[ahadas]

Arik Hadas has uploaded a new change for review.

Change subject: core: fix conflicting getter definitions
......................................................................

core: fix conflicting getter definitions

For several boolean fields in RunVmParams we have both "isX" & "getX"
getters, thus json produce: IllegalArgumentException: Conflicting
getter definitions for property.

The solution is to mark the "getX" getter with JsonIgnore.

Change-Id: I37b5e286c21814e757e6369ef9b2a642ac0c621d
Signed-off-by: Arik Hadas <aha...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RunVmParams.java
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/63/28263/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RunVmParams.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RunVmParams.java
index e9aa955..fe0afc1 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RunVmParams.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RunVmParams.java
@@ -1,5 +1,6 @@
 package org.ovirt.engine.core.common.action;
 
+import org.codehaus.jackson.annotate.JsonIgnore;
 import org.ovirt.engine.core.common.businessentities.BootSequence;
 import org.ovirt.engine.core.common.businessentities.DbUser;
 import org.ovirt.engine.core.common.businessentities.InitializationType;
@@ -193,6 +194,7 @@
     }
 
     // for REST-API test compat
+    @JsonIgnore
     public Boolean getBootMenuEnabled() {
         return bootMenuEnabled;
     }
@@ -205,6 +207,7 @@
         this.bootMenuEnabled = bootMenuEnabled;
     }
 
+    @JsonIgnore
     public Boolean getSpiceFileTransferEnabled() { return 
spiceFileTransferEnabled; }
 
     public Boolean isSpiceFileTransferEnabled() { return 
spiceFileTransferEnabled; }
@@ -212,6 +215,7 @@
     public void setSpiceFileTransferEnabled(Boolean spiceFileTransferEnabled) 
{ this.spiceFileTransferEnabled = spiceFileTransferEnabled; }
 
     // for REST-API test compat
+    @JsonIgnore
     public Boolean getSpiceCopyPasteEnabled() { return spiceCopyPasteEnabled; }
 
     public Boolean isSpiceCopyPasteEnabled() { return spiceCopyPasteEnabled; }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I37b5e286c21814e757e6369ef9b2a642ac0c621d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches