User tests: Successful: Unsuccessful:
Pull Request for Issue # .
When data for a Repeatable field from Joomla 3 migrated to a Subfields on Joomla 4, the value for media field type is stored in String, not in JSON format as expected, thus it throws some warnings when the field is being displayed (see #32611 (comment) )
This PR attempts to solve that.
Install Joomla 3.10.0. You can download the nightly build package here https://developer.joomla.org/nightlies/Joomla_3.10.0-alpha6-dev-Development-Full_Package.zip to install it.
Access to Content -> Fields, create a Repeatable field. This field should contains at least one Media field for testing purpose (see screenshot for example)
Create one or two articles. When you add article, navigate to Fields tab, enter data for the repeatable field which you created above. See screenshot for example data
Upgrade your site to latest Joomla 4.0-dev package (generate for this PR). Just download the updated package here https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33111/downloads/42298/Joomla_4.0.0-beta8-dev+pr.33111-Development-Update_Package.zip, then go to Components -> Joomla ! Update, look Upload & Update tab and install that update package.
Check the articles on upgraded site, edit these article, check and make sure data in Fields tab is there.
Set the article to Featured so that it is displayed on homepage of the site
Access to the site in the frontend, see the article with custom fields data displayed. Make sure there are no warning. Unfortunately, the image which you select in the field for article could not be displayed because of a bug (is solved in a different PR), so ignore it. Just make sure there are no warnings and the test could be marked as success
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_admin |
Labels |
Added:
?
|
Labels |
Added:
?
|
Labels |
Added:
?
Removed: ? |
I have tested this item
Test was successful! Thanks for the work!
@joomdonation thanks looks good but only did a code review
I have tested this item
I tested this during JoomlaDayUSA BF@H successfully on a live website running php 7.3.26 and mysql: 10.4.17-MariaDB. Others were unsuccessful tho.
Following nr.4 in testing instructions the test fail.
When using the custom URL https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33111/downloads/42298/pr_list.xml it works just fine.
I am writing this just for the record.
The failure was caused because the attributes
?joomla_image_width=710&joomla_image_height=202
where missing in the field value which migrated from J3.10
On comparison between successful and unsuccessful tests we found out the difference was the upgrade package used to upgrade J!3_10 to J!4
The one offered above
https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33111/downloads/42298/Joomla_4.0.0-beta8-dev+pr.33111-Development-Update_Package.zip,
was producing errors on migration.
Using
https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33111/downloads/42298/pr_list.xml
as update server worked correctly
@TLWebdesign @BertaOctech Maybe it is because my testing instructions is outdated.
For the issue with the image mentioned by @BertaOctech , I remember it was fixed in a different PR already
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-24 08:09:38 |
Closed_By | ⇒ | rdeutz | |
Labels |
Added:
?
Removed: ? |
@HLeithner Could you please take a look when you have some free time?