Skip to content

Fix: What can I use as the default value in com_fields field imagelist#24643

Closed
ReLater wants to merge 7 commits intojoomla:stagingfrom
ReLater:patch-1
Closed

Fix: What can I use as the default value in com_fields field imagelist#24643
ReLater wants to merge 7 commits intojoomla:stagingfrom
ReLater:patch-1

Conversation

@ReLater
Copy link
Contributor

@ReLater ReLater commented Apr 19, 2019

Pull Request for Issue #15173

Summary of Changes

Testing Instructions

Expected result

  • Field can be saved.
  • You can select an image in articles and the default value is respected when you don't select another one.
  • Image is displayed in frontend in article view.

Actual result

  • Field can't be saved when you fill the Default Value field because validation fails.

@laoneo please check.

@ghost
Copy link

ghost commented Apr 24, 2019

@astridx @joomdonation can you please test?

@ReLater
Copy link
Contributor Author

ReLater commented May 25, 2019

No tests.
No comments of pinged experts here and in > 2 years old issue.
TBH i never use com_fields.
Closing.

@ReLater ReLater closed this May 25, 2019
@infograf768
Copy link
Member

I tested this patch OK here.
See #26350 (comment)

Please @ReLater can you reopen?

@infograf768
Copy link
Member

No time to wait if we want to get this in for 3.9.12
Thanks @ReLater I have redone the patch here #26352

@EJBJane
Copy link

EJBJane commented Sep 18, 2019

I have made 4 screenshots here to show the process. I have purged the cache now also at CDN level and turned it off. The problem of the default image not showing up in frontend persists with me, however the validation error is now solved with patch #26352 .

screencapture-1-field.pdf
screencapture-2-field.pdf
screencapture-3-field.pdf
screencapture-4-field.pdf

@infograf768
Copy link
Member

@EJBJane
Please now post in #26352
Looks like your display problem is not related to Joomla core.
The test is to be done with Protostar and default com_content

@EJBJane
Copy link

EJBJane commented Sep 18, 2019

okay get it

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