Removal of case sensitive
config option
#4823
Unanswered
APickledWalrus
asked this question in
Suggestions
Replies: 1 comment
-
Doesn't this setting mainly impact variable names? I don't mind the syntax addition regardless of the removal. Add option to the bStats to see how many actually use this option https://bstats.org/plugin/bukkit/Skript/722 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I am proposing that we remove the following config option:
Skript/src/main/resources/config.sk
Lines 139 to 142 in 0c74e07
Instead, each syntax making use of this config option should have an individual
with case sensitivity
option that can be specified. This can already be found in EffReplace:Skript/src/main/java/ch/njol/skript/effects/EffReplace.java
Lines 57 to 60 in 0c74e07
Why? In the same way that we removed non-message stuff from non-English localization files (as we decided that non-message stuff shouldn't be translated, since it could break script compatibility between languages), I think that this setting only serves to cause a similar problem. It would also be far more useful for users to be able to switch between case sensitivity checks, rather than be forced to go all in for all syntaxes.
Beta Was this translation helpful? Give feedback.
All reactions