-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Move JLanguage* to namespace #14410
Move JLanguage* to namespace #14410
Conversation
defined('JPATH_PLATFORM') or die; | ||
|
||
/** | ||
* Utitlity class for multilang | ||
* | ||
* @since 2.5.4 | ||
*/ | ||
class JLanguageMultilang | ||
class Multilanguage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason to not capitalize it now? MultiLanguage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it not one word?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depending on the source you use "multilanguage" isn't even a word...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Somebody on twitter told me that it is correct like it is now. But he can't comment because of a ban.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was my source https://en.wiktionary.org/wiki/multilanguage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw that one too. Honestly I'd say it's fine as is. MultiLanguage works too. But I didn't major in English (any variant in case someone wants to comment on en-US-isms), so don't take my word as the gospel.
Moves the classes JLanguageAssociations and JLanguageMultilang to namespace.
To ensure BC, it was tested with lanternfish to have a test with an existing extension. Lanternfish is overriding JLanguageMultilang on the site part of Joomla and that worked. So @wilsonge guess we have a working real world scenario.