Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0029073 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 07. Sales management | major | always | 2015-02-26 11:29 | 2015-03-27 10:25 | |||
| Reporter | egoitz | View Status | public | |||||
| Assigned To | umartirena | |||||||
| Priority | immediate | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | 35a4e54c3ffc | ||||
| Projection | none | ETA | none | Target Version | 3.0PR15Q2 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | eduardo_Argal | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0029073: Review deleting logic of tax lines in C_INVOICELINE_TRG2 | |||||||
| Description | When posting a Sales Invoice, logic for recreating the tax lines should be reviewed in order to modify existing ones instead of deleting and recreating them. | |||||||
| Steps To Reproduce | Using PG: -Enable function statistics: -In postgresql.conf set track_functions = all -/etc/init.d/postgresql reload -Create a sales invoice with one line -Reset pg statistics: select pg_stat_reset(); -Book it * Check function calls: select funcname, calls, total_time, self_time from pg_stat_user_functions order by self_time desc | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||||||||||||||||
|
||||||||||||||||||||||
 Relationships |
|
Notes |
|
|
(0075845) hgbot (developer) 2015-03-23 10:28 |
Repository: erp/devel/pi Changeset: 35a4e54c3ffc47aa4c1ee6507085032b6ac32ae6 Author: Unai Martirena <unai.martirena <at> openbravo.com> Date: Wed Mar 11 13:52:26 2015 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/35a4e54c3ffc47aa4c1ee6507085032b6ac32ae6 [^] Fixes bug 29073: InvoiceLine tax delete only is done when is needed C_INVOICELINE_TRG2 has been changed to better manage when Invoice Line taxes are deleted and created again. Instead of doing it on every time an invoiceline is inserted or updated, it will be done when certain values in Invoice Line are changed and this affects in the related Invoice Line Tax record. The result will be the same, because with the previous code when there was no change in the Invoice Line the Invoice Line taxes were recreated with the same values as before, and now will be skipped. Apart of this previous change, all not used variables have been removed, and in the same way a complete select sentence that was obtaining values that were not used later. --- M src-db/database/model/triggers/C_INVOICELINE_TRG2.xml --- |
|
(0075846) eduardo_Argal (developer) 2015-03-23 10:31 |
Code Reviewed and tested. Performance figures in testing (using 4000 lines invoice) Before: 732696.439; After: 88269.735; |
|
(0075884) hudsonbot (developer) 2015-03-23 16:24 |
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/b9d952133e76 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2015-02-26 11:29 | egoitz | New Issue | |
| 2015-02-26 11:29 | egoitz | Assigned To | => Sandrahuguet |
| 2015-02-26 11:29 | egoitz | Modules | => Core |
| 2015-02-26 11:29 | egoitz | Resolution time | => 1425337200 |
| 2015-02-26 11:29 | egoitz | Triggers an Emergency Pack | => No |
| 2015-02-26 11:30 | egoitz | Relationship added | related to 0027338 |
| 2015-02-26 11:50 | jpcalvente | Issue Monitored: jpcalvente | |
| 2015-02-26 12:04 | egoitz | Target Version | => 3.0PR15Q2 |
| 2015-02-27 11:25 | dmitry_mezentsev | Relationship added | related to 0027160 |
| 2015-03-23 10:28 | eduardo_Argal | Status | new => scheduled |
| 2015-03-23 10:28 | eduardo_Argal | Assigned To | Sandrahuguet => umartirena |
| 2015-03-23 10:28 | hgbot | Checkin | |
| 2015-03-23 10:28 | hgbot | Note Added: 0075845 | |
| 2015-03-23 10:28 | hgbot | Status | scheduled => resolved |
| 2015-03-23 10:28 | hgbot | Resolution | open => fixed |
| 2015-03-23 10:28 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/35a4e54c3ffc47aa4c1ee6507085032b6ac32ae6 [^] |
| 2015-03-23 10:31 | eduardo_Argal | Review Assigned To | => eduardo_Argal |
| 2015-03-23 10:31 | eduardo_Argal | Note Added: 0075846 | |
| 2015-03-23 10:31 | eduardo_Argal | Status | resolved => closed |
| 2015-03-23 16:24 | hudsonbot | Checkin | |
| 2015-03-23 16:24 | hudsonbot | Note Added: 0075884 | |
| 2015-03-27 10:25 | umartirena | Relationship added | related to 0029431 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |