Skip to content

Add default function aliases - #8845

Open
Efnilite wants to merge 4 commits into
dev/featurefrom
feature/function-aliases
Open

Add default function aliases#8845
Efnilite wants to merge 4 commits into
dev/featurefrom
feature/function-aliases

Conversation

@Efnilite

Copy link
Copy Markdown
Member

Problem

Functions with identical implementations previously had to be registered using separate registration calls, resulting in duplicate code.

Solution

This PR adds support for aliases in function signatures. When building a DefaultFunction, the aliases method allows a developer to add additional aliases to an implementation. When this function is registered internally, a function identifier will be added for every alias (including the name) to the appropriate namespace.

Testing Completed

Added integration test for ceil and ceiling.

Supporting Information


Completes: none
Related: none
AI assistance: none

@Efnilite
Efnilite requested a review from a team as a code owner August 18, 2026 11:38
@Efnilite Efnilite added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Aug 18, 2026
@Efnilite
Efnilite requested review from Burbulinis and TheMug06 and removed request for a team August 18, 2026 11:38
@Efnilite Efnilite added the needs reviews A PR that needs additional reviews label Aug 18, 2026
@Efnilite Efnilite moved this to In Review in 2.17 Releases Aug 18, 2026
Comment thread src/main/java/org/skriptlang/skript/common/function/Signature.java Outdated
Comment thread src/main/java/ch/njol/skript/lang/function/FunctionRegistry.java Outdated
Comment thread src/main/java/ch/njol/skript/lang/function/FunctionRegistry.java Outdated
Comment thread src/main/java/ch/njol/skript/lang/function/FunctionRegistry.java Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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: In Review

Development

Successfully merging this pull request may close these issues.

2 participants