Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0014078Openbravo ERPZ. Otherspublic2010-07-28 13:552010-12-09 16:04
iperdomo 
iperdomo 
urgentminoralways
closedfixed 
202.6.30-gentoo-r5
2.50 
 
User Interface Selector
No
0014078: Fix JSLint suggestion on making functions in a loop
jslint:./web/org.openbravo.userinterface.selector/js/ob_selector_widget.js:490:11: Be careful when making functions within a loop. Consider putting the function in a closure.
* JSLint the ob_selector_widget.js file
No tags attached.
blocks feature request 0013458 closed iperdomo It should be possible to assignes default values for fields on selectors 
Issue History
2010-07-28 13:55iperdomoNew Issue
2010-07-28 13:55iperdomoAssigned To => iperdomo
2010-07-28 13:56iperdomoStatusnew => scheduled
2010-07-28 13:56iperdomofix_in_branch => pi
2010-07-28 13:56iperdomoRelationship addedblocks 0013458
2010-07-28 14:07hgbotCheckin
2010-07-28 14:07hgbotNote Added: 0029645
2010-07-28 14:07hgbotStatusscheduled => resolved
2010-07-28 14:07hgbotResolutionopen => fixed
2010-07-28 14:07hgbotFixed in SCM revision => http://code.openbravo.com/erp/mods/org.openbravo.userinterface.selector/rev/bec25112971bd8a735da96e18e25bd5bc70562ae [^]
2010-08-25 19:01shuehnerNote Added: 0030327
2010-08-25 19:01shuehnerStatusresolved => closed
2010-12-09 16:04anonymoussf_bug_id0 => 3133336

Notes
(0029645)
hgbot   
2010-07-28 14:07   
Repository: erp/mods/org.openbravo.userinterface.selector
Changeset: bec25112971bd8a735da96e18e25bd5bc70562ae
Author: Iván Perdomo <ivan.perdomo <at> openbravo.com>
Date: Wed Jul 28 14:05:48 2010 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.userinterface.selector/rev/bec25112971bd8a735da96e18e25bd5bc70562ae [^]

Fixes issue 14078: Moved the function to a closure

---
M web/org.openbravo.userinterface.selector/js/ob_selector_widget.js
---
(0030327)
shuehner   
2010-08-25 19:01   
Tested that the different button on top of the selector (remove filter...) are still working fine with the modified function definition.