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

Version 1.21.5.3 #908

Merged
merged 19 commits into from
Jul 24, 2017
Merged

Version 1.21.5.3 #908

merged 19 commits into from
Jul 24, 2017

Conversation

zackkatz
Copy link
Member

@zackkatz zackkatz commented Jul 24, 2017

┆Issue is synchronized with this Asana task

soulseekah and others added 19 commits July 1, 2017 18:39
Since WordPress 4.8 the preview is generated via a REST API call to the
oembed/proxy endpoint, which in turn does a remote request to a
registered oEmbed provider.

The default oEmbed provider fetches the content of a local URL, which,
for a GravityView entry URL, ends up being the View instead of the
Entry. Register the simplest provider for entry requests and serve the
entry as needed.
If a form is trashed the view that connects to it will fatal. Fix this
quickly.
@jakejackson1 can you apply this and see if it works for your case? Not
sure why it would, but this now uses the `admin_init` hook they show in
the EDDSL sample.

Related to #896
For now, just log an error.

Closes #905
These errors were an edge case, migrating the `{prefix}_options` table,
but it's possible that others could have this affect them, too.

Fixes #907
When saving a custom field, our code will load the value of the saved
field back into the entry. For later versions of GF the value for
checkboxes is an array. So the `gform_field_values` would contain a
really bad encoded string that fails to validate. Saving again kills
at least custom field file uploads.

This fix checks to see whether the value is an array to begin with and
pulls out the needed subvalue.

https://secure.helpscout.net/conversation/394406485/9835/?folderId=408730
I don't recall why we added this to `gv_empty()` in the first
place.…but it doesn't really make much sense.

Thoughts, @soulseekah ?
Saw an empty View ID in the blacklist, might as well fix it.
It was looking bad for smaller screens, the screenshots were huge. Get
rid of the `.gv-welcome-screenshots` class and use core instead.
@zackkatz zackkatz merged commit 15bc191 into master Jul 24, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 18.083% when pulling 8e07abf on develop into c38d6c1 on master.

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