The backtrace shows that the abort you are seeing is not from the PLATFORM_ASSERT.
Anyway since the assert and the new if are testing different things they can be in either order. Also the assert is most likely compiled out in most builds.
Best to put exactly what Neil did to avoid merge issues later.