Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build #8949 broke command: yell (C) to make all NPCs follow #30457

Closed
Daxtgnomoria opened this issue May 12, 2019 · 2 comments · Fixed by #30458
Closed

Build #8949 broke command: yell (C) to make all NPCs follow #30457

Daxtgnomoria opened this issue May 12, 2019 · 2 comments · Fixed by #30458
Assignees
Labels
<Bug> This needs to be fixed NPC / Factions NPCs, AI, Speech, Factions, Ownership

Comments

@Daxtgnomoria
Copy link

Daxtgnomoria commented May 12, 2019

Describe the bug
Before Build #8949 you could make all NPCs follow you at the same time, using the command yell (c) and selecting the option to tell everyone to follow at the same time. After the refine of the menu, this command is acknowledged by all NPCs, but they just stand still and remain in Guard position. Using the same menu (yell) to ask them individually to follow works fine, but not for everyone at the same time. I tested it with 3 NPCs.

To Reproduce
Steps to reproduce the behavior:

  1. Spawn more than one NPC.
  2. Recruit them.
  3. Use command "yell" (C) and select option (f) "Tell someone to follow" and in the next menu select option "Everyone".
  4. Walk away.

Expected behavior
All NPCs should leave their guard position and start following the player.

Versions and configuration(please complete the following information):

  • OS: Windows 10
  • Game Version: 0.D-2863-gca633a0
  • Graphics version: Tiles - MXotto+
  • Mods loaded: dda, disable NPC needs

Additional context
Link to TEST savegame with 3 NPCs already recruited and in guard position, for tests.
http://s000.tinyupload.com/?file_id=31458396185137637669

@esotericist
Copy link
Contributor

Damn, I swear I tested the hell out of that. Let me look into it

@esotericist esotericist self-assigned this May 12, 2019
@esotericist
Copy link
Contributor

... Found the problem, and I'm stupid. I'll have a PR for this shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants