Change order of events in player::fire_gun

[mugling]

The order of events in player::fire_gun is significant. When the last unit of ammunition is consumed item::ammo_consume() should unset curammo.

The order of events is updated to:

1. Select target
2. Chance to misfire (player::handle_gun_damage)
3. Update recoil
4. Take shot (projectile_attack)
5. Play sound effects
6. Use up ammo (item::ammo_consume)
7. Drain UPS (should be a function)
8. Train (player::practice)

This is the minimal required changes to hopefully fix #14524.