From ed7952945a6037bb67c2baa199046a4a415be28b Mon Sep 17 00:00:00 2001
From: Nicolas Micoud <58596990+nmicoud@users.noreply.github.com>
Date: Fri, 29 May 2020 14:49:55 +0200
Subject: [PATCH] =?UTF-8?q?IDEMPIERE-4313=20:=20WebServices=20:=20Yes-No?=
 =?UTF-8?q?=20output=20values=20not=20consistent=20be=E2=80=A6=20(#85)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* IDEMPIERE-4313 : WebServices : Yes-No output values not consistent between Read and Query

* Update code using Carlos's suggestion
---
 .../org/idempiere/adinterface/ModelADServiceImpl.java    | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/ModelADServiceImpl.java b/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/ModelADServiceImpl.java
index d0064b5d5c..e52aec4f1f 100644
--- a/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/ModelADServiceImpl.java
+++ b/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/ModelADServiceImpl.java
@@ -1390,8 +1390,13 @@ public class ModelADServiceImpl extends AbstractService implements ModelADServic
 					if (po.get_Value(i) != null){						
 						if(po.get_Value(i) instanceof byte[]){
 							dfid.setVal(new String(Base64.encodeBase64((byte[]) po.get_Value(i))));
-						}else						
-						    dfid.setVal(po.get_Value(i).toString());
+						}
+						else if(po.get_Value(i) instanceof Boolean) {
+							dfid.setVal((Boolean)po.get_Value(i) ? "Y" : "N");
+						}
+						else {
+							dfid.setVal(po.get_Value(i).toString());
+						}
 					}else
 						dfid.setVal(null);
 				}