-
Notifications
You must be signed in to change notification settings - Fork 443
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
Fix #1921 and others untranslated terms #1923
Conversation
Fix #1921 and others untranslated terms
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 would split the PR into 2
1 for simple static gettext and other for the once that take vars..
src/QueryView.php
Outdated
@@ -302,8 +302,8 @@ function getQueryFormInput($queryParameters) | |||
extract($queryParameters); | |||
|
|||
$input = ''; | |||
$label = '<label>' . $qrp_Name . '</label>'; | |||
$helpMsg = '<div>' . $qrp_Description . '</div>'; | |||
$label = '<label>' . gettext ($qrp_Name) . '</label>'; |
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.
these do not work...
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.
Sorry ... I tested this before submit PR, and works fine...
src/SystemSettings.php
Outdated
@@ -128,7 +128,7 @@ | |||
<li class="<?php if ($category == 'Church Information') { | |||
echo 'active'; | |||
} ?>"><a href="#<?= str_replace(" ", '', $category) ?>" data-toggle="tab" | |||
aria-expanded="false"><?= $category ?></a> | |||
aria-expanded="false"><?= gettext($category) ?></a> |
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.
the categories need to be localized
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.
The categories are in POEditor.com since older versions. This stop working in 2.6.0-RC1 ....
revert gettext($category)
Fix #1921 and others untranslated terms