Skip to content

Fix Incorrect Command Cooldown Storage Warning#8304

Merged
sovdeeth merged 2 commits intoSkriptLang:dev/patchfrom
erenkarakal:patch/fix-command-cooldown-storage
Dec 2, 2025
Merged

Fix Incorrect Command Cooldown Storage Warning#8304
sovdeeth merged 2 commits intoSkriptLang:dev/patchfrom
erenkarakal:patch/fix-command-cooldown-storage

Conversation

@erenkarakal
Copy link
Member

Problem

Using the code:

command /rtp:
  cooldown: 5 minutes
  cooldown storage: {rtp.command::%player's uuid%}
  trigger:
    stop

The command would always print a warning saying Variable {var} was not a date! You may be using this variable elsewhere because it doesn't check if the variable is set first

Solution

Added a != null check

Testing Completed

Manually tested to verify the issue is solved


Completes: none
Related: https://discord.com/channels/135877399391764480/836220422223036467/1444729250224672979

@erenkarakal erenkarakal requested review from a team and sovdeeth as code owners December 1, 2025 11:27
@erenkarakal erenkarakal requested review from cheeezburga and removed request for a team December 1, 2025 11:27
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Dec 1, 2025
Copy link
Member

@APickledWalrus APickledWalrus left a comment

Choose a reason for hiding this comment

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

Seems fine

@skriptlang-automation skriptlang-automation bot added patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. and removed needs reviews A PR that needs additional reviews labels Dec 1, 2025
@github-project-automation github-project-automation bot moved this to Awaiting Merge in 2.13 Releases Dec 2, 2025
@sovdeeth sovdeeth merged commit 06878b5 into SkriptLang:dev/patch Dec 2, 2025
5 of 6 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Dec 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 Dec 2, 2025
@sovdeeth sovdeeth moved this from Done - Awaiting Release to Done - Released in 2.13 Releases Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants

Comments