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

Mercenary doesn't guard the palisade #217

Merged
merged 4 commits into from Apr 5, 2021
Merged

Mercenary doesn't guard the palisade #217

merged 4 commits into from Apr 5, 2021

Conversation

AmProsius
Copy link
Owner

Describe the bug
A Mercenary is supposed to guard the palisade during the afternoon, but a typo in the waypoint name prevents him from doing so.

Expected behavior
A Mercenary now correctly guards the palisade during the afternoon.

Additional context
Bug and fix provided by NikX94.

@AmProsius AmProsius added the validation: required This issue needs validation from one of the validators. label Mar 28, 2021
@AmProsius
Copy link
Owner Author

FUNC VOID Rtn_FMCstart_750 ()
{
TA_GuardPalisade (01,00,13,00, "FMC_PATH19");
TA_GuardPalisade (13,00,01,00, "FMC_Path19");
};

changed to

FUNC VOID Rtn_FMCstart_750 ()
{
    TA_GuardPalisade    (01,00,13,00, "FMC_PATH19");
    TA_GuardPalisade    (13,00,01,00, "FMC_PATH19");
};

@AmProsius
Copy link
Owner Author

I'm not sure if this really is a bug we have to fix. From my understanding, if an NPC doesn't find the next waypoint, he will continue his current routine (which in this case is the exact same action he is supposed to do). Can you confirm this behavior, @szapp?

@AmProsius AmProsius added the provided fix This issue has a fix provided in the comments. label Mar 28, 2021
@N1kX94
Copy link

N1kX94 commented Mar 31, 2021

The error will be that the NPC will be located at the old point from the previous routine, but will not do anything. There is also an error in the fact that the NPC can disappear from the world for this time and return back when the point is available.
In Gothic 2, there are NPCs that disappear due to errors in the routine.

This fix will only improve the atmosphere of the game, nothing more.

@szapp szapp added compatibility: easy This issue is easy to make compatible. impl: replace assign/push str This issue requires replacing pushed strings or string assignments in the scripts. type: session fix The fix for this issues is persistent across a session. labels Mar 31, 2021
@szapp szapp added this to NPC daily routine in Fix templates Mar 31, 2021
@AmProsius AmProsius added this to To Do in v1.1.0 via automation Mar 31, 2021
@AmProsius AmProsius self-assigned this Mar 31, 2021
@AmProsius AmProsius added this to the v1.1.0 milestone Mar 31, 2021
@AmProsius AmProsius requested a review from szapp April 1, 2021 10:17
@AmProsius AmProsius removed their assignment Apr 1, 2021
@szapp szapp moved this from To Do to In Progress in v1.1.0 Apr 1, 2021
Copy link
Collaborator

@szapp szapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test passes, but I have suggestion for the changelog:

CHANGELOG.md Outdated Show resolved Hide resolved
docs/changelog_de.md Outdated Show resolved Hide resolved
@AmProsius AmProsius requested a review from szapp April 3, 2021 12:42
@szapp szapp merged commit fa1560d into master Apr 5, 2021
v1.1.0 automation moved this from In Progress to Done Apr 5, 2021
@szapp szapp deleted the bug217 branch April 5, 2021 13:05
szapp added a commit that referenced this pull request Apr 19, 2021
@szapp szapp removed the validation: required This issue needs validation from one of the validators. label May 14, 2021
szapp added a commit that referenced this pull request Feb 6, 2022
szapp added a commit that referenced this pull request Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility: easy This issue is easy to make compatible. impl: replace assign/push str This issue requires replacing pushed strings or string assignments in the scripts. provided fix This issue has a fix provided in the comments. type: session fix The fix for this issues is persistent across a session.
Projects
Fix templates
Change daily routine waypoint
v1.1.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants