Feature RTC Language Change PR-5.1-dev PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
23 Feb 2024

The code loads a joomla language string with two params The text string ignores the first param and just outputs the second with no additional data

In other words both the language string and the first param can be removed without any impact.

Pull Request for Issue #41233

Actual result BEFORE applying this Pull Request

image

image

Expected result AFTER applying this Pull Request

image

image

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

avatar brianteeman brianteeman - open - 23 Feb 2024
avatar brianteeman brianteeman - change - 23 Feb 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2024
Category Administration Language & Strings Libraries
avatar Quy Quy - test_item - 23 Feb 2024 - Tested successfully
avatar Quy
Quy - comment - 23 Feb 2024

I have tested this item ✅ successfully on a9cb0a7


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

avatar viocassel viocassel - test_item - 23 Feb 2024 - Tested successfully
avatar viocassel
viocassel - comment - 23 Feb 2024

I have tested this item ✅ successfully on a9cb0a7


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

avatar Quy Quy - change - 23 Feb 2024
Status Pending Ready to Commit
Labels Added: Language Change PR-5.0-dev
avatar Quy
Quy - comment - 23 Feb 2024

RTC


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

avatar Quy Quy - change - 25 Feb 2024
Labels Added: RTC
avatar brianteeman
brianteeman - comment - 8 Apr 2024

is there a reason that this RTC pr is not being merged?

avatar Quy Quy - change - 9 Apr 2024
Title
[5.0] Remove useless string
[5.1] Remove useless string
Labels Added: PR-5.1-dev
Removed: PR-5.0-dev
avatar HLeithner
HLeithner - comment - 24 Apr 2024

@bembelimen any reason this to keep the language string? Don't think so but just want to ask if any language override can make sense here.

avatar HLeithner
HLeithner - comment - 24 Apr 2024

This pull request has been automatically rebased to 5.2-dev.

avatar HLeithner HLeithner - change - 24 Apr 2024
Title
[5.1] Remove useless string
[5.2] Remove useless string
avatar HLeithner HLeithner - edited - 24 Apr 2024
avatar brianteeman
brianteeman - comment - 24 Apr 2024

Closed due to lack of interest or response from release leads

avatar brianteeman brianteeman - close - 24 Apr 2024
avatar brianteeman brianteeman - change - 24 Apr 2024
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2024-04-24 19:09:06
Closed_By brianteeman
Labels Added: Feature PR-5.2-dev

Add a Comment

Login with GitHub to post a comment