ExprRandom | Removal and replacement #5254
Unanswered
Fusezion
asked this question in
Suggestions
Replies: 1 comment 1 reply
-
This is great in all, but the problem with most community-driven frameworks and languages is legacy code, if the syntax changes, servers that update will face problems with their skripts and old skritps from 3 years ago that still use the |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Suggestion
ExprRandom
from skript's expressionsExprClassinfo
random %*type%
patternfirst/last [%-number%] %*type%
pattern%number%(st|nd|rd|th) %*type%
patternRemoval of ExprRandom brings room for a more central based expression vs. one just for a single pattern expression
Why?
Currently we have the
ExprRandom
class which is just a single expression pattern[a] random %*type% [out] of %objects%
while this probably had plenty of use alone the fact we can do
random %*type%....
should meanfirst string out of
should also work but it doesn't. Removal of this Specified name to a general purpose nameExprClassinfo
is a name that can include more options.Whether
random %*type%....
should be made to support an optional type definition is up for debates doing that would mean removal ofrandom element
makes sense in theExprElement
class we'd need to figure out proper syntax for that portion asrandom of %objects%
doesn't look goodOther
No response
Agreement
Beta Was this translation helpful? Give feedback.
All reactions