PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
8 Oct 2025

Pull Request for Issue #46192 .

Summary of Changes

When mail is disabled on a site the NEW button is not present in the admin privacy request screen.

This PR adds an on screen message using the existing language string

Testing Instructions

Disable mail sending in global config and go the privacy request screen

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

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 - 8 Oct 2025
avatar brianteeman brianteeman - change - 8 Oct 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Oct 2025
Category Administration
avatar Sulpher
Sulpher - comment - 8 Oct 2025

Thank you for this PR, Brian.

I am ready to test it. Unfortunately, I am not well familiar with full cycle of work with Github (that's why I am not testing other PRs), but if it is possible to make temporary upgrade pack to install on Joomla 6 beta, I'd be happy to test it. I've seen in another issue such a link (which is cool).

avatar brianteeman brianteeman - change - 8 Oct 2025
Labels Added: PR-6.0-dev
avatar brianteeman
brianteeman - comment - 8 Oct 2025

@Sulpher At the bottom of every PR is a block showing the tests that have been run. At the bottom of the list of tests is a link to download you might have to wait until it says are tests have completed

image

That will take you to a page where you can download a full joomla release with the PR applied or alternatively a custom update server you can use in your joomla admin

avatar richard67
richard67 - comment - 8 Oct 2025

@brianteeman Is this only an issue in 6.x? Or also in 5.x? If also in 5.x, the PR should maybe be made for 5.4-dev.

avatar Sulpher
Sulpher - comment - 8 Oct 2025

@brianteeman ok, I understood. Thanks.

@richard67

@brianteeman Is this only an issue in 6.x? Or also in 5.x? If also in 5.x, the PR should maybe be made for 5.4-dev.

I faced with this case on Joomla 5.

avatar richard67
richard67 - comment - 8 Oct 2025

@brianteeman Maybe better redo than rebase? Rebase from branch X to branch Y is always a problem when branch X is in advance to branch Y with commits.

avatar brianteeman brianteeman - close - 8 Oct 2025
avatar brianteeman brianteeman - change - 8 Oct 2025
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2025-10-08 12:41:37
Closed_By brianteeman
avatar joomla-cms-bot joomla-cms-bot - change - 8 Oct 2025
Category Administration Repository Administration com_admin SQL
avatar brianteeman
brianteeman - comment - 8 Oct 2025

see #46256

avatar richard67
richard67 - comment - 8 Oct 2025

Thanks.

Add a Comment

Login with GitHub to post a comment