Skip to content

[4.3] Convert content fields plugin to services#38470

Merged
obuisard merged 11 commits intojoomla:4.3-devfrom
Digital-Peak:plugins/content/fields
Dec 12, 2022
Merged

[4.3] Convert content fields plugin to services#38470
obuisard merged 11 commits intojoomla:4.3-devfrom
Digital-Peak:plugins/content/fields

Conversation

@laoneo
Copy link
Copy Markdown
Member

@laoneo laoneo commented Aug 16, 2022

Summary of Changes

Converts the content fields plugin to services.

Testing Instructions

  • Create an article custom field
  • Create an article and set as content "Field demo {field 1}".
  • Define a value for the custom field in the article form
  • Open the article on the front

Actual result BEFORE applying this Pull Request

The field is rendered within the article description.

Expected result AFTER applying this Pull Request

The field is rendered within the article description.

@laoneo laoneo changed the title [4.3] Change the content fields plugin to service provider [4.3] Convert content fields plugin to services Aug 24, 2022
laoneo and others added 2 commits November 7, 2022 10:03
@heelc29
Copy link
Copy Markdown
Contributor

heelc29 commented Nov 7, 2022

I have tested this item ✅ successfully on f598104


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

@obuisard
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on a2cab0b


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

@obuisard obuisard added this to the Joomla! 4.3.0 milestone Dec 12, 2022
@obuisard obuisard merged commit 2423bad into joomla:4.3-dev Dec 12, 2022
@obuisard
Copy link
Copy Markdown
Contributor

Thank you Allon @laoneo for the PR

@laoneo laoneo deleted the plugins/content/fields branch December 12, 2022 11:42
richard67 added a commit to richard67/joomla-cms that referenced this pull request Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants