Skip to content

Conversation

@chmst
Copy link
Contributor

@chmst chmst commented Nov 19, 2025

Pull Request for Issue #
see https://forum.joomla.de/thread/23070-e-mail-benachrichtigung-bei-neuen-nachrichten/?postID=176266#post176266

Note: The issue does not exist in 5.4.

Summary of Changes

Define $item as \stdClass().

Testing Instructions

On a J6.0 with PHP 8.3 or 8.4 set debug-modus on and error-reporting to "maximum"
Go to Users - messages - private messaging, Click the Settings-Button top left.

Actual result BEFORE applying this Pull Request

grafik

Change Settings, new Settings are not stored.

Expected result AFTER applying this Pull Request

No deprecated messages, Settings are stored correctly

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

@ChristineWk
Copy link

I have tested this item ✅ successfully on a875028


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

@CMS-Manufaktur
Copy link

I have tested this item ✅ successfully on a875028

Test after modifying the file was successful. The error messages disappear and the settings are saved.


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

@richard67
Copy link
Member

System tests are failing. All failures belong to test script site/components/com_privacy/Request.cy.js and are related to privacy requests.

@ChristineWk
Copy link

I have tested this item ✅ successfully on 6be0cec


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

@richard67
Copy link
Member

System tests are still failing, same issue as before the last commit.

@chmst
Copy link
Contributor Author

chmst commented Nov 20, 2025

I see

@CMS-Manufaktur
Copy link

I have tested this item ✅ successfully on 6be0cec


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

@chmst chmst marked this pull request as draft November 20, 2025 18:21
@chmst
Copy link
Contributor Author

chmst commented Nov 20, 2025

Converted to draft due to failing system tests.

@chmst chmst marked this pull request as ready for review November 23, 2025 10:50
@chmst
Copy link
Contributor Author

chmst commented Nov 23, 2025

As all checks have passed, may I ask you for a new test, @ChristineWk and @CMS-Manufaktur? Thank you for your patience. And Thank you @LadySolveig and @bembelimen.

@ChristineWk
Copy link

I have tested this item ✅ successfully on d86bd12


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

@exlemor
Copy link

exlemor commented Nov 28, 2025

I have tested this item ✅ successfully on d86bd12

I have tested this successfully, Thank you @chmst!

To be clear, 2 situations have been detected and corrected with this PR:

  1. you must make some changes to the toggles: My Settings -> Lock Inbox and/or Email New Messages and/or Auto-delete Messages settings AND press Save and Close so that the database has some data in it BEFORE you see the Deprecated messages

  2. the settings BEFORE the PR are not saved.

After the PR, both the Deprecated messages AND the settings are saved correctly.


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

@muhme
Copy link
Contributor

muhme commented Nov 28, 2025

I have tested this item ✅ successfully on d86bd12

Tested with JBT

  • Seen before the PR the deprecations and the values are not saved
  • Applied the PR with Patch Tester and no more deprecations and values are saved

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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug PR-6.0-dev RTC This Pull Request is Ready To Commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants