? ? ? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
20 Apr 2021

Pull Request for Issue # .

Summary of Changes

With PR #32864, @brianteeman made some changes to language files to change subfields plugin to subform plugin. However, the actually plugin code is still uses old name subfields. This PR changes the plugin code to have it completely renamed from subfields to subform

Testing Instructions

  1. Install Joomla 4 nightly build. Or if you have Joomla 4 installation already, please update that installation to Joomla 4 nightly build
  2. Access to Content -> Fields, create at least one subfields custom field. For testing purpose, maybe you can create two text custom fields (these fields should have Categories set to None) and one subfields like in the attached screenshot (the subfields could have two text fields in it's Fields setting)
  • List of Fields
    subfields

  • Subfields settings

subfields_setting

  1. Create a new article, go to Fields tab and enter data for that subfields custom field for the article

  2. Download update package for this PR at https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33200/downloads/42161/Joomla_4.0.0-beta8-dev+pr.33200-Development-Update_Package.zip , go to System -> Update -> Joomla, install that update package

  3. Now, access to Content -> Fields again, confirm that the subfields which you created before is now become subform

subfields_become_subform

  1. Edit that subform custom field (before it was subfields), confirm that Type now become Subform (Subform). Also, the Fields setting still same as before
    type_changed

  2. Edit the article, go to Fields tab, confirm that the data which you entered for the custom fields before is still there.

  3. View the article from frontend (maybe make it featured so that it is displayed at homepage by default) and confirm that the data you entered for the custom fields is being displayed properly

article

avatar joomdonation joomdonation - open - 20 Apr 2021
avatar joomdonation joomdonation - change - 20 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Apr 2021
Category SQL Administration com_admin Postgresql Language & Strings Front End Plugins
dba1cfc 20 Apr 2021 avatar joomdonation CS
avatar joomdonation joomdonation - change - 20 Apr 2021
Labels Added: ? ?
avatar richard67
richard67 - comment - 20 Apr 2021

Several mistakes in SQL found, made a PR. See joomdonation#1 .

avatar joomla-cms-bot joomla-cms-bot - change - 20 Apr 2021
Category SQL Administration com_admin Postgresql Language & Strings Front End Plugins SQL Administration com_admin Postgresql Language & Strings Installation Front End Plugins
avatar joomla-cms-bot joomla-cms-bot - change - 20 Apr 2021
Category SQL Administration com_admin Postgresql Language & Strings Front End Plugins Installation SQL Administration com_admin Postgresql Language & Strings Installation Libraries Front End Plugins
avatar joomdonation joomdonation - change - 20 Apr 2021
The description was changed
avatar joomdonation joomdonation - edited - 20 Apr 2021
avatar joomdonation
joomdonation - comment - 20 Apr 2021

I updated testing instructions. This PR is now ready for testing. Please help testing it.

avatar Quy
Quy - comment - 20 Apr 2021

Subforms should not be listed in subforms.

Rename/update?

\administrator\components\com_fields\src\Field\SubfieldstypeField.php

avatar joomdonation
joomdonation - comment - 20 Apr 2021

@Quy I thought about that before and decided that it is not necessary to change. We have Subform. That Subform contains some fields, so the fields which is used inside that Subform could be called as Subfields.

That field type is used to list all fields which can be used in Subform, so leave the field type as how it was before is OK, I think

avatar joomla-cms-bot joomla-cms-bot - change - 20 Apr 2021
Category SQL Administration com_admin Postgresql Language & Strings Front End Plugins Installation Libraries SQL Administration com_admin Postgresql com_fields Language & Strings Installation Libraries Front End Plugins
avatar joomdonation
joomdonation - comment - 20 Apr 2021

@Quy Or maybe we should rename it to SubfieldsField.php. Guess type is not needed in the field type/ filename/classname. I can make the change if needed.

avatar joomdonation
joomdonation - comment - 20 Apr 2021

So I think Subfields is a better name compare to Subfieldstype and made the change.

avatar Quy Quy - test_item - 22 Apr 2021 - Tested successfully
avatar Quy
Quy - comment - 22 Apr 2021

I have tested this item successfully on 1a398f4


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

avatar jwaisner jwaisner - test_item - 22 Apr 2021 - Tested successfully
avatar jwaisner
jwaisner - comment - 22 Apr 2021

I have tested this item successfully on 1a398f4


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

avatar jwaisner jwaisner - change - 22 Apr 2021
Status Pending Ready to Commit
avatar Quy
Quy - comment - 22 Apr 2021

RTC


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

avatar wilsonge wilsonge - close - 23 Apr 2021
avatar wilsonge wilsonge - merge - 23 Apr 2021
avatar wilsonge wilsonge - change - 23 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-23 07:44:19
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 23 Apr 2021

Thanks!

Add a Comment

Login with GitHub to post a comment