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

Encode value when an array #19026

Closed
wants to merge 1 commit into from

Conversation

stutteringp0et
Copy link
Contributor

This adjustment allows complex field types (types with more than one input) to store and retrieve data.

Summary of Changes

Test value before save, if it's an array, json encode it

Testing Instructions

Hard to say without installing a 3rd party custom field plugin

Expected result

it should be transparent

Actual result

transparent on my end

Documentation Changes Required

none

This allows complex fields to store their values as json
@mbabker
Copy link
Contributor

mbabker commented Dec 8, 2017

Duplicate of #19025 (if that PR is accepted eventually things will merge up into 4.0, no need to submit the same fix against both versions as long as we are still branch merging).

@mbabker mbabker closed this Dec 8, 2017
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

3 participants