Openbravo Issue Tracking System - Modules
View Issue Details
0039903ModulesRemittancepublic2019-01-09 17:392019-02-04 09:54
ngarcia 
markmm82 
urgentmajoralways
closedfixed 
5
 
 
Sandrahuguet
0039903: Duplicated records shown in the last records of Settle/Protest Remittance if some lines have same payments and scroll is shown
Duplicated records shown in the last records of Settle/Protest Remittance if some lines have same payments and scroll is shown

The getRemittanceLine query is launched with an offset, this offset is calculated with the remittance lines but the lines shown in the Settle/Protest Remittance window are the payments included in the remittance lines that could be less
Having a remittance with more than 28 lines and same payments on it lines. The problem can be also reproduced with less records by reducing the size of the browser

As group admin role:
   Go to Settle/Protest Remittance
   Select the previously created Remittance
   Click on Search
   Move the scroll bar to the end and check some records are duplicated in the end
   The number in the right side is also wrong as it is showing more records than the available (for example: 111-138 / 135)
   If you click in one of the last records the original record will be selected
No tags attached.
Issue History
2019-01-09 17:39ngarciaNew Issue
2019-01-09 17:39ngarciaAssigned To => Triage Finance
2019-01-09 17:39ngarciaResolution time => 1548802800
2019-01-09 17:39ngarciaIssue Monitored: networkb
2019-01-09 18:58PracticsIssue Monitored: Practics
2019-01-10 08:22SandrahuguetAssigned ToTriage Finance => markmm82
2019-01-11 18:01markmm82Statusnew => scheduled
2019-01-21 02:42markmm82Note Added: 0109028
2019-01-29 15:25markmm82Note Added: 0109311
2019-01-31 17:08SandrahuguetRelationship addedrelated to 0040091
2019-01-31 17:19SandrahuguetRelationship deletedrelated to 0040091
2019-02-04 09:51hgbotCheckin
2019-02-04 09:51hgbotNote Added: 0109511
2019-02-04 09:51hgbotStatusscheduled => resolved
2019-02-04 09:51hgbotResolutionopen => fixed
2019-02-04 09:51hgbotFixed in SCM revision => http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/f21d7ee630cd2c938f3a356c87d74ed58a4c28e3 [^]
2019-02-04 09:54SandrahuguetReview Assigned To => Sandrahuguet
2019-02-04 09:54SandrahuguetNote Added: 0109512
2019-02-04 09:54SandrahuguetStatusresolved => closed

Notes
(0109028)
markmm82   
2019-01-21 02:42   
Test Plan:
Create some invoices and payments to have a remittance with more than 28 lines and same payments on it lines.

As group admin role:
   Go to Settle/Protest Remittance
   Select the previously created Remittance
   Click on Search
   Move the scroll bar to the end and check there aren't duplicated records.
   Also, the pagination is right when you scroll the grid
(0109311)
markmm82   
2019-01-29 15:25   
Issue reproduced by the following steps:
Apply Remittance dataset
As Group Admin:
Create 58 sales invoice headers as this:
  Organization = F&B España - Región Norte
  Business Partner = Alimentos y Supermercados, S.A
  Payment Method = Remittance
  Invoice Date = Today (for example: 29-01-2019)
Add one line on each with any product and complete.

Create a Remittance
  Organization = F&B España - Región Norte
  Document Type = Remittance
  Transaction Date = Today (for example: 29-01-2019)
  Expected Date = A date after the payment scheduled for the invoice (for example: 29-05-2019)
  Name = RemitTest
  Financial Account = Cuenta de Banco - EUR
Save it

Execute Select Orders or Invoices process.
Notice the 58 lines belonging to the 58 invoices are shown in the grid.
Select all the lines and Process

Process the remittance.

Go to Settle/Protest Remittance, filter by:
  Organization = F&B España - Región Norte
  You can select the previously created Remittance or leave the filter emptu if you were working in a clean environment without any other remittances.

   Move the scroll bar to the end and check the last record is duplicated in the end
   Also notice that should be 58 lines but the line number is 59.
   The number in the right side is also wrong as it is showing more records than the available (for example: 42-69 / 59)
   If you click in the last record the original record will be selected
(0109511)
hgbot   
2019-02-04 09:51   
Repository: erp/mods/org.openbravo.module.remittance
Changeset: f21d7ee630cd2c938f3a356c87d74ed58a4c28e3
Author: Mark <markmm82 <at> gmail.com>
Date: Sun Jan 20 18:26:26 2019 -0500
URL: http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/f21d7ee630cd2c938f3a356c87d74ed58a4c28e3 [^]

Fixes issue 39903:Avoid duplicated records in the last records of Settle/Protest
Remittance if some lines have same payments and scroll is shown

Avoid to return the last record when page size is 0. This way it is not considered the last record
twice if.
Also was fixed the pagination in the report to get the less vaue between the pagination
last record and the total of records, taking into account that it is possible records
with the same payment.

---
M src/org/openbravo/module/remittance/ad_forms/CancelReturnRemittance.html
M src/org/openbravo/module/remittance/ad_forms/CancelReturnRemittance.java
---
(0109512)
Sandrahuguet   
2019-02-04 09:54   
Code review + testing OK