Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035252Openbravo ERP07. Sales managementpublic2017-02-13 22:112017-03-23 16:52
Reportermarkmm82 
Assigned Tocollazoandy4 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version3.0PR17Q2 
Merge Request Status
Review Assigned Tomarkmm82
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-04-28
Regression introduced in release3.0PR16Q3
Regression introduced by commithttp://code.openbravo.com/erp/devel/pi/rev/b115497583a1 [^]
Triggers an Emergency PackNo
Summary0035252: Selected General Ledger filter value changes when pagination is executed in the Journal Entries report
DescriptionSelected General Ledger filter value changes when pagination is executed in the Journal Entries report. It seems that the report always reload the GL field and use the first value in the combo to execute the report causing be impossible to see more entries than retrieved in the first page if the selected General Ledger is not the first one in the General Ledger filter.
Steps To Reproduce1. Go to Journal Entries Report.
2. Select a valid From and To dates.
3. Set Organization filter as "F&B España S.A"
4. Set General Ledger filter as "F&B International Group US/A/US Dollar".
5. Run the report, and notice:
   Organization filter = "F&B España S.A"
   General Ledger filter as "F&B International Group US/A/US Dollar"
   In the grid of resulting values you can see entries related to US Dollar.
   At top, you will see that a yellow arrow with hint "Go to Next Record Range" has been enabled.
6. Go to the next page and notice:
   Organization filter = "F&B España S.A"
   General Ledger filter has changed to "F&B España, S.A. US/A/Euro". This is wrong.
   In the grid of resulting values you can see entries related to Euro instead of Dollar. This is wrong.
   Now two arrows are enabled (to go to the next page, and to go back to the previous page)
7. If you go to the previous page you should see the same than obtained in step 5 but you get other results related to "F&B España, S.A. US/A/Euro" GL.
   
Proposed SolutionPass the General Ledger filter as parameter when pagination is executed, just like is currently doing with the Organization filter.
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00352763.0PR17Q1 closed collazoandy4 Selected General Ledger filter value changes when pagination is executed in the Journal Entries report 
depends on backport 00352773.0PR16Q4.3 closed collazoandy4 Selected General Ledger filter value changes when pagination is executed in the Journal Entries report 
depends on backport 00352783.0PR16Q3.6 closed collazoandy4 Selected General Ledger filter value changes when pagination is executed in the Journal Entries report 
caused by defect 00325353.0PR16Q3 closed markmm82 Report's currency and report's general ledger should be defaulted to organization's currency and organization's general ledger 
related to defect 0035266 closed collazoandy4 "General Ledger" filter is removed when pressing "next page" in General Ledger Report HTML 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-02-13 22:11markmm82New Issue
2017-02-13 22:11markmm82Assigned To => Triage Finance
2017-02-13 22:11markmm82Modules => Core
2017-02-13 22:11markmm82Triggers an Emergency Pack => No
2017-02-13 22:16markmm82Description Updatedbug_revision_view_page.php?rev_id=14572#r14572
2017-02-14 17:55vmromanosRelationship addedrelated to 0035266
2017-02-14 22:24markmm82Assigned ToTriage Finance => collazoandy4
2017-02-15 16:45markmm82Relationship addedcaused by 0032535
2017-02-15 16:48markmm82Statusnew => acknowledged
2017-02-15 16:56markmm82Statusacknowledged => scheduled
2017-02-15 17:43markmm82Review Assigned To => markmm82
2017-02-15 17:43markmm82Resolution time => 1487286000
2017-02-15 17:43markmm82Regression level => Production - Confirmed Stable
2017-02-15 17:43markmm82Regression date => 2016-04-28
2017-02-15 17:43markmm82Regression introduced in release => 3.0PR16Q3
2017-02-15 17:43markmm82Regression introduced by commit => http://code.openbravo.com/erp/devel/pi/rev/b115497583a1 [^]
2017-02-20 15:39hgbotCheckin
2017-02-20 15:39hgbotNote Added: 0094477
2017-02-20 15:39hgbotStatusscheduled => resolved
2017-02-20 15:39hgbotResolutionopen => fixed
2017-02-20 15:39hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/a01dd96ef7eb8e6d9fe2841fae4eddd4c5f82437 [^]
2017-02-20 15:54markmm82Note Added: 0094483
2017-02-20 15:54markmm82Statusresolved => closed
2017-02-20 15:54markmm82Fixed in Version => 3.0PR17Q2
2017-03-15 20:19hudsonbotCheckin
2017-03-15 20:19hudsonbotNote Added: 0095198

Notes
(0094477)
hgbot   
2017-02-20 15:39   
Repository: erp/devel/pi
Changeset: a01dd96ef7eb8e6d9fe2841fae4eddd4c5f82437
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Thu Feb 16 11:46:12 2017 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/a01dd96ef7eb8e6d9fe2841fae4eddd4c5f82437 [^]

Fixes issue 35252: General Ledger filter changes with pagination in JE Report

Selected General Ledger filter value changes when pagination is executed in the
Journal Entries report

The General Ledger filter is not kept in the session to make use in the pagination.
For the solution, was saved the General Ledger Filter in the session every time
the search is executed (FIND action). By default, if there is any GL in the
session, the GL field will use the defaulted for the selected org. When paginating
always is used the account schema id stored in the session.

---
M src/org/openbravo/erpCommon/ad_reports/ReportGeneralLedgerJournal.java
---
(0094483)
markmm82   
2017-02-20 15:54   
Code review + testing OK
(0095198)
hudsonbot   
2017-03-15 20:19   
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/54e102bef53e [^]
Maturity status: Test