Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0029774 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Cash up | minor | always | 2015-05-05 11:32 | 2015-05-28 09:28 | |||
| Reporter | malsasua | View Status | public | |||||
| Assigned To | malsasua | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | RR15Q3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | e5228d71126f | ||||
| Projection | none | ETA | none | Target Version | RR15Q3 | |||
| OS | Linux 32 bit | Database | PostgreSQL | Java version | 1.6.0_18 | |||
| OS Version | Community Appliance | Database version | 8.3.9 | Ant version | 1.7.1 | |||
| Product Version | SCM revision | |||||||
| Review Assigned To | marvintm | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0029774: [cashup refactor] improve the performance of cashup | |||||||
| Description | Change the way that the invoices created in the cashup are inserted in database: instead of to do an insert of one record by one record, the inserts are done with insert select, so the performance is better Project: http://wiki.openbravo.com/wiki/Projects:CashUp_Improve_Performance [^] | |||||||
| Steps To Reproduce | n/a | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||||||||||||||||
|
||||||||||||||||||||||
 Relationships |
|
Notes |
|
|
(0077009) hgbot (developer) 2015-05-05 14:05 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 3e3be280c7ca6fa14de6d864558f32dd98a06159 Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Tue May 05 14:04:32 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3e3be280c7ca6fa14de6d864558f32dd98a06159 [^] [cashup refactor] fixed issue 29774: changed the way that the records are inserted in the database --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java A src/org/openbravo/retail/posterminal/OrderGroupingProcessor_data.xsql --- |
|
(0077014) hgbot (developer) 2015-05-05 15:56 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 411a1e9884622bb8070f283ec744e5604d7ca5c4 Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Tue May 05 15:55:59 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/411a1e9884622bb8070f283ec744e5604d7ca5c4 [^] [cashup refactor] related to issue 29774: updated text license --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor_data.xsql --- |
|
(0077046) hgbot (developer) 2015-05-06 11:12 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: f3caa1b60a50ea3e1184f83e1b6878088028571e Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Wed May 06 11:11:39 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f3caa1b60a50ea3e1184f83e1b6878088028571e [^] [cashup refactor] fixed issue 29774: orders in cashup are not filtered by terminalId, only is filtered by cashupId --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java M src/org/openbravo/retail/posterminal/OrderGroupingProcessor_data.xsql --- |
|
(0077096) hgbot (developer) 2015-05-07 16:30 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 28882912b8a6ff4ae005af22a5b3783323579105 Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Tue May 12 16:29:17 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/28882912b8a6ff4ae005af22a5b3783323579105 [^] [cashup refactor] fixed issue 29774: changed 'group by' expression, removing constant sting --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java M src/org/openbravo/retail/posterminal/OrderGroupingProcessor_data.xsql --- |
|
(0077327) hgbot (developer) 2015-05-12 11:08 |
Repository: tools/automation/pi-mobile Changeset: c1c04ba1c3a5953acbc10e4d1d956be53cb390db Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Tue May 05 14:21:23 2015 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/c1c04ba1c3a5953acbc10e4d1d956be53cb390db [^] Related to issue 29530, verifies issue 29774: cashup refactor] improve the performance of cashup --- M src-test/org/openbravo/test/mobile/common/selenium/database/InvoiceLinesOfDocumentnoRow.java M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseConstants.java M src-test/org/openbravo/test/mobile/common/selenium/database/WebPOSDatabaseHelper.java M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java M src-test/org/openbravo/test/mobile/common/selenium/testhelpers/WebPOSSnippet.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/Layaway_Cre1Fin1.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/Layaway_Cre1Fin2.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/Layaway_Cre1Par2Fin1.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/Layaway_Cre1Par2Fin2.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/Layaway_Cre1Par2Fin3.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpGroupingWithDifferentsBP.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpNoGroupingWithDifferentsBP.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpSplitLines.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpSplitLinesNotIncludeTaxes.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpWithPartialPaidCredit.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpWithTotalPaidCredit.java --- |
|
(0077328) hgbot (developer) 2015-05-12 11:08 |
Repository: tools/automation/pi-mobile Changeset: bb9819cf007e6b307d6f7c086a6da6eb0fd4aba7 Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Mon May 11 18:27:17 2015 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/bb9819cf007e6b307d6f7c086a6da6eb0fd4aba7 [^] verified issue 29774: removed unused variables --- M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpGroupingWithDifferentsBP.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpNoGroupingWithDifferentsBP.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpSplitLines.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpSplitLinesNotIncludeTaxes.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpWithPartialPaidCredit.java M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpWithTotalPaidCredit.java --- |
|
(0077685) hgbot (developer) 2015-05-25 10:11 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: e5228d71126f3cd8a21d57c1f5ccda7689e025fa Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Mon May 25 10:10:35 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/e5228d71126f3cd8a21d57c1f5ccda7689e025fa [^] [cashup refactor] fixed issue 29774: small changes in the code to do it more clear --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java --- |
|
(0077708) hgbot (developer) 2015-05-25 12:59 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: d5bfab028cd340c468dfa61be4875ba0a52eff7c Author: Antonio Moreno <antonio.moreno <at> openbravo.com> Date: Mon May 25 12:59:24 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d5bfab028cd340c468dfa61be4875ba0a52eff7c [^] Related to issue 29774. Made the code safer to avoid any potential collision. --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java --- |
|
(0077853) hgbot (developer) 2015-05-28 09:28 |
Repository: tools/automation/pi-mobile Changeset: 039bb38b2e8a0df6cec4e82d3dd5ea75903a437c Author: Miguel A. Alsasua <miguel.alsasua <at> openbravo.com> Date: Thu May 21 20:09:46 2015 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/039bb38b2e8a0df6cec4e82d3dd5ea75903a437c [^] verified issue 29774:[cashup refactor] added two test cases with Multitax --- A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpSplitLinesMultitax.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/CashUpSplitLinesMultitaxNotIncludeTaxes.java --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2015-05-05 11:32 | malsasua | New Issue | |
| 2015-05-05 11:32 | malsasua | Assigned To | => Retail |
| 2015-05-05 11:32 | malsasua | Triggers an Emergency Pack | => No |
| 2015-05-05 12:02 | malsasua | Description Updated | View Revisions |
| 2015-05-05 14:05 | hgbot | Checkin | |
| 2015-05-05 14:05 | hgbot | Note Added: 0077009 | |
| 2015-05-05 14:05 | hgbot | Status | new => resolved |
| 2015-05-05 14:05 | hgbot | Resolution | open => fixed |
| 2015-05-05 14:05 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3e3be280c7ca6fa14de6d864558f32dd98a06159 [^] |
| 2015-05-05 15:56 | hgbot | Checkin | |
| 2015-05-05 15:56 | hgbot | Note Added: 0077014 | |
| 2015-05-06 11:12 | hgbot | Checkin | |
| 2015-05-06 11:12 | hgbot | Note Added: 0077046 | |
| 2015-05-06 11:12 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3e3be280c7ca6fa14de6d864558f32dd98a06159 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f3caa1b60a50ea3e1184f83e1b6878088028571e [^] |
| 2015-05-06 14:13 | malsasua | Assigned To | Retail => malsasua |
| 2015-05-07 10:19 | malsasua | Relationship added | related to 0029002 |
| 2015-05-07 16:30 | hgbot | Checkin | |
| 2015-05-07 16:30 | hgbot | Note Added: 0077096 | |
| 2015-05-07 16:30 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f3caa1b60a50ea3e1184f83e1b6878088028571e [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/28882912b8a6ff4ae005af22a5b3783323579105 [^] |
| 2015-05-07 18:51 | malsasua | Description Updated | View Revisions |
| 2015-05-12 11:08 | hgbot | Checkin | |
| 2015-05-12 11:08 | hgbot | Note Added: 0077327 | |
| 2015-05-12 11:08 | hgbot | Checkin | |
| 2015-05-12 11:08 | hgbot | Note Added: 0077328 | |
| 2015-05-25 10:11 | hgbot | Checkin | |
| 2015-05-25 10:11 | hgbot | Note Added: 0077685 | |
| 2015-05-25 10:11 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/28882912b8a6ff4ae005af22a5b3783323579105 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/e5228d71126f3cd8a21d57c1f5ccda7689e025fa [^] |
| 2015-05-25 12:59 | hgbot | Checkin | |
| 2015-05-25 12:59 | hgbot | Note Added: 0077708 | |
| 2015-05-25 13:00 | marvintm | Review Assigned To | => marvintm |
| 2015-05-25 13:00 | marvintm | Status | resolved => closed |
| 2015-05-25 13:00 | marvintm | Fixed in Version | => RR15Q3 |
| 2015-05-28 09:28 | hgbot | Checkin | |
| 2015-05-28 09:28 | hgbot | Note Added: 0077853 | |
| 2015-06-15 08:26 | alostale | Relationship added | causes 0030174 |
| 2016-01-26 19:51 | dmitry_mezentsev | Relationship added | causes 0031820 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |