Incorrect value for is_active

[pocallaghan-function]

Pretty sure this doesn't do what it was intended to. The bool cast applies to the left hand argument as opposed to the result of the comparison. In Magento a product has a status of 2 when it is explicitly disabled. In that instance this line essentially becomes:

$tmp['is_active'] = (bool) 2 == 1;

PHP will bool cast the 2 to be true in which case it's:

$tmp['is_active'] = true == 1;

Which is true.

The result of this is that every product that has a status explicitly set, regardless of enabled or disabled will return is_active = true.