[TF2] Fix #618 causing money to be aggressively collected in the playable area

[mtxfellen]

Description

The PR #618, merged in 52e568b happened to introduce a bug in CCurrencyPack::ComeToRest() that this PR aims to address:

• Use PointIsWithin() instead of WorldSpaceSurroundingBounds() so that complex trigger shapes are accounted for.
  • We received some reports about money being auto-collected on some maps in large, perfectly accessible locations.

This PR also fixes some other minor bugs in this method:

• Add missing early returns that could've rarely allowed currency to be auto-collected multiple times.
• Prevent money being collected in disabled respawn rooms.
• Move m_bDistributed check so that red money can be auto-collected if it drops outside the playable space.
• Normalise GetCollisionOrigin() and GetAbsOrigin() usage as GetAbsOrigin().
  • They are the same for this entity.

[FlaminSarge]

If the respawn room gets disabled before the money comes to rest, will this result in the money not autocollecting? Some maps may disable certain respawn rooms for robots mid-game but may not make those areas walkable for players.

[mtxfellen]

That would indeed happen, but I'm operating on the assumption that there's not really much reason to disable spawnroom entities in MvM except for making an area walkable and using RecomputeBlockers, given that robots won't attempt to hotswap their loadouts and their actual spawnpoints will continue to function as the mission maker directs independent of the spawnroom's status. Though of course if there's any maps this change would break, it would be worth reviewing this part again.