Skip to content

[Hotfix] Fix Quest Ownership Edge Case#4977

Merged
Akkadius merged 2 commits intomasterfrom
hotfix/quest_ownership
Aug 3, 2025
Merged

[Hotfix] Fix Quest Ownership Edge Case#4977
Akkadius merged 2 commits intomasterfrom
hotfix/quest_ownership

Conversation

@Kinglykrab
Copy link
Copy Markdown
Contributor

Description

  • Fixes an issue where ownership was not applied properly.

Type of change

  • Bug fix

Testing

Before

[QuestErrors] Zone [sebilis] Script Error | Package [qst_npc_640] Event [EVENT_SPAWN] Error [Perl runtime error: Can't call method "GetOwnerID" on an undefined value at ./quests/global/640.pl line 2.]
[QuestErrors] Zone [shadeweaver] Script Error | Package [qst_global_npc] Event [EVENT_COMBAT] Error [Perl runtime error: Can't call method "GetID" on an undefined value at ./quests/global/global_npc.pl line 177.]

After

image

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@Akkadius Akkadius merged commit d142e1c into master Aug 3, 2025
0 of 2 checks passed
@Akkadius Akkadius deleted the hotfix/quest_ownership branch August 3, 2025 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants