Basic Code Cleanup

Help refactor
this
JavaScript
function
:
function
processData
(
data
)
{
var
result
=
[
]
;
for
(
var
i
=
0
;
i
<
data
.
length
;
i
++
)
{
if
(
data
[i]
.
active
==
true
)
{
if
(
data
[i]
.
type
==
'user'
)
{
var
item
=
{
id
:
data
[i]
.
id
,
name
:
data
[i]
.
name
,
role
:
data
[i]
.
role
||
'guest'
}
;
result
.
push
(
item
)
;
}
}
}
return
result
;
}
Focus on
:
1.
Modern JavaScript features
2.
Code readability
3.
Performance
4.
Error handling