Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0025033Openbravo ERPB. User interfacepublic2013-10-28 13:502013-10-30 20:47
Reporteregoitz 
Assigned ToAugustoMauch 
PrioritynormalSeveritycriticalReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0MP29 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2013-09-11
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/290c9b12a211 [^]
Triggers an Emergency PackNo
Summary0025033: Removing the filter show refresh the window if the window was openend with the default filter clause
DescriptionRemoving the filter show refresh the window if the window was openend with the default filter clause
Steps To Reproduce-Access to saler order window and remove the defult filter clause by clicking on the funnel
-On sales order window create 10 orders on draft
-Apply filter to shon only 4 of that rows
-Save the filtering as a view and set us default.
-Logout and login
-Access to the window and see that only the 4 rows are shown due to the defult view
-Remove the filter by clicking the funnel
-The window is not refreshed so the default filter clause is still applied so only 10 rows are shown
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00332043.0PR16Q3 closed caristu Javascript error appears in console after opening a window with a saved view 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-10-28 13:50egoitzNew Issue
2013-10-28 13:50egoitzAssigned To => AugustoMauch
2013-10-28 13:50egoitzModules => Core
2013-10-28 13:50egoitzTriggers an Emergency Pack => No
2013-10-28 13:54egoitzRegression level => Production - QA Approved
2013-10-29 11:06egoitzSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=5165#r5165
2013-10-29 13:05AugustoMauchIssue Monitored: alostale
2013-10-29 13:06AugustoMauchReview Assigned To => alostale
2013-10-29 13:06AugustoMauchSeveritymajor => critical
2013-10-29 13:10AugustoMauchRegression date => 2013-09-11
2013-10-29 13:10AugustoMauchRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/290c9b12a211 [^]
2013-10-29 13:11hgbotCheckin
2013-10-29 13:11hgbotNote Added: 0061935
2013-10-29 13:11hgbotStatusnew => resolved
2013-10-29 13:11hgbotResolutionopen => fixed
2013-10-29 13:11hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ab075f83c2f8cc8c198d8b2541ef90de3134fbbc [^]
2013-10-30 11:02alostaleNote Added: 0061967
2013-10-30 11:02alostaleStatusresolved => closed
2013-10-30 11:02alostaleFixed in Version => 3.0MP29
2013-10-30 20:47hudsonbotCheckin
2013-10-30 20:47hudsonbotNote Added: 0061981
2016-06-13 11:34caristuRelationship addedrelated to 0033204

Notes
(0061935)
hgbot   
2013-10-29 13:11   
Repository: erp/devel/pi
Changeset: ab075f83c2f8cc8c198d8b2541ef90de3134fbbc
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Tue Oct 29 13:05:23 2013 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ab075f83c2f8cc8c198d8b2541ef90de3134fbbc [^]

Fixes issue 25033: Problems with personalization and grid adaptive filtering

There was a problem when a saved view with no filter clause is applied over a view that had an active filter clause.

Consider the following sequence of events:
- A window (i.e. Sales Order) is opened
- An initial request to the datasource before the personalized view is applied. Suppose the datasource returns fewer than 100 records
- The personalized view is applied. As there is fewer than 100 records, the filtering is done locally. This is wrong, because the initial dataset was built over a filterClause, and the personalize view that is being applied does not have a filter clause. In this case the filtering should not be done locally.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0061967)
alostale   
2013-10-30 11:02   
code reviewed + verified
(0061981)
hudsonbot   
2013-10-30 20:47   
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/80f375c041fe [^]

Maturity status: Test