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
Language related issues. #1484
Language related issues. #1484
Conversation
Added missing line
Correct a typo and add a missing line
Great thanks!
|
@@ -1,5 +1,8 @@ | |||
<?php defined('BASEPATH') or exit('No direct script access allowed'); | |||
// Arabic | |||
$lang['no_username_value_provided'] = 'No username value provided.'; |
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.
Please remove no_username_value_provided and no_password_value_provided and move invalid_credentials_provided to the end of the file before // End
application/controllers/Login.php
Outdated
@@ -59,19 +59,19 @@ public function validate() | |||
$username = request('username'); | |||
|
|||
if (empty($username)) { | |||
throw new InvalidArgumentException('No username value provided.'); | |||
throw new InvalidArgumentException(lang('no_username_value_provided')); |
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.
This does not need to be translated because the JS will not allow for empty username submissions any longer
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.
Done, see 3a0f41d
application/controllers/Login.php
Outdated
} | ||
|
||
$password = request('password'); | ||
|
||
if (empty($password)) { | ||
throw new InvalidArgumentException('No password value provided.'); | ||
throw new InvalidArgumentException(lang('no_password_value_provided')); |
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.
This does not need to be translated because the JS will not allow for empty username submissions any longer
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.
Done, see 3a0f41d
Changed as requested |
Language related updates