Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0027280 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 07. Sales management | major | always | 2014-08-05 15:17 | 2015-01-26 16:22 | |||
| Reporter | ngarcia | View Status | public | |||||
| Assigned To | reinaldoguerra | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | e96f620e6758 | ||||
| Projection | none | ETA | none | Target Version | 3.0PR15Q1 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | pramakrishnan | |||||||
| OBNetwork customer | OBPS | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 30125 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0027280: Commission payment details are duplicated if the invoice has fractionated payment terms | |||||||
| Description | Commission payment details are duplicated if the invoice has fractionated payment terms | |||||||
| Steps To Reproduce | As group admin role: Create a Payment Term header: Overdue Payment Days Rule: 30 Add a line to it: Percentage due: 50 Overdue Payment Days Rule: 0 Create a Commission with: Business Partner / Sales Rep.: define one Frequency Type: Monthly Basis Document: Invoice Basis Status. Fully paid docuemnts Basis Amount: Net amount Add a line to it: Based on Sales Rep. = Y Multiplier amount: 0.5 Create a Sales Invoice and associate the previously created payment terms and the sales representative to it. Complete it and pay the whole amount Generate Commission Go to the Commission Payment - Amounts - Details tab and check the invoice appears twice | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0071837) reinaldoguerra (viewer) 2014-11-20 01:54 edited on: 2014-12-11 00:15 |
Test plan used: 1- Login into Openbravo ERP 2- Go to Payment Term window and create a new header record. 3- Set Overdue Payment Days Rule: 30 4- Add a new line to it 5- Set Percentage due: 50 and Overdue Payment Days Rule: 0 6- Go to Commission window and create a new header 7- Select a Business Partner / Sales Rep (create a new one if is required). Set Frequency Type: Monthly Basis Document: Invoice Basis Status. Fully paid docuemnts Basis Amount: Net amount 8- Go to lines tab and create a new record 9- Check Based on Sales Rep. flag and set Multiplier amount: 0.5 10- Go to Sales Invoice window and create a new one selecting the same sales representative and the previously created payment terms. 11- Add a line with a product 11- Complete it 12- Pay the whole amount by clicking add payment button, realize that two record are selected in the invoice grid as the invoice has fractionated payment terms. Complete the process. 13- Go back to Commission window, select previous commision record and click in Generate Commission button 14- Go to the Commission Payment window, select created record, go to Amounts tab and then select Details tab. 15- Realize that one record per invoice is shown. 16- Execute ant update.database and realize that duplicated data are deleted for other Commission Payments. Also quantities and amounts are correctly updated. |
|
(0072417) hgbot (developer) 2014-12-11 09:51 |
Repository: erp/devel/pi Changeset: e96f620e675872dff29917e9e2363d533592dc57 Author: Reinaldo Guerra <reinaldo.guerra <at> peoplewalking.com> Date: Wed Dec 10 17:48:00 2014 -0500 URL: http://code.openbravo.com/erp/devel/pi/rev/e96f620e675872dff29917e9e2363d533592dc57 [^] Fixed bug 27280: Commission payment details are duplicated in some cases. Generate commision process was changed to exclude the same records when retrieving data to be inserted in the payment detail table. This information comes duplicated due to the payment is divided in two records as the invoice has fractionated payment terms. Thus the stored procedure was changed only in those cases when the commission basis document is an invoice, and the information related with the payments and its products are joined in the same query, avoiding duplicated tuples. Now there are not duplicated fields when processing commissions for invoices with fractionated payment terms. Also a module script was developed to correct inconsistent data. When executing an ant update database duplicated data in commission detail table will be deleted, also amounts and quantities will be updated in commission amount and commission payment tables with the correct information. --- M src-db/database/model/functions/C_COMMISSION_PROCESS.xml A src-util/modulescript/build/classes/org/openbravo/modulescript/DeleteDuplicateCommissionDetail.class A src-util/modulescript/build/classes/org/openbravo/modulescript/DeleteDuplicateCommissionDetailData.class A src-util/modulescript/src/org/openbravo/modulescript/DeleteDuplicateCommissionDetail.java A src-util/modulescript/src/org/openbravo/modulescript/DeleteDuplicateCommissionDetail_data.xsql --- |
|
(0072418) pramakrishnan (viewer) 2014-12-11 09:53 |
Code Review + Testing - OK. |
|
(0072443) hgbot (developer) 2014-12-12 07:04 |
Repository: erp/devel/pi Changeset: 70a223e52c0844e430449ed1dd8655c31b19dc0d Author: Reinaldo Guerra <reinaldo.guerra <at> peoplewalking.com> Date: Thu Dec 11 12:43:38 2014 -0500 URL: http://code.openbravo.com/erp/devel/pi/rev/70a223e52c0844e430449ed1dd8655c31b19dc0d [^] Related to issue 27280: Changes required in module script in order to improve it Some changes were made in Delete Duplicate Commission Detail module script in order to improve it. A short comment indicating the issue to solve, was added in the java class and the dummy sql method was replaced for the really used selecting logic. --- M src-util/modulescript/build/classes/org/openbravo/modulescript/DeleteDuplicateCommissionDetail.class M src-util/modulescript/build/classes/org/openbravo/modulescript/DeleteDuplicateCommissionDetailData.class M src-util/modulescript/src/org/openbravo/modulescript/DeleteDuplicateCommissionDetail.java M src-util/modulescript/src/org/openbravo/modulescript/DeleteDuplicateCommissionDetail_data.xsql --- |
|
(0073207) hudsonbot (viewer) 2014-12-30 23:27 |
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/6525fe229e06 [^] Maturity status: Test |
|
(0073210) hudsonbot (viewer) 2014-12-30 23:27 |
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/6525fe229e06 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2014-08-05 15:17 | ngarcia | New Issue | |
| 2014-08-05 15:17 | ngarcia | Assigned To | => jonalegriaesarte |
| 2014-08-05 15:17 | ngarcia | OBNetwork customer | => No |
| 2014-08-05 15:17 | ngarcia | Modules | => Core |
| 2014-08-05 15:17 | ngarcia | Support ticket | => 30125 |
| 2014-08-05 15:17 | ngarcia | Resolution time | => 1409608800 |
| 2014-08-05 15:17 | ngarcia | Triggers an Emergency Pack | => No |
| 2014-08-05 15:18 | ngarcia | Issue Monitored: networkb | |
| 2014-10-02 11:07 | ngarcia | OBNetwork customer | No => Yes |
| 2014-10-02 11:42 | jonalegriaesarte | Assigned To | jonalegriaesarte => AtulOpenbravo |
| 2014-10-02 11:42 | jonalegriaesarte | Target Version | => 3.0PR15Q1 |
| 2014-11-19 00:08 | reinaldoguerra | Assigned To | AtulOpenbravo => reinaldoguerra |
| 2014-11-19 00:08 | reinaldoguerra | Status | new => scheduled |
| 2014-11-20 01:54 | reinaldoguerra | Note Added: 0071837 | |
| 2014-11-20 01:56 | reinaldoguerra | Note Edited: 0071837 | View Revisions |
| 2014-12-06 05:05 | reinaldoguerra | Note Edited: 0071837 | View Revisions |
| 2014-12-11 00:15 | reinaldoguerra | Note Edited: 0071837 | View Revisions |
| 2014-12-11 09:51 | hgbot | Checkin | |
| 2014-12-11 09:51 | hgbot | Note Added: 0072417 | |
| 2014-12-11 09:51 | hgbot | Status | scheduled => resolved |
| 2014-12-11 09:51 | hgbot | Resolution | open => fixed |
| 2014-12-11 09:51 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/e96f620e675872dff29917e9e2363d533592dc57 [^] |
| 2014-12-11 09:53 | pramakrishnan | Review Assigned To | => pramakrishnan |
| 2014-12-11 09:53 | pramakrishnan | Note Added: 0072418 | |
| 2014-12-11 09:53 | pramakrishnan | Status | resolved => closed |
| 2014-12-12 07:04 | hgbot | Checkin | |
| 2014-12-12 07:04 | hgbot | Note Added: 0072443 | |
| 2014-12-30 23:27 | hudsonbot | Checkin | |
| 2014-12-30 23:27 | hudsonbot | Note Added: 0073207 | |
| 2014-12-30 23:27 | hudsonbot | Checkin | |
| 2014-12-30 23:27 | hudsonbot | Note Added: 0073210 | |
| 2015-01-26 16:22 | ngarcia | Issue cloned | 0028772 |
| 2015-06-15 16:08 | ngarcia | Relationship added | causes 0030184 |
| 2015-06-16 19:31 | aferraz | Relationship deleted | causes 0030184 |
| 2015-06-16 19:31 | aferraz | Relationship added | related to 0030184 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |