I understand. However, functionality should never be changed in order to make something conform to a given style.

In this case, if object.harassExecuteOld returns false I want the entire function to return false. 

If I made the change you suggested. Then when object.harassExecuteOld returns false, the function will still attempt to cast spells, which could make the entire function return true. 

Clearly that changes how the function works, which is bad.