Skip to content

fix: spawn all npcs in already loaded chunks on start#1255

Merged
derklaro merged 5 commits intonightlyfrom
fix/npc-spawns
Jun 15, 2023
Merged

fix: spawn all npcs in already loaded chunks on start#1255
derklaro merged 5 commits intonightlyfrom
fix/npc-spawns

Conversation

@0utplay
Copy link
Member

@0utplay 0utplay commented Jun 14, 2023

Motivation

When a npc is located in the spawn chunks of the world the npc won't get spawned. This is caused by the fact that the chunk load event is called before our plugin is started.

Modification

Spawned all npcs on start that are located in already loaded chunks

Result

No non existing npcs

@0utplay 0utplay added v: 4.X This pull should be included in the 4.0 release t: fix A pull request introducing a fix for a bug. in: module An issue/pull request releated to one of the internal modules labels Jun 14, 2023
@0utplay 0utplay added this to the 4.0.0-RC9 milestone Jun 14, 2023
@0utplay 0utplay requested a review from derklaro June 14, 2023 21:08
@0utplay 0utplay self-assigned this Jun 14, 2023
@0utplay 0utplay marked this pull request as ready for review June 14, 2023 21:08
@github-actions
Copy link

Test Results

  47 files  ±0    47 suites  ±0   1m 25s ⏱️ +16s
322 tests ±0  322 ✔️ ±0  0 💤 ±0  0 ±0 
652 runs  ±0  652 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 94d9f43. ± Comparison against base commit c37d489.

@derklaro derklaro merged commit a10a86f into nightly Jun 15, 2023
@derklaro derklaro deleted the fix/npc-spawns branch June 15, 2023 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: module An issue/pull request releated to one of the internal modules t: fix A pull request introducing a fix for a bug. v: 4.X This pull should be included in the 4.0 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants