Roy Golan has uploaded a new change for review.

Change subject: REST: fix VmMapper of protection deletion
......................................................................

REST: fix VmMapper of protection deletion

fix wrong behaviour which also failed tests with NPE

Change-Id: Ibb2c1402b23d415e11d9f04cf3559f3885bca83c
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=871371
Signed-off-by: Roy Golan <rgo...@redhat.com>
---
M 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/57/9757/1

diff --git 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
index 4f82bb3..65474b4 100644
--- 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
+++ 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
@@ -163,7 +163,7 @@
         if (vm.isSetStateless()) {
             staticVm.setis_stateless(vm.isStateless());
         }
-        if (vm.isDeleteProtected()) {
+        if (vm.isSetDeleteProtected()) {
             staticVm.setDeleteProtected(vm.isDeleteProtected());
         }
         if (vm.isSetHighAvailability()) {


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

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

Reply via email to