Fixed wrong mission on non-allied guards

[davidpwbrown]

Summary

SUMMARY: Bugfixes "Fixes all refugee centre NPCS becoming allies

Purpose of change

Fixes #29063

Describe the solution

The problem wasnt actually the shout, all refugees became followers as soon as they were close enough.
This was because of a change in the order of NPC_MISSION enums - they were all set to mission_guard_ally instead of mission_guard. And the game logic for picking up who is a follower nearby checks their mission type.
( The issue was seemingly logged before the change to enums was merged - and it seems there was a merge conflict somewhere, perhaps in what mlangsdorf and I were doing - as I very deliberately was careful with the order of mission enums - as I thought there was a risk of this happening )

This changes the order to match what all the NPCS mission type is in JSON, existing NPC allied guards may now have problems, I cant think of a way to validate that, and it's a lesser problem than the current bug.

Not many people would have saved in the window for this bug with a follower guarding nearby, thereby serializing the followers mission, so hopefully it wont screw up too many savegames.

Describe alternatives you've considered

N/A

Additional context

N/A

[kevingranade]

This pull request has been mentioned on Cataclysm: Dark Days Ahead. There might be relevant details there:

https://discourse.cataclysmdda.org/t/major-game-breaking-bug/19430/9