Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0029774
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Cash upminoralways2015-05-05 11:322015-05-28 09:28
ReportermalsasuaView Statuspublic 
Assigned Tomalsasua 
PrioritynormalResolutionfixedFixed in VersionRR15Q3
StatusclosedFix in branchFixed in SCM revisione5228d71126f
ProjectionnoneETAnoneTarget VersionRR15Q3
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionCommunity ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionSCM revision 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0029774: [cashup refactor] improve the performance of cashup

DescriptionChange 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 Reproducen/a
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0029002 closedmalsasua Performance problem doing concurrent cash ups 
causes defect 0030174RR15Q3 closedmtaal Triggers temporarily disabled for whole application through webpos 
causes defect 0031820 closedranjith_qualiantech_com Duplicated invoice lines are added to an invoice during the cash up if the order line is already invoiced in more than one line 

-  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
---

- 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 OBNetwork customer => No
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

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker