Functions with plural params not recognized

[ShaneBeee]

Skript/Server Version

> sk info
[12:05:35 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[12:05:35 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[12:05:35 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[12:05:35 INFO]: [Skript] Server Version: 1.21.8-10-c8a8c0e (MC: 1.21.8)
[12:05:35 INFO]: [Skript] Skript Version: 2.12.0 (skriptlang-github)
[12:05:35 INFO]: [Skript] Installed Skript Addons: None
[12:05:35 INFO]: [Skript] Installed dependencies: None

Bug Description

When using a function with a plural parameter, the function fails to be usable.

Expected Behavior

The function should work as it used to

Steps to Reproduce

run the following code

function test(p: potion effect types):
	broadcast {_p::*}

on load:
	test(night vision and glowing)

Errors or Screenshots

get the following error:

[12:05:20 INFO]: Line 6: (test.sk)
[12:05:20 INFO]:     The function 'test(potioneffecttype, potioneffecttype)' does not exist.
[12:05:20 INFO]:     Line: test(night vision and glowing)

Other

No response

Agreement

• I have read the guidelines above and affirm I am following them with this report.

[APickledWalrus]

A check for this exists, but the parameters for the converter check are swapped

Skript/src/main/java/ch/njol/skript/lang/function/FunctionRegistry.java

Lines 435 to 441 in 3fb60bd

	// make sure all types in the passed array are valid for the array parameter
	Class<?> arrayType = candidate.args[0].componentType();
	for (Class<?> arrayArg : provided.args) {
	if (!Converters.converterExists(arrayType, arrayArg)) {
	continue candidates;
	}
	}

Thus, this issue doesn't appear for simple types like integers, but it can for potion effect types.

However, this check in general should probably be moved to a later point.