Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0033082 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] Z. Others | major | always | 2016-05-31 11:27 | 2016-06-17 19:38 | |||
| Reporter | maite | View Status | public | |||||
| Assigned To | AtulOpenbravo | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR16Q3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 16b9d36614a8 | ||||
| Projection | none | ETA | none | Target Version | 3.0PR16Q3 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | aferraz | |||||||
| Web browser | ||||||||
| Modules | Advanced Payables and Receivables Mngmt | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0033082: Payment Schedule Detail duplicated when using "Modify Payment Plan" process under certain circumstances | |||||||
| Description | Payment schedule detail is duplicated when "Modify Payment Plan" process is run from a partial payment plan record which has payment in Void status | |||||||
| Steps To Reproduce | 0. As system administrator, access to "Windows, tabs and fields" and go to Sales Invoice window | Payment Plan tab to set "Modify payment plan" field as DISPLAYED. Logout/login 1. Access Payment Terms window and configure new record as "Overdue Payment Days Rule"=30 in Header. Add Line with "Percentage due"=30 and "Overdue Payment Days Rule"=15 2. Register Sales invoice for España org using this payment terms. Add any line and process invoice 3. Access Financial Account "Cuenta de banco", go to Payment Method tab and set "Transferencia" payment method as "Automatic", "Simple exec process" and "Deferred=YES" 4. From invoice header, run "Add Payment" process. Select only first payment plan record and process 5. Navigate to created Payment In document, which should be in "Awaiting Exec" status. Click on "Reactivate" button and select VOID action. Realize that payment is correct and it only has one detail related 6. Access Sales Invoice and go to Payment Plan tab. Run "Modify Payment Plan" process from previous record and change Payment Method info to "Cheque". Process 7. Access again to previous Payment In record, refresh document and realize that it has 2 details, which is not correct 8. Access Sales Invoice and go to Payment Plan tab, select second line, the one that still has Transferencia as Payment Method. Check that it has a payment detail and that is not correct. (Received Amount = 30 and Amount = 70, related to payment in (1000513). 9. Run Modify Payment Plan for this second payment plan line, change it to Cheque. Press Submit. Refresh. Check that now this payment plan has two payment details lines which is not correct. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0087081) AtulOpenbravo (developer) 2016-06-08 08:47 edited on: 2016-06-14 13:08 |
Test Plan - As system administrator, access to "Windows, tabs and fields" and go to Sales Invoice window | Payment Plan tab to set "Modify payment plan" field as DISPLAYED. Logout/login. - Access Payment Terms window and configure new record as "Overdue Payment Days Rule"=30 in Header. Add Line with "Percentage due"=30 and "Overdue Payment Days Rule"=15. - Register Sales invoice for España org using this payment terms. Add any line and process invoice. - Go to payment plan tab and check that there are two payment plan lines, first one with an expected date 15 days after invoice date for a 30% of the total gross amount of the invoice. - Access Financial Account "Cuenta de banco", go to Payment Method tab and set "Transferencia" payment method as "Automatic", "Simple exec process" and "Deferred=YES". - From invoice header, run "Add Payment" process. Select only first payment plan record and process. - Navigate to created Payment In document, which should be in "Awaiting Exec" status. Click on "Reactivate" button and select VOID action. Realize that payment is correct and it only has one detail related - Access Sales Invoice and go to Payment Plan tab. Run "Modify Payment Plan" process from previous record and change Payment Method info to "Cheque". Process. - Access again to previous Payment In record, refresh document and check that it has only 1 detail, which is correct. |
|
(0087251) psanjuan (manager) 2016-06-14 13:13 |
Issue verified. |
|
(0087254) psanjuan (manager) 2016-06-14 14:00 edited on: 2016-06-15 10:08 |
Test Plan - As system administrator, access to "Windows, tabs and fields" and go to Sales Invoice window | Payment Plan tab to set "Modify payment plan" field as DISPLAYED. Logout/login. Cheque - Access Payment Terms window and configure new record as "Overdue Payment Days Rule"=30 in Header. Add Line with "Percentage due"=30 and "Overdue Payment Days Rule"=15. - Register Sales invoice for F&B España org using this payment terms. Add any line and process invoice. - Go to payment plan tab and check that there are two payment plan lines, first one with an expected date 15 days after invoice date for a 30% of the total gross amount of the invoice, for instance 30.00 (total gross amount 100.00). - Access Financial Account "Cuenta de banco", go to Payment Method tab and set "Transferencia" payment method as "Automatic", "Simple exec process" and "Deferred=YES". - From invoice header, run "Add Payment" process. Select only first payment plan record and process. - Navigate to created Payment In document, which should be in "Awaiting Exec" status. Click on "Reactivate" button and select VOID action. Realize that payment is correct and it only has one detail related. - Access Sales Invoice and go to Payment Plan tab. Run "Modify Payment Plan" process from previous record and change Payment Method info to "Cheque". Process. - Verify that there is only one line in Payment Details sub tab, still referring to Transferencia Payment Method in Status Void. - Access again to previous Payment In record, refresh document and check that it has only 1 detail, which is correct. - Go back to the sales invoice. Go to Payment Plan tab and select second line. Change Payment Method to Check by using "Modify Payment Plan" button. - Check that payment method is changed to Cheque and that there is no payment details associated to this payment plan line. Repeat same steps as above for another method of payment that is not an "Automatic" one. - Add a payment for the payment plan line changed to Cheque (that is setup as Manual one). - Reactivate that payment once received and delete its lines. - Change that payment plan line to "Transferencia". - Check it is possible to make that change while no additional payment detail line is created. |
|
(0087289) psanjuan (manager) 2016-06-15 10:08 |
Test plan above verified. Issue can be code reviewed and closed. |
|
(0087303) hgbot (developer) 2016-06-15 14:23 |
Repository: erp/devel/pi Changeset: 16b9d36614a8ca6269c942ab40778e89a8b5dbfd Author: Atul Gaware <atul.gaware <at> openbravo.com> Date: Wed Jun 08 11:58:42 2016 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/16b9d36614a8ca6269c942ab40778e89a8b5dbfd [^] Fixes issue 33082: Payment Schedule Detail duplicated in "Modify Payment Plan" Avoid creation of payment schedule detail for cancelled psd if there exists for a payment detail. --- M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java --- |
|
(0087304) hgbot (developer) 2016-06-15 14:23 |
Repository: erp/devel/pi Changeset: ae98f8c225239988a5a24b6696e23128214a3504 Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Wed Jun 15 14:20:57 2016 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/ae98f8c225239988a5a24b6696e23128214a3504 [^] Related to issue 33082: Code review improvements --- M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java --- |
|
(0087305) hgbot (developer) 2016-06-15 14:27 |
Repository: erp/devel/pi Changeset: be5679c6e512b2e83b181b5d43d4c884452efddc Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Wed Jun 15 14:26:32 2016 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/be5679c6e512b2e83b181b5d43d4c884452efddc [^] Related to issue 33082: Improve method doc --- M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java --- |
|
(0087306) aferraz (manager) 2016-06-15 14:28 |
Code review OK |
|
(0087620) hudsonbot (developer) 2016-06-17 19:38 |
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/0dc7be081b1c [^] Maturity status: Test |
|
(0087621) hudsonbot (developer) 2016-06-17 19:38 |
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/0dc7be081b1c [^] Maturity status: Test |
|
(0087622) hudsonbot (developer) 2016-06-17 19:38 |
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/0dc7be081b1c [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2016-05-31 11:27 | maite | New Issue | |
| 2016-05-31 11:27 | maite | Assigned To | => Triage Finance |
| 2016-05-31 11:27 | maite | Modules | => Advanced Payables and Receivables Mngmt |
| 2016-05-31 11:27 | maite | Resolution time | => 1465855200 |
| 2016-05-31 11:27 | maite | Triggers an Emergency Pack | => No |
| 2016-05-31 11:27 | maite | Issue Monitored: networkb | |
| 2016-05-31 11:27 | maite | Target Version | => 3.0PR16Q3 |
| 2016-05-31 11:37 | heccam | Issue Monitored: heccam | |
| 2016-05-31 13:13 | aferraz | Assigned To | Triage Finance => AtulOpenbravo |
| 2016-06-06 11:11 | AtulOpenbravo | Status | new => scheduled |
| 2016-06-08 08:47 | AtulOpenbravo | Note Added: 0087081 | |
| 2016-06-14 13:08 | psanjuan | Note Edited: 0087081 | View Revisions |
| 2016-06-14 13:13 | psanjuan | Note Added: 0087251 | |
| 2016-06-14 13:25 | psanjuan | Steps to Reproduce Updated | View Revisions |
| 2016-06-14 13:26 | psanjuan | Steps to Reproduce Updated | View Revisions |
| 2016-06-14 14:00 | psanjuan | Note Added: 0087254 | |
| 2016-06-14 14:00 | psanjuan | Steps to Reproduce Updated | View Revisions |
| 2016-06-14 14:11 | psanjuan | Note Edited: 0087254 | View Revisions |
| 2016-06-14 14:14 | psanjuan | Note Edited: 0087254 | View Revisions |
| 2016-06-14 15:24 | psanjuan | Note Edited: 0087254 | View Revisions |
| 2016-06-14 15:28 | psanjuan | Relationship added | related to 0033260 |
| 2016-06-14 15:28 | psanjuan | Note Edited: 0087254 | View Revisions |
| 2016-06-15 10:08 | psanjuan | Note Edited: 0087254 | View Revisions |
| 2016-06-15 10:08 | psanjuan | Note Added: 0087289 | |
| 2016-06-15 14:23 | hgbot | Checkin | |
| 2016-06-15 14:23 | hgbot | Note Added: 0087303 | |
| 2016-06-15 14:23 | hgbot | Status | scheduled => resolved |
| 2016-06-15 14:23 | hgbot | Resolution | open => fixed |
| 2016-06-15 14:23 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/16b9d36614a8ca6269c942ab40778e89a8b5dbfd [^] |
| 2016-06-15 14:23 | hgbot | Checkin | |
| 2016-06-15 14:23 | hgbot | Note Added: 0087304 | |
| 2016-06-15 14:27 | hgbot | Checkin | |
| 2016-06-15 14:27 | hgbot | Note Added: 0087305 | |
| 2016-06-15 14:28 | aferraz | Review Assigned To | => aferraz |
| 2016-06-15 14:28 | aferraz | Note Added: 0087306 | |
| 2016-06-15 14:28 | aferraz | Status | resolved => closed |
| 2016-06-15 14:28 | aferraz | Fixed in Version | => 3.0PR16Q3 |
| 2016-06-17 19:38 | hudsonbot | Checkin | |
| 2016-06-17 19:38 | hudsonbot | Note Added: 0087620 | |
| 2016-06-17 19:38 | hudsonbot | Checkin | |
| 2016-06-17 19:38 | hudsonbot | Note Added: 0087621 | |
| 2016-06-17 19:38 | hudsonbot | Checkin | |
| 2016-06-17 19:38 | hudsonbot | Note Added: 0087622 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |