Skip to content

Conversation

@TFSMads
Copy link
Contributor

@TFSMads TFSMads commented Oct 22, 2025

Problem

When setting a chest from a variable where the chest name is set, it won't apply to the actual chest.

Solution

I have added copying of name from itemmeta to the container in ItemType#copyContainerState.

Testing Completed

I have written a test script that i have run successfully on versions: 1.20.6, 1.21.3 , 1.21.4, 1.21.5, 1.21.8 and 1.21.10

Supporting Information


Completes: #8246
Related: none

@TFSMads TFSMads requested review from a team and sovdeeth as code owners October 22, 2025 12:43
@TFSMads TFSMads requested review from Pesekjak and removed request for a team October 22, 2025 12:43
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Oct 22, 2025
@TFSMads TFSMads changed the base branch from master to dev/patch October 22, 2025 12:45
TFSMads and others added 2 commits October 22, 2025 05:20
Co-authored-by: Eren <67760502+erenkarakal@users.noreply.github.com>
Co-authored-by: Eren <67760502+erenkarakal@users.noreply.github.com>
Copy link
Member

@erenkarakal erenkarakal left a comment

Choose a reason for hiding this comment

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

lgtm

Co-authored-by: SirSmurfy2 <82696841+Absolutionism@users.noreply.github.com>
@skriptlang-automation skriptlang-automation bot removed the needs reviews A PR that needs additional reviews label Oct 22, 2025
@skriptlang-automation skriptlang-automation bot added the patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. label Oct 23, 2025
@sovdeeth sovdeeth moved this to In Review in 2.13 Releases Oct 23, 2025
@sovdeeth sovdeeth moved this from In Review to Awaiting Merge in 2.13 Releases Oct 23, 2025
@sovdeeth sovdeeth added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Oct 23, 2025
@sovdeeth sovdeeth linked an issue Oct 23, 2025 that may be closed by this pull request
1 task
@APickledWalrus APickledWalrus merged commit 9e9ce15 into SkriptLang:dev/patch Nov 2, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Nov 2, 2025
@skriptlang-automation skriptlang-automation bot added completed The issue has been fully resolved and the change will be in the next Skript update. and removed patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. labels Nov 2, 2025
@APickledWalrus APickledWalrus moved this from Done - Awaiting Release to Done - Released in 2.13 Releases Nov 2, 2025
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
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. completed The issue has been fully resolved and the change will be in the next Skript update.

Projects

No open projects
Status: Done - Released

Development

Successfully merging this pull request may close these issues.

block name set from variable not working

5 participants