? Language Change ? Pending

User tests: Successful: Unsuccessful:

avatar sandewt
sandewt
26 Nov 2022

Pull Request for Issue # .

Summary of Changes

Add rel="noopener noreferrer" in language string

Testing Instructions

Code review

Actual result BEFORE applying this Pull Request

Missing rel="noopener noreferrer" in the language string, because the link contains target="_blank"

Expected result AFTER applying this Pull Request

Language string contains rel="noopener noreferrer"

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

avatar sandewt sandewt - open - 26 Nov 2022
avatar sandewt sandewt - change - 26 Nov 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Nov 2022
Category Administration Language & Strings
avatar sandewt sandewt - change - 26 Nov 2022
The description was changed
avatar sandewt sandewt - edited - 26 Nov 2022
avatar sandewt sandewt - change - 26 Nov 2022
Title
[4.2] missing rel="noopener noreferrer" in language string
[4.2] missing rel="noopener noreferrer" in link language string
avatar sandewt sandewt - edited - 26 Nov 2022
avatar brianteeman brianteeman - test_item - 26 Nov 2022 - Tested successfully
avatar brianteeman
brianteeman - comment - 26 Nov 2022

I have tested this item successfully on 9649fd7


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39317.

avatar richard67
richard67 - comment - 26 Nov 2022

Do we have that string elsewhere, e.g. in the language file for the api client or the site, where it should be fixed, too? Am off my desk now so can’t comfortably check.

Update: I meanwhile checked. All fine, no other file with that string.

avatar richard67 richard67 - test_item - 26 Nov 2022 - Tested successfully
avatar richard67
richard67 - comment - 26 Nov 2022

I have tested this item successfully on 9649fd7


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39317.

avatar richard67 richard67 - change - 26 Nov 2022
Status Pending Ready to Commit
Labels Added: Language Change ?
avatar richard67
richard67 - comment - 26 Nov 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39317.

avatar sandewt
sandewt - comment - 26 Nov 2022

Update: I meanwhile checked. All fine, no other file with that string.

@richard67 Thank you, your comment triggered me.

We missed the installation folder:

INSTL_DATABASE_VALIDATION_ERROR="Sprawdź swoje dane uwierzytelniające, typ bazy danych, nazwę bazy danych lub nazwę hosta. Jeśli masz zainstalowany MySQL 8, przeczytaj <a href=\"https://docs.joomla.org/Joomla_and_MySQL_8#Workaround_to_get_Joomla_working_with_MySQL_8\" target=\"_blank\">wiki</a>, aby uzyskać więcej informacji."

and

INSTL_DATABASE_HOST_IS_NOT_LOCALHOST_GENERAL_MESSAGE="正在尝试使用一个不在您本地服务器上的数据库主机。出于安全考虑,需要验证主机帐户的所有权。<a href=\"%s\" target=\"_blank\">请阅读文档</a> 获取更多信息"

avatar richard67
richard67 - comment - 26 Nov 2022

Update: I meanwhile checked. All fine, no other file with that string.

@richard67 Thank you, your comment triggered me.

We missed the installation folder:

INSTL_DATABASE_VALIDATION_ERROR="Sprawdź swoje dane uwierzytelniające, typ bazy danych, nazwę bazy danych lub nazwę hosta. Jeśli masz zainstalowany MySQL 8, przeczytaj <a href=\"https://docs.joomla.org/Joomla_and_MySQL_8#Workaround_to_get_Joomla_working_with_MySQL_8\" target=\"_blank\">wiki</a>, aby uzyskać więcej informacji."

and

INSTL_DATABASE_HOST_IS_NOT_LOCALHOST_GENERAL_MESSAGE="正在尝试使用一个不在您本地服务器上的数据库主机。出于安全考虑,需要验证主机帐户的所有权。<a href=\"%s\" target=\"_blank\">请阅读文档</a> 获取更多信息"

@sandewt These are translations. If the English strings are ok, the translations have to be fixed on Crowdin, as far as I know.

avatar sandewt
sandewt - comment - 26 Nov 2022

If the English strings are ok, the translations have to be fixed on Crowdin, as far as I know.

I also found the same issue in a language pack. I will pass this on to the relevant translator.

@MartijnMaandag

avatar laoneo laoneo - change - 28 Nov 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-11-28 08:57:54
Closed_By laoneo
Labels Added: ?
avatar laoneo laoneo - close - 28 Nov 2022
avatar laoneo laoneo - merge - 28 Nov 2022
avatar laoneo
laoneo - comment - 28 Nov 2022

Thank you!

Add a Comment

Login with GitHub to post a comment