Openbravo Issue Tracking System - Modules
View Issue Details
0038288ModulesRemittancepublic2018-04-06 11:452018-05-09 19:29
ngarcia 
collazoandy4 
urgentmajoralways
closedfixed 
5
 
 
dmiguelez
0038288: Sales Invoice Payment Plan Due date can be modified when related payment is in Remitted status
Sales Invoice Payment Plan Due date can be modified when related payment is in Remitted status

Remittances only accept documents with due date prior to their expected date, so if the due date of a payment plan is increased it will not fulfill that condition
As system admin role:
   Set as Displayed the 'Sales Invoice - Payment Plan - Modify Payment Plan'
   Compile the tab and restart tomcat

As group admin role:
   Create a Sales Invoice, set Remittance as its payment method, add a line to it and complete it
   Add a payment to it, execute it and include it on a remittance
   Return to the Sales Invoice, select the payment plan record and Modify Payment Plan
   Check you can increase the due date and you should not
No tags attached.
related to feature request 0038425 new Triage Omni OMS Support for Editable Payment Plan 
causes defect 0038509 closed AtulOpenbravo Sales Invoice Payment Plan Due date cannot be modified when related remitted payment has been protested and voided 
Issue History
2018-04-06 11:45ngarciaNew Issue
2018-04-06 11:45ngarciaAssigned To => Triage Finance
2018-04-06 11:45ngarciaResolution time => 1524780000
2018-04-06 12:10PracticsIssue Monitored: Practics
2018-04-06 19:12dmiguelezAssigned ToTriage Finance => collazoandy4
2018-04-09 23:41collazoandy4Statusnew => scheduled
2018-04-10 23:16collazoandy4Note Added: 0103801
2018-04-23 13:11ngarciaRelationship addedrelated to 0038425
2018-04-23 18:08collazoandy4Note Edited: 0103801bug_revision_view_page.php?bugnote_id=0103801#r17045
2018-04-30 16:54hgbotCheckin
2018-04-30 16:54hgbotNote Added: 0104172
2018-04-30 16:54hgbotStatusscheduled => resolved
2018-04-30 16:54hgbotResolutionopen => fixed
2018-04-30 16:54hgbotFixed in SCM revision => http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/c369ffae37fc26a073a460ffb7faa03364a381b9 [^]
2018-04-30 16:54hgbotCheckin
2018-04-30 16:54hgbotNote Added: 0104173
2018-04-30 16:54hgbotCheckin
2018-04-30 16:54hgbotNote Added: 0104174
2018-04-30 16:54hgbotCheckin
2018-04-30 16:54hgbotNote Added: 0104175
2018-04-30 16:54hgbotCheckin
2018-04-30 16:54hgbotNote Added: 0104176
2018-04-30 16:54hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/c369ffae37fc26a073a460ffb7faa03364a381b9 [^] => http://code.openbravo.com/erp/devel/pi/rev/bbec1f5c73008c05f8c22c03cf61addf6ea0ce80 [^]
2018-04-30 16:54hgbotCheckin
2018-04-30 16:54hgbotNote Added: 0104177
2018-04-30 16:56dmiguelezReview Assigned To => dmiguelez
2018-04-30 16:56dmiguelezNote Added: 0104178
2018-04-30 16:56dmiguelezStatusresolved => closed
2018-05-09 16:12ngarciaIssue cloned0038509
2018-05-09 16:12ngarciaRelationship addedcaused by 0038509
2018-05-09 16:12ngarciaRelationship replacedcauses 0038509
2018-05-09 19:29hudsonbotCheckin
2018-05-09 19:29hudsonbotNote Added: 0104411
2018-05-09 19:29hudsonbotCheckin
2018-05-09 19:29hudsonbotNote Added: 0104412

Notes
(0103801)
collazoandy4   
2018-04-10 23:16   
(edited on: 2018-04-23 18:08)
Test Plan
  As System Admin role:
   Set as Displayed the Modify Payment Plan field in Payment Plan tab of Sales Invoice
   Compile the tab and restart tomcat

  As International Group Admin Role:
   Create a new Sales Invoice for any bp, set Transferencia as payment method, add a line to it and complete it
     Go to Payment Plan tab and clic on Modify Payment Plan button and check the process ends without error
   
