related to bug 23706, related to bug 23705
authorAsier Lostalé <asier.lostale@openbravo.com>
Wed, 01 May 2013 12:53:04 +0200
changeset 20252 eb2fcc35a947
parent 20251 13bebe76e732
child 20253 1ec1a7c8ac33
child 20259 b67b46460127
related to bug 23706, related to bug 23705

Added comment not to forget to review change for issue 23705 when 23706 is solved
modules/org.openbravo.service.json/src/org/openbravo/service/json/DefaultJsonDataService.java
--- a/modules/org.openbravo.service.json/src/org/openbravo/service/json/DefaultJsonDataService.java	Wed May 01 12:51:36 2013 +0200
+++ b/modules/org.openbravo.service.json/src/org/openbravo/service/json/DefaultJsonDataService.java	Wed May 01 12:53:04 2013 +0200
@@ -169,6 +169,10 @@
           jsonResponse.put(JsonConstants.RESPONSE_TOTALROWS, 1);
           return jsonResult.toString();
         } else if (parameters.containsKey(JsonConstants.DISTINCT_PARAMETER)) {
+          // TODO: BaseOBObjects created by this query are not valid, see issue #23705, when this is
+          // fixed, IdentifierProvider should be revisited to remove code handling this
+          // incorrectness
+
           // when distinct an array of values is returned
           // the first value is the BaseObObject the other values
           // are part of the order by and such and can be ignored