Feature RTC PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
3 Jun 2024

Summary of Changes

Update fields plugins to use SubscriberInterface and event classes.

Testing Instructions

Create/edit custom fields, all should work as before.

Actual result BEFORE applying this Pull Request

Works

Expected result AFTER applying this Pull Request

Works

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: joomla/Manual#177
  • No documentation changes for manual.joomla.org needed
avatar Fedik Fedik - open - 3 Jun 2024
avatar Fedik Fedik - change - 3 Jun 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2024
Category Front End Plugins
avatar Fedik Fedik - change - 4 Jun 2024
Labels Added: Feature PR-5.2-dev
avatar exlemor exlemor - test_item - 23 Jul 2024 - Tested successfully
avatar exlemor
exlemor - comment - 23 Jul 2024

I have tested this item ✅ successfully on 6295367

I created a Textarea field, a radio button field, Media (image) and it worked perfectly.


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

avatar Frostholm7 Frostholm7 - test_item - 8 Aug 2024 - Tested successfully
avatar Frostholm7
Frostholm7 - comment - 8 Aug 2024

I have tested this item ✅ successfully on 6295367

with new installation with 5.1 dev and 5.2 dev
Have I tested
New Group
New Subfild
New calendar
New Articles
Filled in the new fields
Saved
Checked logs
everything ok


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43611.
avatar Quy Quy - change - 8 Aug 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 8 Aug 2024

RTC


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

avatar Quy Quy - change - 15 Aug 2024
Labels Added: RTC
avatar Hackwar
Hackwar - comment - 29 Aug 2024

I'm not convinced that the subform field works and since none of the testers claimed to have tested that, I'd like to wait with merging.

avatar exlemor
exlemor - comment - 29 Aug 2024

@Hackwar - I just retested it with the subform field... Here is a screenshot:


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43611.
avatar exlemor
exlemor - comment - 29 Aug 2024

Here it is (had to use GitHub, Joomla Issue Tracker wouldn't let me add the image :/

The subform field appears at the top and the regular custom field on the bottom.

for-PR-43611-for-Hackwar

avatar HLeithner
HLeithner - comment - 2 Sep 2024

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

avatar HLeithner HLeithner - change - 2 Sep 2024
Title
[5.2][Events] Use event classes for custom fields plugins
[5.3] [Events] Use event classes for custom fields plugins
avatar HLeithner HLeithner - edited - 2 Sep 2024
avatar laoneo laoneo - change - 10 Oct 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-10-10 18:57:53
Closed_By laoneo
Labels Added: PR-5.3-dev
Removed: PR-5.2-dev
avatar laoneo laoneo - close - 10 Oct 2024
avatar laoneo laoneo - merge - 10 Oct 2024
avatar laoneo
laoneo - comment - 10 Oct 2024

Thanks!

Add a Comment

Login with GitHub to post a comment