Test Plan II
   Create a new Sales Invoice for any bp, set Remittance as its payment method, add a line to it and complete it
   Add a payment to it, execute it and include it on a remittance
   Go to the previous invoice created, select the payment plan record and clic on Modify Payment Plan
   Check the process shows the error message 'The invoice is already included in a Remittance.'

Test Plan III
   Create a new Sales Invoice for any bp, set Remittance as its payment method, add a line to it and complete it
   Create a new Remittance header and add the previous invoice as remittance line (Select Order or Invoices button)
   Go to the previous invoice created, select the payment plan record and clic on Modify Payment Plan
   Check the process shows the error message 'The invoice is already included in a Remittance.'

(0104172)
hgbot   
2018-04-30 16:54   
Repository: erp/mods/org.openbravo.module.remittance
Changeset: c369ffae37fc26a073a460ffb7faa03364a381b9
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Tue Apr 10 17:12:25 2018 -0400
URL: http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/c369ffae37fc26a073a460ffb7faa03364a381b9 [^]

Fixes issue 38288: Sales Invoice Payment Plan Due date can be modified when
related payment is in Remitted status

Remittances only accept documents with due date prior to their expected date,
the due date of a payment plan can be increased because the process doesn't
properly check the that condition

A new hook is created to check the edited payment schedule lines and avoid validation
issues

---
A src/org/openbravo/module/remittance/hooks/REMModifyPaymentPlanHook.java
---
(0104173)
hgbot   
2018-04-30 16:54   
Repository: erp/mods/org.openbravo.module.remittance
Changeset: 5264700466418e90fa9e084b073f7488f3bb9931
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Apr 23 09:10:35 2018 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/5264700466418e90fa9e084b073f7488f3bb9931 [^]

Related to Issue 38288. Changed exception thrown.

If the Invoice is already included in a Remittance, or if it has
a Payment included in a Remittance, then an exception is thrown

---
M src/org/openbravo/module/remittance/hooks/REMModifyPaymentPlanHook.java
---
(0104174)
hgbot   
2018-04-30 16:54   
Repository: erp/mods/org.openbravo.module.remittance
Changeset: d80ccca9457b27fb19445d5c1e1b61772c90ab9c
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Apr 30 11:09:03 2018 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/d80ccca9457b27fb19445d5c1e1b61772c90ab9c [^]

Related to issue 38288. Removed @ signs from message since
are no longer needed with OBMessageUtils API

---
M src/org/openbravo/module/remittance/hooks/REMModifyPaymentPlanHook.java
---
(0104175)
hgbot   
2018-04-30 16:54   
Repository: erp/mods/org.openbravo.module.remittance
Changeset: 94dd91908282844e5e83fe9ef616d56d7894a31e
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Apr 30 16:53:41 2018 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/94dd91908282844e5e83fe9ef616d56d7894a31e [^]

Related to issue 38288. Updated Module version

---
M src-db/database/sourcedata/AD_MODULE.xml
---
(0104176)
hgbot   
2018-04-30 16:54   
Repository: erp/devel/pi
Changeset: bbec1f5c73008c05f8c22c03cf61addf6ea0ce80
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Tue Apr 10 17:12:31 2018 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/bbec1f5c73008c05f8c22c03cf61addf6ea0ce80 [^]

Fixes issue 38288: Sales Invoice Payment Plan Due date can be modified when
related payment is in Remitted status

Remittances only accept documents with due date prior to their expected date,
the due date of a payment plan can be increased because the process doesn't
properly check the that condition

A new hook is created to check the edited payment schedule lines and avoid validation
issues

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java
A modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ModifyPaymentPlanHook.java
A modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanHookCaller.java
---
(0104177)
hgbot   
2018-04-30 16:54   
Repository: erp/devel/pi
Changeset: f3cb10dc3207e8f3d6f91ce56e8d98669ae2b7b7
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Apr 23 09:17:37 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f3cb10dc3207e8f3d6f91ce56e8d98669ae2b7b7 [^]

Related to Issue 38288. Code review changes

* Moved validation of Payment Plan before performing any logic
* Changed logic to retrieve messages, it now usses API from OBMessageUtils

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java
---
(0104178)
dmiguelez   
2018-04-30 16:56   
Code Review + Testing Ok.

Needs to be updated to include a dependency for 18Q3
(0104411)
hudsonbot   
2018-05-09 19:29   
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/2be7d3efe606 [^]
Maturity status: Test
(0104412)
hudsonbot   
2018-05-09 19:29   
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/2be7d3efe606 [^]
Maturity status: Test