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

Add support for CMB2 custom fields #1166

Closed
DevinWalker opened this issue Oct 28, 2016 · 4 comments
Closed

Add support for CMB2 custom fields #1166

DevinWalker opened this issue Oct 28, 2016 · 4 comments
Assignees
Milestone

Comments

@DevinWalker
Copy link
Member

DevinWalker commented Oct 28, 2016

Issue Overview

When I was doing work on Zapier and Recurring I noticed our nice welcome texts are missing in the setting tabs:

Branch 1.7: Nice welcomes :)

2016-10-27_17-26-19
2016-10-27_17-26-10

Branch 1.8: No welcomes :(

2016-10-27_17-18-02
2016-10-27_17-17-53

@DevinWalker DevinWalker added this to the 1.8 milestone Oct 28, 2016
@DevinWalker
Copy link
Member Author

@ravinderk I know you're not able to reproduce...from debugging I was able to narrow it down to get_filtered_addon_settings: https://github.com/WordImpress/Give/blob/release/1.8/includes/admin/settings/class-settings-cmb2-backward-compatibility.php#L259

When the $settings for in the fields look OK, but then nothing is output. I'd like to dig into why this is happening for Zapier. I can't find any difference between the other Add-ons in how it registers it's settings.

@ravinderk
Copy link
Collaborator

ravinderk commented Dec 7, 2016

@DevinWalker CMB2 backward compatibility breaks because WordPress updated there hook system in version 4.7.

[07-Dec-2016 14:44:35 UTC] PHP Notice:  Undefined index: function in wp-content/plugins/Give/includes/admin/settings/class-settings-cmb2-backward-compatibility.php on line 509

For ref: https://make.wordpress.org/core/2016/09/08/wp_hook-next-generation-actions-and-filters/

@DevinWalker
Copy link
Member Author

@ravinderk it's still not working for me :(

2016-12-07_10-51-49

@DevinWalker
Copy link
Member Author

I just reviewed again and my Add-on settings pages appear working nicely albeit minor CSS that needs to be addressed with Zapier:

2016-12-28_20-14-39

Closing.

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

No branches or pull requests

2 participants