? ? Pending

User tests: Successful: Unsuccessful:

avatar astridx
astridx
29 Jun 2020

Pull Request for Issue # #29825.

Summary of Changes

If the accessiblemedia field within in a subfield, value is an object stdClass. I extended the check for this. Also, I added the unit tests for the field and do not use $field->apivalueany more because the name is misleading and it is not possible in a subfield custom field.

Testing Instructions

  1. Please test like explained here: #27712

  2. Please also check whether the media custom field can be used within a subform custom field. Note: If you insert the media custom field in a subform custom field, you can only select (when editing the article) an image file after you have saved the article. This issue is not related to this PR. Please see #26711.

  3. Check also the rendering in the front end for the custom field of type media and subfield incluing media field.

Documentation Changes Required

Yes

26a38fc 29 Jun 2020 avatar astridx redu
avatar astridx astridx - open - 29 Jun 2020
avatar astridx astridx - change - 29 Jun 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jun 2020
Category Libraries Front End Plugins Unit Tests
avatar astridx astridx - change - 30 Jun 2020
Labels Added: ? ?
avatar Quy Quy - test_item - 30 Jun 2020 - Tested successfully
avatar Quy
Quy - comment - 30 Jun 2020

I have tested this item successfully on 62c45d2


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

avatar astridx astridx - change - 1 Jul 2020
Labels Added: ?
Removed: ?
avatar wilsonge
wilsonge - comment - 1 Jul 2020

The API Value allows you to override the value returned by the API application. Nothing more.

avatar wilsonge wilsonge - close - 1 Jul 2020
avatar wilsonge wilsonge - merge - 1 Jul 2020
avatar wilsonge wilsonge - change - 1 Jul 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-01 23:44:31
Closed_By wilsonge
Labels Added: ?
Removed: ?
avatar wilsonge
wilsonge - comment - 1 Jul 2020

Thanks!

Add a Comment

Login with GitHub to post a comment