? ? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
23 Apr 2021

Pull Request for Issue # .

Summary of Changes

In the PR #33200 (which was merged), I renamed the plugin from Subfields to Subform. However, there are still few instances of Subfields which should be updated to Subform. This PR just makes that final clean up.

Testing Instructions

  1. In update package for the original PR 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 . You can go to System -> Update -> Joomla to update that package
  2. Access to Content -> Fields, create a new Subform custom field
  3. Before patch, in the Fields setting of that Subform custom field, you can choose the same field multiple times and the system still save it
  4. After patch, if you choose same field multiple times, only one will be saved
    subform-fields
avatar joomdonation joomdonation - open - 23 Apr 2021
avatar joomdonation joomdonation - change - 23 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Apr 2021
Category Administration com_fields Front End Plugins
avatar Quy Quy - test_item - 23 Apr 2021 - Tested successfully
avatar Quy
Quy - comment - 23 Apr 2021

I have tested this item successfully on 4dc9a40


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

avatar Quy Quy - change - 23 Apr 2021
Labels Added: ?
avatar eddiekonczal
eddiekonczal - comment - 23 Apr 2021

By "Subform custom field" do you mean a "Subfields" custom field?


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

avatar Quy
Quy - comment - 23 Apr 2021

Please use the download link at the bottom of the page.

download

avatar eddiekonczal
eddiekonczal - comment - 23 Apr 2021

What is the URL of the page with the Download link?


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

avatar RickR2H RickR2H - test_item - 24 Apr 2021 - Tested successfully
avatar RickR2H
RickR2H - comment - 24 Apr 2021

I have tested this item successfully on 4af60a9


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

avatar richard67 richard67 - alter_testresult - 24 Apr 2021 - Quy: Tested successfully
avatar richard67
richard67 - comment - 24 Apr 2021

I've restored @Quy 's test result in the issue tracker so it's counted properly. The commit which had invalidated it was just a change in a comment.

avatar richard67 richard67 - change - 24 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 24 Apr 2021

RTC again (somehow the bot removed it)


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

avatar rdeutz rdeutz - change - 24 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-24 09:22:48
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 24 Apr 2021
avatar rdeutz rdeutz - merge - 24 Apr 2021
avatar eddiekonczal
eddiekonczal - comment - 27 Apr 2021

https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33262/downloads/42332/

When I go to the above URL and try to download the update package, I get:

**Not Found

The requested URL was not found on this server.**

Download link:

https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33262/downloads/42332/Joomla_4.0.0-beta8-dev+pr.33262-Development-Update_Package.zip

avatar joomdonation
joomdonation - comment - 27 Apr 2021

Thanks @eddiekonczal. This PR was merged, so we do not need to test it anymore.

avatar PhilETaylor
PhilETaylor - comment - 27 Apr 2021

You can of course use the Nightly build if you want to see this now its merged https://developer.joomla.org/nightlies/Joomla_4.0.0-beta8-dev-Development-Full_Package.zip

Add a Comment

Login with GitHub to post a comment