Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0032604 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | have not tried | 2016-04-05 16:07 | 2016-06-17 19:36 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR16Q3 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 2c4101ab3ad8 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | caristu | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0032604: Stock by Warehouse widget logs error message | |||||||
Description | Whenever Stock by Warehouse widget is shown in workspace, an error message is logged in openbravo.log. The problem is caused by an incorrect definition of quantity on hand summary function. | |||||||
Steps To Reproduce | Add Stock by Warehouse widget to workspace -> check log: ERROR org.openbravo.client.querylist.QueryListDataSource - Exception while fetching the summary columns of the widget Stock by Warehouse. It is not supported using as summaries columns that are defined using a subquery, or that are defined using a summary function. Query = select sum(qtyonhand) from ProductStockView ps left outer join ps.product as Product left outer join ps.storageBin as Locator left outer join Locator.warehouse as Warehouse left outer join Product.productCategory as ProductCategory left outer join Warehouse.organization as Organization left outer join ps.uOM as UOM left outer join ps.attributeSetValue as AttributeSetInstance where ps.client.id=:client and ps.organization.id IN (:organizationList) and Product.stocked='Y' and 1=1 group by Organization.name, Warehouse.name, Organization.id, Product.searchKey, Product.name, Product.id, ProductCategory.name, ProductCategory.id, UOM.name, AttributeSetInstance.description,Warehouse.id | |||||||
Proposed Solution | Do not define summary function for quantity on hand because: * does it make any sense to sum up quantities of different products? * it has a performance penalty to do this calculation: - extra request is performed to calculate it - query can be expensive | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||||||||||||||||||
|
Notes | |
(0085447) hgbot (developer) 2016-04-05 16:30 |
Repository: erp/devel/pi Changeset: 2c4101ab3ad83e2858914e203696731c8b7198bd Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Apr 05 16:26:21 2016 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/2c4101ab3ad83e2858914e203696731c8b7198bd [^] fixed bug 32604: Stock by Warehouse widget logs error message Quantiy on hand summary was incorrectly defined. This summary has been removed because functionally it does not make sense to sum quantities of different products and it would add performance degradation. --- M modules/org.openbravo.client.widgets/src-db/database/sourcedata/OBCQL_QUERY_COLUMN.xml --- |
(0085511) caristu (developer) 2016-04-08 10:30 |
Verified |
(0087422) hudsonbot (developer) 2016-06-17 19:36 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/0dc7be081b1c [^] Maturity status: Test |
Issue History | |||
Date Modified | Username | Field | Change |
2016-04-05 16:07 | alostale | New Issue | |
2016-04-05 16:07 | alostale | Assigned To | => platform |
2016-04-05 16:07 | alostale | Modules | => Core |
2016-04-05 16:07 | alostale | Triggers an Emergency Pack | => No |
2016-04-05 16:09 | alostale | Relationship added | related to 0029174 |
2016-04-05 16:09 | alostale | Assigned To | platform => alostale |
2016-04-05 16:09 | alostale | Status | new => scheduled |
2016-04-05 16:09 | alostale | Review Assigned To | => caristu |
2016-04-05 16:21 | alostale | Proposed Solution updated | |
2016-04-05 16:30 | hgbot | Checkin | |
2016-04-05 16:30 | hgbot | Note Added: 0085447 | |
2016-04-05 16:30 | hgbot | Status | scheduled => resolved |
2016-04-05 16:30 | hgbot | Resolution | open => fixed |
2016-04-05 16:30 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/2c4101ab3ad83e2858914e203696731c8b7198bd [^] |
2016-04-08 10:30 | caristu | Note Added: 0085511 | |
2016-04-08 10:30 | caristu | Status | resolved => closed |
2016-04-08 10:30 | caristu | Fixed in Version | => 3.0PR16Q3 |
2016-06-17 19:36 | hudsonbot | Checkin | |
2016-06-17 19:36 | hudsonbot | Note Added: 0087422 | |
2016-09-22 17:01 | aferraz | Relationship added | related to 0034014 |
2017-05-17 13:30 | aferraz | Relationship added | related to 0036024 |
Copyright © 2000 - 2009 MantisBT Group |