Shouldn't this function just return `item.active` instead of also setting `$scope.currentItem`?