Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036427Retail ModulesWeb POSpublic2017-06-05 11:302017-08-02 12:35
Reporteregoitz 
Assigned Toebecerra 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q2.2Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerOBPS
Support ticket47012
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036427: Perforamnce problems loading the master data for discounts.
DescriptionOn environment with a lot of product characteristics, the load of the
DiscountFiltercharactaritic model takes long.
Steps To Reproduce-Login on webpos on an environment with a big amount of characteristics
*The query of the DiscountFiltercharactaritic model takes long.
Proposed SolutionON modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/master/DiscountFilterCharacteristic.java
replace
hql += " left join c.characteristic.productCharacteristicValueList cvl ";
with
hql += " left join c.characteristic.characteristicValueList cvl ";

and
hql += " and m_isparent_ch_value(cvl.characteristicValue.id, c.chValue.id, c.characteristic.id) != -1 ";
with
 hql += " and m_isparent_ch_value(cvl.id, c.chValue.id, c.characteristic.id) != -1 ";


Also edit the file
modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/master/DiscountFilterCharacteristicProperties.java
by replacing
add(new HQLProperty("distinct(coalesce(c.id,cvl.characteristicValue.id))", "id"));
with
add(new HQLProperty("distinct(coalesce(c.id,cvl.id))", "id"));

and
add(new HQLProperty("cvl.characteristicValue.id", "chValue"));
with
add(new HQLProperty("cvl.id", "chValue"));
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0036175 closed ebecerra Perforamnce problems loading the master data for discounts. 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-07-06 08:47marvintmTypedefect => backport
2017-07-06 08:47marvintmTarget Version => RR17Q2.2
2017-08-01 12:12hgbotCheckin
2017-08-01 12:12hgbotNote Added: 0098305
2017-08-01 12:12hgbotStatusscheduled => resolved
2017-08-01 12:12hgbotResolutionopen => fixed
2017-08-01 12:12hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/31e4ae9b1728a4afcccb7c391392124513dcdb2f [^] => http://code.openbravo.com/retail/backports/3.0RR17Q2.2/org.openbravo.retail.posterminal/rev/8a65af42c7e85a48ed83b7d12ef19369de3b6078 [^]
2017-08-02 12:35migueldejuanaReview Assigned To => migueldejuana
2017-08-02 12:35migueldejuanaNote Added: 0098331
2017-08-02 12:35migueldejuanaStatusresolved => closed

Notes
(0098305)
hgbot   
2017-08-01 12:12   
Repository: retail/backports/3.0RR17Q2.2/org.openbravo.retail.posterminal
Changeset: 8a65af42c7e85a48ed83b7d12ef19369de3b6078
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Tue Aug 01 12:12:00 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR17Q2.2/org.openbravo.retail.posterminal/rev/8a65af42c7e85a48ed83b7d12ef19369de3b6078 [^]

Fixed issue 36427: Optimize loading the master data for discounts

---
M src/org/openbravo/retail/posterminal/master/DiscountFilterCharacteristic.java
M src/org/openbravo/retail/posterminal/master/DiscountFilterCharacteristicProperties.java
---
(0098331)
migueldejuana   
2017-08-02 12:35   
Reviewed