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

[5.0] revert string removal #42388

Merged
merged 1 commit into from
Nov 24, 2023
Merged

Conversation

brianteeman
Copy link
Contributor

COM_USERS_MAIL_PLEASE_FILL_IN_THE_FORM_CORRECTLY="Please fill in the form correctly."

This string was marked as deprecated in #39374 and removed from 5.0 as a result of the deprecation

However this was in error as the string is used

$this->setError(Text::_('COM_USERS_MAIL_PLEASE_FILL_IN_THE_FORM_CORRECTLY'));

This simple PR restores the string.

I have made it against 5.0 as I see this as a bugfix but maintainers may decide its a new string and therefore must be in 5.1

Pull Request for Issue # .

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

COM_USERS_MAIL_PLEASE_FILL_IN_THE_FORM_CORRECTLY="Please fill in the form correctly."

This string was marked as deprecated in joomla#39374 and removed from 5.0 as a result of the deprecation

However this was in error as the string is used https://github.com/joomla/joomla-cms/blob/43a9aab4992d3048e8f2ee5091cd76959fc52acc/administrator/components/com_users/src/Model/MailModel.php#L126

This simple PR restores the string.

I have made it against 5.0 as I see this as a bugfix but maintainers may decide its a new string and therefore must be in 5.1
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.0-dev labels Nov 22, 2023
@ChristineWk
Copy link

I have tested this item ✅ successfully on d3c311e


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

@Quy
Copy link
Contributor

Quy commented Nov 23, 2023

I have tested this item ✅ successfully on d3c311e


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

@Quy
Copy link
Contributor

Quy commented Nov 23, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 23, 2023
@HLeithner HLeithner added the bug label Nov 23, 2023
@bembelimen bembelimen merged commit 3c39160 into joomla:5.0-dev Nov 24, 2023
4 checks passed
@bembelimen
Copy link
Contributor

Thx

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 24, 2023
@bembelimen bembelimen added this to the Joomla! 5.0.1 milestone Nov 24, 2023
@brianteeman brianteeman deleted the mailerror branch November 24, 2023 08:57
Kostelano added a commit to JPathRu/localisation that referenced this pull request Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants