Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 7 commits into from
Closed

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.

@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.

None yet

4 participants