Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0030410 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | major | always | 2015-07-22 18:17 | 2015-10-26 10:51 | |||
Reporter | migueldejuana | View Status | public | |||||
Assigned To | Sandrahuguet | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | RR16Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 16c1cef39f5c | ||||
Projection | none | ETA | none | Target Version | RR16Q1 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | marvintm | |||||||
OBNetwork customer | OBPS | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0030410: [Stock Criteria] Products out of the limit will not be filtered by stock | |||||||
Description | Stock criteria flow: - Search by filters entered by user and limit defined in product model (100 by default). - Filter by stock in the result of product of previous filter. First query has a limit and will not show products that fulfill the filters, so we will not be able to apply stock filter to some products. | |||||||
Steps To Reproduce | - Search products without filter and choose a product outside the shown list. - Update the stock of that product to 123456. - Search again and add stock filter equals '123456'. The choosen product will not appear in the list. | |||||||
Proposed Solution | Include stockfilter using addWhereFilter function. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|||||||||||||||
|
![]() |
|
(0080757) adrianromero (viewer) 2015-09-30 16:44 edited on: 2015-09-30 16:48 |
The problem is in function *postProcess: function (collection, callback)* of the filter component in module org.openbravo.retail.stockcriteria. This function is invoked after all the SQL filters of the criteria selected by the cashier. This *postProcess* function receives as a parameter all the products that returns the SQL filters and calculates the stock for each product and returns the products filtered by the stock criteria. This stock calculation is performed in the server side by the process org.openbravo.retail.stockcriteria.Stockcriteria that performs a query for each product and returns only the products that fulfills the stock criteria. The problem is that if the result of the execution of the SQL filters has been cropped because the SQL limi, there can be products that fulfills the whole criteria selected by the cashier but are never sent to the Stockcriteria process. A fix of this problem will probably require a refactor of the base criteria functionality and also a rewrite of the stock criteria filter functionality. |
(0080859) Orekaria (viewer) 2015-10-07 13:47 |
missing resolution time and OBNetwork customer mark |
(0080950) hgbot (developer) 2015-10-13 10:38 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: ef886e7af50cfd9ae5c68bf497d4546647fc0991 Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Thu Oct 08 16:07:55 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/ef886e7af50cfd9ae5c68bf497d4546647fc0991 [^] related to issue 30410 ability to force remote in normal mode --- M web/org.openbravo.mobile.core/source/data/ob-dal.js M web/org.openbravo.mobile.core/source/retail/component/ob-retail-searchproductcharacteristic.js --- |
(0081017) Sandrahuguet (viewer) 2015-10-14 16:18 |
Repository: erp/pmods/org.openbravo.retail.stockcriteria Changeset: 585f5d75379d4e1f4308ec370b9cb4abb1a776e1 Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Wed Oct 14 11:25:50 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.stockcriteria/rev/585f5d75379d4e1f4308ec370b9cb4abb1a776e1 [^] [^] Fixed bug 30401 Products out of the limit will not be filtered by stock --- M web/org.openbravo.retail.stockcriteria/js/hookStockCriteria.js A src/org/openbravo/retail/stockcriteria/ProductProperties.java A src/org/openbravo/retail/stockcriteria/StockChHQLCriteria.java A src/org/openbravo/retail/stockcriteria/StockChValueHQLCriteria.java A src/org/openbravo/retail/stockcriteria/StockHQLCriteria.java R src/org/openbravo/retail/stockcriteria/MGetStock.java R src/org/openbravo/retail/stockcriteria/StockCondition.java R src/org/openbravo/retail/stockcriteria/StockFilters.java R src/org/openbravo/retail/stockcriteria/StockInfo.java R src/org/openbravo/retail/stockcriteria/Stockcriteria.java |
(0081031) hgbot (developer) 2015-10-15 09:45 |
Repository: tools/automation/pi-mobile Changeset: c02c2bb550c2401ef7a2b9ea13b380162eaded6c Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Thu Oct 15 09:44:02 2015 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/c02c2bb550c2401ef7a2b9ea13b380162eaded6c [^] Related to issue 30410 Added automatic test --- A src-test/org/openbravo/test/mobile/retail/extmodules/selenium/tests/stockcriteria/Issue30410_CheckStockCriteria.java --- |
(0081086) hgbot (developer) 2015-10-19 13:32 |
Repository: erp/pmods/org.openbravo.retail.stockcriteria Changeset: 6b0d96d6a2899cbf23a50e2ec2d28206b2e73383 Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Mon Oct 19 13:31:36 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.stockcriteria/rev/6b0d96d6a2899cbf23a50e2ec2d28206b2e73383 [^] related to issue 30410 fixed filter by more than one criteria --- M src/org/openbravo/retail/stockcriteria/StockChHQLCriteria.java M src/org/openbravo/retail/stockcriteria/StockChValueHQLCriteria.java M web/org.openbravo.retail.stockcriteria/js/hookStockCriteria.js --- |
(0081135) adrianromero (viewer) 2015-10-20 13:49 |
Products with stock "null" appear when filtering by stock. For example, create a new product without any movement transaction Go to web pos and in the search panel add a filter by stock: For example with stock greater than 0 and less than 10 Check that this product appears with label stock: null |
(0081169) hgbot (developer) 2015-10-21 11:30 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 16c1cef39f5c6af8117b7fc070ea5df3bedbbd58 Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Wed Oct 21 11:29:23 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/16c1cef39f5c6af8117b7fc070ea5df3bedbbd58 [^] fixed issue 30410 added forceremote deleted by mistake --- M web/org.openbravo.mobile.core/source/retail/component/ob-retail-searchproductcharacteristic.js --- |
![]() |
|||
Date Modified | Username | Field | Change |
2015-07-22 18:17 | migueldejuana | New Issue | |
2015-07-22 18:17 | migueldejuana | Assigned To | => Retail |
2015-07-22 18:17 | migueldejuana | OBNetwork customer | => No |
2015-07-22 18:17 | migueldejuana | Triggers an Emergency Pack | => No |
2015-09-29 15:21 | Orekaria | Target Version | RR15Q4 => RR16Q1 |
2015-09-30 16:44 | adrianromero | Note Added: 0080757 | |
2015-09-30 16:48 | adrianromero | Note Edited: 0080757 | View Revisions |
2015-10-07 11:01 | avicente | Issue Monitored: avicente | |
2015-10-07 12:12 | agete | Issue Monitored: agete | |
2015-10-07 13:47 | Orekaria | Note Added: 0080859 | |
2015-10-07 13:47 | Orekaria | Status | new => feedback |
2015-10-07 16:32 | networkb | OBNetwork customer | No => Yes |
2015-10-07 16:32 | networkb | Resolution time | => 1445292000 |
2015-10-07 16:32 | networkb | Status | feedback => new |
2015-10-07 16:32 | Orekaria | Status | new => acknowledged |
2015-10-07 17:18 | Sandrahuguet | Assigned To | Retail => Sandrahuguet |
2015-10-07 17:18 | Sandrahuguet | Status | acknowledged => scheduled |
2015-10-13 10:38 | hgbot | Checkin | |
2015-10-13 10:38 | hgbot | Note Added: 0080950 | |
2015-10-14 16:18 | Sandrahuguet | Note Added: 0081017 | |
2015-10-14 16:18 | Sandrahuguet | Status | scheduled => resolved |
2015-10-14 16:18 | Sandrahuguet | Resolution | open => fixed |
2015-10-14 16:19 | Sandrahuguet | Fixed in SCM revision | => https://code.openbravo.com/erp/pmods/org.openbravo.retail.stockcriteria/rev/585f5d75379d4e1f4308ec370b9cb4abb1a776e1 [^] |
2015-10-15 09:45 | hgbot | Checkin | |
2015-10-15 09:45 | hgbot | Note Added: 0081031 | |
2015-10-15 09:45 | Sandrahuguet | Review Assigned To | => marvintm |
2015-10-19 13:32 | hgbot | Checkin | |
2015-10-19 13:32 | hgbot | Note Added: 0081086 | |
2015-10-20 13:49 | adrianromero | Note Added: 0081135 | |
2015-10-20 13:49 | adrianromero | Status | resolved => new |
2015-10-20 13:49 | adrianromero | Resolution | fixed => open |
2015-10-20 14:27 | Orekaria | Status | new => acknowledged |
2015-10-20 15:16 | Orekaria | Status | acknowledged => scheduled |
2015-10-21 11:30 | hgbot | Checkin | |
2015-10-21 11:30 | hgbot | Note Added: 0081169 | |
2015-10-21 11:30 | hgbot | Status | scheduled => resolved |
2015-10-21 11:30 | hgbot | Resolution | open => fixed |
2015-10-21 11:30 | hgbot | Fixed in SCM revision | https://code.openbravo.com/erp/pmods/org.openbravo.retail.stockcriteria/rev/585f5d75379d4e1f4308ec370b9cb4abb1a776e1 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/16c1cef39f5c6af8117b7fc070ea5df3bedbbd58 [^] |
2015-10-26 10:51 | marvintm | Status | resolved => closed |
2015-10-26 10:51 | marvintm | Fixed in Version | => RR16Q1 |
2016-05-18 16:51 | Sandrahuguet | Relationship added | related to 0032834 |
2016-05-18 16:59 | Sandrahuguet | Relationship added | blocks 0032866 |
Copyright © 2000 - 2009 MantisBT Group |