RTC Language Change PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
27 Mar 2025

Summary of Changes

replacement pr for #45192 #45191

In the admin articles view when filtering by author you have the option to filter the authors "created by me"
This PR adds that option to the articles featured view in the admin AND the articles modal view in the frontend

There is a check in the Author column to display the author name if the id is not 0 else to display the author as None

I believe this check to be incorrect. When a user is deleted from the users table the created_by in the content table is untouched so the condition in the query is never passed.

This PR changes the condition to check for the author_name in the featured articles and this way we can satisfy the condition and are able to display the author as None

New in this PR

The string displayed in the filter dropdown for "Created by me" has been added to the site language and changed to be wrapped in - like other filter strings
image

The display of the string "None" to indicate no author in the list view has been wrapped in [ ] to differentiate it from a user who could theoretically be called none

image

Testing Instructions

Create two users
Create featured articles with each user
Delete one of the users
Check the author column in the admin content list and content featured list
Check the author filter in the admin content list and content featured list AND the article modals

Expected result AFTER applying this Pull Request

The articles view in the admin does have the option "created by me"
The features articles view in the admin does have the option "created by me"
The article modal when accessed from the CMS Content - Article button when editing an article in the front end does have the option "created by me"
The article modal when accessed from the CMS Content - Article button when editing an article in the admin does have the option "created by me"
the filters are displayed as shown as in the screenshot above
The deleted author is shown as in the screenshot above

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 - 27 Mar 2025
avatar brianteeman brianteeman - change - 27 Mar 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Mar 2025
Category Administration com_content Language & Strings Front End
avatar fgsw fgsw - test_item - 27 Mar 2025 - Tested successfully
avatar fgsw
fgsw - comment - 27 Mar 2025

I have tested this item ✅ successfully on e548167


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

avatar fgsw
fgsw - comment - 27 Mar 2025

@brianteeman Part of the "Expected result …" should be changed:

The features articles view in the admin does not have the option "created by me"
The article modal when accessed from the CMS Content - Article button when editing an article in the front end does not have the option "created by me"

avatar brianteeman brianteeman - change - 27 Mar 2025
The description was changed
avatar brianteeman brianteeman - edited - 27 Mar 2025
avatar brianteeman
brianteeman - comment - 27 Mar 2025

@fgsw thanks for spotting that - I have updated the instructions

avatar viocassel viocassel - test_item - 28 Mar 2025 - Tested successfully
avatar viocassel
viocassel - comment - 28 Mar 2025

I have tested this item ✅ successfully on e548167


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

avatar richard67 richard67 - change - 28 Mar 2025
Status Pending Ready to Commit
Labels Added: Language Change PR-5.4-dev
avatar richard67
richard67 - comment - 28 Mar 2025

RTC


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

avatar richard67
richard67 - comment - 30 Mar 2025

@brianteeman Part of the "Expected result …" should be changed:

The features articles view in the admin does not have the option "created by me"
The article modal when accessed from the CMS Content - Article button when editing an article in the front end does not have the option "created by me"

@fgsw thanks for spotting that - I have updated the instructions

@brianteeman It seems you have fixed only the 2nd one but not the 1st one he had mentioned, because that still is

The features articles view in the admin does not have the option "created by me"

avatar brianteeman brianteeman - change - 30 Mar 2025
The description was changed
avatar brianteeman brianteeman - edited - 30 Mar 2025
avatar brianteeman
brianteeman - comment - 30 Mar 2025

Updated

avatar richard67
richard67 - comment - 30 Mar 2025

@brianteeman Do I observe right that for successful filtering for deleted users (- None -) it needs PR #45186 in addition? It seems that PR has a mistake as it breaks the "Created by me" filter, see my test result there.

And should we use the same, dashes for the filtering options and squared brackets for the display in the list, also at other places in the administrator, e.g. in PRs #45223 for the username in the User Notes list and in PR #45201 for the field group name in fields lists?

Just asking for your advise. Nothing to be changed in this PR here, it is fine.

avatar brianteeman
brianteeman - comment - 30 Mar 2025
avatar richard67 richard67 - change - 13 Apr 2025
Labels Added: RTC
avatar richard67 richard67 - change - 13 Apr 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-04-13 12:21:55
Closed_By richard67
avatar richard67 richard67 - close - 13 Apr 2025
avatar richard67 richard67 - merge - 13 Apr 2025
avatar richard67
richard67 - comment - 13 Apr 2025

Thanks all (author and testers).

avatar brianteeman
brianteeman - comment - 13 Apr 2025

thanks

Add a Comment

Login with GitHub to post a comment