Reply to thread

Looking at this code as it stands now, I don't think there's a situation where the normal flow would lead to returning anything but the active upgrade. Any error that occurs from the upgrade insert is set to throw an exception and these were the only situations that could return false (if throwing was disabled). Any sort of DB exception would throw an exception regardless.

So I think this should be resolved as is in 2.0.12.