Skip to content

Fix ExprNamed ArrayStoreException#8763

Open
erenkarakal wants to merge 1 commit into
SkriptLang:dev/patchfrom
erenkarakal:patch/fix-property-named-arr-store-exception
Open

Fix ExprNamed ArrayStoreException#8763
erenkarakal wants to merge 1 commit into
SkriptLang:dev/patchfrom
erenkarakal:patch/fix-property-named-arr-store-exception

Conversation

@erenkarakal

Copy link
Copy Markdown
Member

Problem

set {_inv} to shulker box inventory named {_} throws an ArrayStoreException because it doesn't convert InventoryType to an Inventory

Solution

In ExprNamed, if no name is given, create an inventory with no title instead

Testing Completed

manual ingame test & regression test

Supporting Information


Completes: #5923
Related: none
AI assistance: none

@erenkarakal erenkarakal requested a review from a team as a code owner July 8, 2026 12:50
@erenkarakal erenkarakal requested review from Pesekjak and cheeezburga and removed request for a team July 8, 2026 12:50
@erenkarakal erenkarakal added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Jul 8, 2026
@erenkarakal erenkarakal linked an issue Jul 8, 2026 that may be closed by this pull request
1 task

@APickledWalrus APickledWalrus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Part of me wonders if we shouldn't return anything, but that's breaking behavior I suppose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An issue that needs to be fixed. Alternatively, a PR fixing an issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] StackTrace when defining names of inventory types

4 participants