Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0042399 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | major | always | 2019-11-26 11:43 | 2019-12-09 11:23 | |||
Reporter | egoitz | View Status | public | |||||
Assigned To | caristu | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR20Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | f45e0aa5902d | ||||
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 | alostale | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0042399: Toolbar reports are using the readonly pool if configured | |||||||
Description | Based on the documentation [1] the reports printed from the standard toolbar button should use the standard pool and not the readonly pool but it is now working properly as same queries are executed in the readonly database. [1] http://wiki.openbravo.com/wiki/Read-Only_Pool [^] And starting from 3.0PR18Q2 release, the documents printed with the standard print toolbar button, are never using the read-only pool (even if configured) but the standard pool. See here for additional informatio | |||||||
Steps To Reproduce | 1- Configure Openbravo to use a readonly database 2- Configure postgresql on the readonly database to print all the queries (log_min_duration_statement = 0) 3- Go to purchase invoice window 4- Print the invoice using the toolbar button 5- Check the log on the readonly database. The query is printed on the log. | |||||||
Proposed Solution | The class Report.java [1] is using the readonly pool on the constructor: getReadOnlyConnectionProvider public Report(DocumentType documentType, String documentId, String strLanguage, String templateId, boolean multiReport, OutputTypeEnum outputTypeString) throws ReportingException, ServletException { this(DalConnectionProvider.getReadOnlyConnectionProvider(), documentType, documentId, strLanguage, templateId, multiReport, outputTypeString); } [1] https://code.openbravo.com/erp/devel/pi/file/tip/src/org/openbravo/erpCommon/utility/reporting/Report.java [^] | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Notes | |
(0116143) hgbot (developer) 2019-12-09 10:24 |
Repository: erp/devel/pi Changeset: f45e0aa5902dbb37011ac716555e74830c09f0a5 Author: Carlos Aristu <carlos.aristu <at> openbravo.com> Date: Tue Nov 26 18:58:41 2019 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/f45e0aa5902dbb37011ac716555e74830c09f0a5 [^] fixes BUG-42399: toolbar reports should not use read-only pool The reports generated with the print toolbar button which are based in the PrintController engine, should not make use of the read-only pool. This is because more than reports they are rendering information of the transactional data. This should have been fixed as part of [1], but that fix is missing the part of the Report information retrieval. [1] https://issues.openbravo.com/view.php?id=37933 [^] --- M src/org/openbravo/erpCommon/utility/reporting/Report.java --- |
(0116146) alostale (manager) 2019-12-09 11:23 |
https://gitlab.com/openbravocode/product/openbravo/merge_requests/21 [^] |
Issue History | |||
Date Modified | Username | Field | Change |
2019-11-26 11:43 | egoitz | New Issue | |
2019-11-26 11:43 | egoitz | Assigned To | => platform |
2019-11-26 11:43 | egoitz | Modules | => Core |
2019-11-26 11:43 | egoitz | Resolution time | => 1576105200 |
2019-11-26 11:43 | egoitz | Triggers an Emergency Pack | => No |
2019-11-26 13:27 | AugustoMauch | Relationship added | related to 0037933 |
2019-11-26 18:34 | caristu | Assigned To | platform => caristu |
2019-11-28 13:32 | caristu | Status | new => acknowledged |
2019-11-28 13:32 | caristu | Status | acknowledged => scheduled |
2019-12-09 10:24 | hgbot | Checkin | |
2019-12-09 10:24 | hgbot | Note Added: 0116143 | |
2019-12-09 10:24 | hgbot | Status | scheduled => resolved |
2019-12-09 10:24 | hgbot | Resolution | open => fixed |
2019-12-09 10:24 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/f45e0aa5902dbb37011ac716555e74830c09f0a5 [^] |
2019-12-09 11:22 | alostale | Review Assigned To | => alostale |
2019-12-09 11:22 | alostale | Status | resolved => closed |
2019-12-09 11:22 | alostale | Fixed in Version | => 3.0PR20Q1 |
2019-12-09 11:23 | alostale | Note Added: 0116146 |
Copyright © 2000 - 2009 MantisBT Group |