Skip to content

Wandering traders - Material cannot be null #94

@weeryan17

Description

@weeryan17

Expected behavior

Not get an error when spawning in a wandering trader

Observed/Actual behavior

Getting an error when spawning in wandering trader

Steps/models to reproduce

Enable wandering traders feature.
Spawn wandering traders

Plugin and Datapack List

plugins:

ArmoredElytra, BlueBorder, BlueMap, CoreProtect, FastChunkPregenerator, floodgate, Geyser-Spigot, GSit, NoChatReports, NoEndermanGrief

Server Version Info

This server is running Paper version 1.21.4-138-main@5395ae3 (2025-02-02T21:42:11Z) (Implementing API version 1.21.4-R0.1-SNAPSHOT)

Plugin Version

0.5.0

Other

Stack trance:
https://pastebin.com/dzJNN0sT

Looks like at some point the secondary item in the wandering traders list (https://github.com/MC-Machinations/PaperTweaks/blob/master/src/main/resources/data/wandering_trades.json) was changed to null instead of air and the code was never updated to check for null instead of air
https://github.com/MC-Machinations/PaperTweaks/blob/master/src/main/java/me/machinemaker/papertweaks/modules/hermitcraft/wanderingtrades/Trade.java#L49

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions