Skip to content

Fix name of event-inv being wrong#8438

Open
sovdeeth wants to merge 2 commits intoSkriptLang:dev/patchfrom
sovdeeth:patch/allow-event-overrides-in-type-property-handlers
Open

Fix name of event-inv being wrong#8438
sovdeeth wants to merge 2 commits intoSkriptLang:dev/patchfrom
sovdeeth:patch/allow-event-overrides-in-type-property-handlers

Conversation

@sovdeeth
Copy link
Member

Problem

on inventory open:
  broadcast name of event-inventory
  wait 1 tick
  broadcast name of event-inventory

name is always "crafting" even if you currently have a gui open and you open another one on top
if you wait a tick its fine
!open new chest inventory named "abc" to all players
!open new chest inventory named "abcd" to all players
Crafting
abc
Crafting
abcd

Solution

Adds a default convert(event, T) method to the expression property handler so expressions can override behavior for event-specific values.

Testing Completed

Manual confirmation

Supporting Information


Completes: none
Related: none
AI assistance: none

@sovdeeth sovdeeth requested a review from a team as a code owner February 11, 2026 21:32
@sovdeeth sovdeeth added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Feb 11, 2026
@sovdeeth sovdeeth requested a review from a team as a code owner February 11, 2026 21:32
@sovdeeth sovdeeth requested review from Efnilite and UnderscoreTud and removed request for a team February 11, 2026 21:32
@sovdeeth sovdeeth moved this to In Review in 2.14 Releases Feb 12, 2026
@sovdeeth sovdeeth added the needs reviews A PR that needs additional reviews label Feb 12, 2026
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. needs reviews A PR that needs additional reviews

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants

Comments