Skip to content

Add missing PDC syntaxes#8528

Merged
sovdeeth merged 4 commits intoSkriptLang:dev/featurefrom
sovdeeth:feature/extra-pdc-expressions
Apr 12, 2026
Merged

Add missing PDC syntaxes#8528
sovdeeth merged 4 commits intoSkriptLang:dev/featurefrom
sovdeeth:feature/extra-pdc-expressions

Conversation

@sovdeeth
Copy link
Copy Markdown
Member

@sovdeeth sovdeeth commented Apr 7, 2026

Problem

The original PDC PR forgot the necessary has tag condition syntax and the all tags expression.

Solution

Adds a condition to check if a holder has a specific tag, and an expression to get the keys available on a holder.
Reworked the edit/get container utility methods and put them in a utils class.

Testing Completed

Added automated tests via Claude Code.

Supporting Information


Completes: none
Related: none
AI assistance: Claude generated the test scripts.

@sovdeeth sovdeeth requested a review from a team as a code owner April 7, 2026 05:14
@sovdeeth sovdeeth added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. enhancement Feature request, an issue about something that could be improved, or a PR improving something. labels Apr 7, 2026
@sovdeeth sovdeeth requested review from TheMug06 and cheeezburga and removed request for a team April 7, 2026 05:14
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Apr 7, 2026
Comment thread src/test/skript/tests/syntaxes/expressions/ExprAllPersistentDataKeys.sk Outdated
Comment thread src/test/skript/tests/syntaxes/conditions/CondHasPersistentDataTag.sk Outdated
Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com>
Copy link
Copy Markdown
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.

Looking good so far

Comment thread src/main/java/org/skriptlang/skript/bukkit/pdc/PDCUtils.java Outdated
@github-project-automation github-project-automation bot moved this to In Review in 2.15 Releases Apr 9, 2026
@sovdeeth sovdeeth requested a review from APickledWalrus April 11, 2026 00:25
registry.register(SyntaxRegistry.EXPRESSION,
SyntaxInfo.Expression.builder(ExprAllPersistentDataKeys.class, String.class)
.addPatterns(
"[all [[of] the]] [persistent] data [tag] keys of %objects%",
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.

Suggested change
"[all [[of] the]] [persistent] data [tag] keys of %objects%",
"[all [[of] the]|the] [persistent] data [tag] keys of %objects%",

@github-project-automation github-project-automation bot moved this from In Review to Awaiting Merge in 2.15 Releases Apr 11, 2026
@sovdeeth sovdeeth merged commit 046bf68 into SkriptLang:dev/feature Apr 12, 2026
13 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.15 Releases Apr 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. enhancement Feature request, an issue about something that could be improved, or a PR improving something. needs reviews A PR that needs additional reviews

Projects

Status: Done - Awaiting Release

Development

Successfully merging this pull request may close these issues.

3 participants