Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0052280Retail ModulesWeb POSpublic2023-04-27 13:252023-04-27 15:48
Reportercaristu 
Assigned Tocaristu 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR23Q3 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0052280: Remote queries with boolean properties are not built properly
DescriptionRemote queries with boolean properties are not built properly because and unneeded upper is added to the column linked to the boolean property:

and upper(businesspa0_.IsCustomer)=$5

The

and businesspa0_.IsCustomer=$5 order
Steps To Reproduce.
Proposed SolutionGenerate the query without that unneeded upper, i.e., instead of:

and upper(businesspa0_.IsCustomer)=$5

the filter should be:

and businesspa0_.IsCustomer=$5
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0052151 closed caristu Masterdata Remote Mode - Phase 2 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2023-04-27 13:25caristuNew Issue
2023-04-27 13:25caristuAssigned To => caristu
2023-04-27 13:25caristuOBNetwork customer => No
2023-04-27 13:25caristuTriggers an Emergency Pack => No
2023-04-27 13:25caristuIssue generated from0052151
2023-04-27 13:25caristuRelationship addedrelated to 0052151
2023-04-27 13:25caristuTypefeature request => defect
2023-04-27 13:31hgbotMerge Request Status => open
2023-04-27 13:31hgbotNote Added: 0148993
2023-04-27 15:47hgbotMerge Request Statusopen => approved
2023-04-27 15:47hgbotNote Added: 0149023
2023-04-27 15:48hgbotResolutionopen => fixed
2023-04-27 15:48hgbotStatusnew => closed
2023-04-27 15:48hgbotFixed in Version => RR23Q3
2023-04-27 15:48hgbotNote Added: 0149024

Notes
(0148993)
hgbot   
2023-04-27 13:31   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/504 [^]
(0149023)
hgbot   
2023-04-27 15:47   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/504 [^]
(0149024)
hgbot   
2023-04-27 15:48   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: d965d72d905093fc10990de5c1d785a0452b4d70
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 27-04-2023 13:29:08
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/d965d72d905093fc10990de5c1d785a0452b4d70 [^]

fixes BUG-52280: build remote filter for boolean properties correctly

  Now the CriteriaConverter passes a boolean property already supported
by the SimpleQueryBuilder that makes the filter by boolean not add an
unneeded upper transformation of the column.

---
M src-test/org/openbravo/mobile/core/master/TestBusinessPartner.java
M src-test/org/openbravo/mobile/core/process/InnerCritera.java
M src-test/org/openbravo/mobile/core/process/RemoteMasterDataTest.java
M web-test/model/remote/RemoteDataController.test.js
M web/org.openbravo.mobile.core/app/model/remote/CriteriaConverter.js
---