Skip to content

Conversation

@heelc29
Copy link
Contributor

@heelc29 heelc29 commented Jan 13, 2026

Summary of Changes

Same as #45644 for new plugin (fields note) of Joomla 6

Info: plugin fields number was fixed during upmerge commit f200a7d

Testing Instructions

Enable Log Deprecated API
image

Open view to add new custom field (/administrator/index.php?option=com_fields&view=field&layout=edit&context=com_content.article)

Check log file administrator\logs\deprecated.php

Actual result BEFORE applying this Pull Request

Log message
deprecated Passing an instance of Joomla\Event\DispatcherInterface to Joomla\CMS\Plugin\CMSPlugin::__construct() will not be supported in 7.0. Starting from 7.0 CMSPlugin class will no longer implement DispatcherAwareInterface. - [ROOT]\libraries\src\Plugin\CMSPlugin.php - Line 134

Expected result AFTER applying this Pull Request

No log message

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@ceford
Copy link
Contributor

ceford commented Jan 14, 2026

I have tested this item 🔴 unsuccessfully on 3ba9fb2

Clearing the deprecated log before selecting New I get two instances like this on lines 11 and 57:

2026-01-14T09:38:09+00:00	WARNING	127.0.0.1	deprecated	Passing an instance of Joomla\Event\DispatcherInterface to Joomla\CMS\Plugin\CMSPlugin::__construct() will not be supported in 7.0. Starting from 7.0 CMSPlugin class will no longer implement DispatcherAwareInterface. - [ROOT]/libraries/src/Plugin/CMSPlugin.php - Line 134
...
2026-01-14T09:38:10+00:00	WARNING	127.0.0.1	deprecated	Passing an instance of Joomla\Event\DispatcherInterface to Joomla\CMS\Plugin\CMSPlugin::__construct() will not be supported in 7.0. Starting from 7.0 CMSPlugin class will no longer implement DispatcherAwareInterface. - [ROOT]/libraries/src/Plugin/CMSPlugin.php - Line 134

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

@ceford
Copy link
Contributor

ceford commented Jan 14, 2026

I have not tested this item.

Looks like I made a mistake!


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

@ceford
Copy link
Contributor

ceford commented Jan 14, 2026

I have tested this item ✅ successfully on 3ba9fb2

Looks like I made a mistake! Tested again and this time it works. I don't know what I did wrong the first time around.


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

Copy link
Contributor

@tecpromotion tecpromotion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review.

@tecpromotion
Copy link
Contributor

I have tested this item ✅ successfully on 3ba9fb2


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

@tecpromotion
Copy link
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 14, 2026
@Bodge-IT
Copy link
Contributor

I have tested this item ✅ successfully on 3ba9fb2


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

@brianteeman
Copy link
Contributor

shouldnt this be for 6.1

@softforge softforge added this to the Joomla! 6.0.3 milestone Jan 14, 2026
@softforge softforge merged commit ea52252 into joomla:6.0-dev Jan 14, 2026
51 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 14, 2026
@Bodge-IT
Copy link
Contributor

@brianteeman #46683 for 6.1

@brianteeman
Copy link
Contributor

@Bodge-IT my point was that this isnt a bug fix therefore it should be for 6.1 and not a patch release

@heelc29 heelc29 deleted the 6.0/plugins/remove-dispatcher branch January 17, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants