Skip to content

Conversation

@sovdeeth
Copy link
Member

Problem

ExprLevel supported past/future values if the default player was used. However, the pattern did not allow the default player, preventing the use of past/future values.

Solution

Used registerDefault instead of register. Improved code quality, added example, added an extra check for changing past/future values in a level change event.

Testing Completed

Manual testing confirmed.

on level change:
    set {_diff} to future xp level - past exp level
    broadcast "%player%'s level changed by %{_diff}%!"
image

Supporting Information


Completes: none
Related: none

@sovdeeth sovdeeth requested a review from a team as a code owner November 22, 2025 09:17
@sovdeeth sovdeeth added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Nov 22, 2025
@sovdeeth sovdeeth requested review from Pesekjak and TheMug06 and removed request for a team November 22, 2025 09:17
@sovdeeth sovdeeth moved this to In Review in 2.13 Releases Nov 22, 2025
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Nov 22, 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. needs reviews A PR that needs additional reviews

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants