Sutty/trabajo-afectivo
5
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Removed not needed forEach(). IE8 is not supporting this.

Browse source
This commit is contained in:
Martin Edenhofer 2012-11-29 20:47:39 +01:00
parent dd82093ca0
commit 28b5bb6cf7
2 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
app/assets/javascripts/app/controllers/_application_controller_form.js.coffee
View file

@ -528,7 +528,7 @@ class App.ControllerForm extends App.Controller
# build options list # build options list
_buildOptionList: (list, attribute) -> _buildOptionList: (list, attribute) ->
list.forEach( (item) => for item in list
# if active or if active doesn't exist # if active or if active doesn't exist
if item.active || !( 'active' of item ) if item.active || !( 'active' of item )
@ -542,7 +542,6 @@ class App.ControllerForm extends App.Controller
value: item.id, value: item.id,
note: item.note, note: item.note,
} }
)
# execute filter # execute filter
_filterOption: (attribute) -> _filterOption: (attribute) ->

8
app/assets/javascripts/app/controllers/navigation.js.coffee
View file

@ -157,10 +157,9 @@ class App.Navigation extends App.Controller
if !roles && item.role if !roles && item.role
match = _.include( item.role, 'Anybody' ) match = _.include( item.role, 'Anybody' )
if roles if roles
roles.forEach( (role) => for role in roles
if !match if !match
match = _.include(item.role, role.name) match = _.include( item.role, role.name )
)
if match if match
level1.push item level1.push item
@ -178,10 +177,9 @@ class App.Navigation extends App.Controller
if !roles if !roles
match = _.include( itemSub.role, 'Anybody' ) match = _.include( itemSub.role, 'Anybody' )
if roles if roles
roles.forEach( (role) => for role in roles
if !match if !match
match = _.include( itemSub.role, role.name ) match = _.include( itemSub.role, role.name )
)
if match if match
dropdown[ item.parent ].push itemSub dropdown[ item.parent ].push itemSub