Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0019111
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Othersminorhave not tried2011-11-18 12:472011-12-13 16:44
ReporterjonalegriaesarteView Statuspublic 
Assigned Toalostale 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchpiFixed in SCM revision27caa9812cd1
ProjectionnoneETAnoneTarget Version3.0MP7
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0019111: Filters in query widgets are not working.

DescriptionFilters in query widgets are not working.

Steps To Reproduce- In pi instance add (for example) Invoices to collect widget.
- In minimized view, try filtering "Document Number" column -> Nothing is filtered.
TagsNo tags attached.
Attached Filespng file icon QueryWidgetFilter.png [^] (35,820 bytes) 2011-11-18 12:47

- Relationships Relation Graph ] Dependency Graph ]
has duplicate defect 0019056 closedalostale Query list widget filters do not work on workspace 

-  Notes
(0043595)
alostale (manager)
2011-11-30 16:50

By design http://wiki.openbravo.com/wiki/Projects/QueryList_Widget/Functional_Specs [^] only maximized view should allow to filter.

Here the issue, is the filter is shown in workspace view when the widget is added. Note filtering disapears on next login.
(0043631)
alostale (manager)
2011-12-01 10:53

Test plan:
-Add a new query list widget and check no filter is shown
-Relogin with that user check filter is not present
-Maximize, check it is possible to filter
Regression risk: none, modifying a property in query list widget that shouldn't affect other pieces.
(0043772)
hgbot (developer)
2011-12-12 08:31

Repository: erp/devel/pi
Changeset: 27caa9812cd15c05dd797765d7ab836cda1d0d0f
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Dec 01 10:54:12 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/27caa9812cd15c05dd797765d7ab836cda1d0d0f [^]

fixed bug 19111: Filters in query widgets shouldn't be shown in workspace

---
M modules/org.openbravo.client.querylist/web/org.openbravo.client.querylist/js/ob-querylist-widget.js
---
(0043794)
iperdomo (reporter)
2011-12-12 12:00

Code review:

For clarity/maintainability purposes, I would change the line:

showFilter = this.viewMode === 'maximized';

to:

showFilter = (this.viewMode === 'maximized');


In this way someone reading the code, will identify quickly that the is testing viewMode against a value and the result is assigned to a variable.


https://code.openbravo.com/erp/devel/pi/rev [^]
/27caa9812cd15c05dd797765d7ab836cda1d0d0f#l1.9
(0043805)
hgbot (developer)
2011-12-12 16:36

Repository: erp/devel/pi
Changeset: fd1af76deeeef4cffe35d9099519dd551c786779
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Mon Dec 12 16:35:19 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/fd1af76deeeef4cffe35d9099519dd551c786779 [^]

related to issue 19111: Applied code review suggestion

---
M modules/org.openbravo.client.querylist/web/org.openbravo.client.querylist/js/ob-querylist-widget.js
---
(0043806)
iperdomo (reporter)
2011-12-12 16:43

Tested on pi @ rev 516b2ef77059.

Using finance role, i removed all querylist widgets
* Added invoice to collect and checked that no filter is present in the workspace
* log-out/log-in and checked again that no filter is present
* maximize the widget and now the filters are present
(0043839)
hudsonbot (developer)
2011-12-13 16:43

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/8a06361cc265 [^]

Maturity status: Test
(0043848)
hudsonbot (developer)
2011-12-13 16:44

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/8a06361cc265 [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2011-11-18 12:47 jonalegriaesarte New Issue
2011-11-18 12:47 jonalegriaesarte Assigned To => alostale
2011-11-18 12:47 jonalegriaesarte File Added: QueryWidgetFilter.png
2011-11-18 12:47 jonalegriaesarte Modules => Core
2011-11-21 08:46 alostale Status new => scheduled
2011-11-21 08:46 alostale fix_in_branch => pi
2011-11-21 08:47 alostale Relationship added has duplicate 0019056
2011-11-30 16:50 alostale Note Added: 0043595
2011-12-01 10:53 alostale Note Added: 0043631
2011-12-12 08:31 hgbot Checkin
2011-12-12 08:31 hgbot Note Added: 0043772
2011-12-12 08:31 hgbot Status scheduled => resolved
2011-12-12 08:31 hgbot Resolution open => fixed
2011-12-12 08:31 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/27caa9812cd15c05dd797765d7ab836cda1d0d0f [^]
2011-12-12 12:00 iperdomo Note Added: 0043794
2011-12-12 16:36 hgbot Checkin
2011-12-12 16:36 hgbot Note Added: 0043805
2011-12-12 16:43 iperdomo Note Added: 0043806
2011-12-12 16:43 iperdomo Status resolved => closed
2011-12-13 16:43 hudsonbot Checkin
2011-12-13 16:43 hudsonbot Note Added: 0043839
2011-12-13 16:44 hudsonbot Checkin
2011-12-13 16:44 hudsonbot Note Added: 0043848


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker