Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Specifying more in details what GetClientLanguage does #332

Closed
wants to merge 2 commits into from
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 3 additions & 0 deletions plugins/include/lang.inc
Expand Up @@ -60,6 +60,9 @@ native SetGlobalTransTarget(client);
/**
* Retrieves the language number of a client.
*
* By default that language number is representing the language
* that the client's Steam application is using.
*
* @param client Client index.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not necessarily true, however, it should be the language that the game is using.

By default, most Source games use the Steam client language, but at least a couple of them actually respect the game language set in Steam (rather than Steam UI language). By changing your text to "that the user's game client is using", it should be accurate for all.

* @return Language number client is using.
* @error Invalid client index or client not connected.
Expand Down