Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Language.get seems to ignore the .lang file #2890

Closed
Matocolotoe opened this issue Mar 27, 2020 · 3 comments
Closed

Language.get seems to ignore the .lang file #2890

Matocolotoe opened this issue Mar 27, 2020 · 3 comments
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. needs testing Needs testing to determine current status or issue validity, or for WIP feature pulls.

Comments

@Matocolotoe
Copy link
Contributor

Description

The get method from the ch.njol.skript.localization.Language class seems to ignore what's written in the .lang file

Steps to Reproduce

The "none" setting is set to 0 in my .lang file
However, !broadcast (class "ch.njol.skript.localization.Language").get("none") broadcasts <none>

Expected Behavior

The "none" setting shouldn't be ignored

Errors / Screenshots

N/A

Server Information

  • Server version/platform: Spigot 1.14.4
  • Skript version: 2.4.1
@Pikachu920
Copy link
Member

in what lang file? did you edit the default one or did you make a new one?

@Matocolotoe
Copy link
Contributor Author

I edited the default one

@ShaneBeee ShaneBeee added the needs testing Needs testing to determine current status or issue validity, or for WIP feature pulls. label Apr 9, 2020
@ShaneBeee
Copy link
Contributor

I marked this as needs-testing, we'll have to test this.

@TPGamesNL TPGamesNL mentioned this issue May 15, 2021
Merged
@TPGamesNL TPGamesNL added the PR available Issues which have a yet-to-be merged PR resolving it label May 15, 2021
@TPGamesNL TPGamesNL added completed The issue has been fully resolved and the change will be in the next Skript update. and removed PR available Issues which have a yet-to-be merged PR resolving it labels Oct 31, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. needs testing Needs testing to determine current status or issue validity, or for WIP feature pulls.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@ShaneBeee @Pikachu920 @APickledWalrus @TPGamesNL @Matocolotoe