Ad Butler Widget not working when being placed by page builder

Hey ??

With regards to the regular widget interface (Appearance > Widgets) we found:

For the regular widget interface (Appearance > Widgets) with Page Builder (PB) activated, you just need to refresh the widget interface and the drop down seems to work. So insert, refresh and it works. Disabling Page Builder and our Widgets Bundle plugin seems to produce exactly the same result in Appearance > Widgets. So not sure this is related to Page Builder. Testing in WP 4.3.

With regards to the general Page Builder challenge, both Greg and I spent time on the challenge. In the end we were unable to pinpoint a specific item to be fixed. Beyond the usual Page Builder widget compatibility challenges that several plugins experience:

https://siteorigin.com/docs/page-builder/widget-compatibility/

I have made modifications to the plugin more compatible with WordPress 4.3’s customizer functionality and it is working. Need version 1.11

I have also followed all compatibility requirements for page builder as outlined in the document, but I have been unable to remove the following console error when the widget is added to a page.
https://imgur.com/10U5csn

The JavaScript is minified and I am unable to debug it further.

@sparklit

Thanks for the effort. @gpriday let me know he’s seen that error before but has never been able to re-create. You could use script_debug in wp-config.php to look into the challenge further.

The error is a result of the following code getting tripped up by a wp_nonce_field in the form

var $$ = $(this);
 var name = /([A-Za-z_]+)\[(.*)\]/.exec( $$.attr('name') );
 // Create an array with the parts of the name
 if(typeof name[2] === 'undefined') {

Continuing to work on the widget issue.

Epic, thanks.

Hi sparklit

I actually happened to see that issue come up with fields that have an empty name attribute. I’ve fixed it by checking the value of name.

https://github.com/siteorigin/siteorigin-panels/commit/9115f5b669f852f9116238eeb1ea270029fde85e

Out of interest, is a nonce field required in a widget form? Page Builder itself has nonce protection through the main post form and the widgets interface uses the savewidgets value for nonce checking. So noncing your widget form is unnecessarily doubling up.

Hi everyone,

Added the ability to manually refresh the zones from the widget interface. Version 1.12 should be compatible with Page Builder. In addition after reviewing some of the core code I removed the nonce check within the widget as well.

Thanks for everyones assistance on this and hopefully things are good.

S

That’s a fantastic result, thanks so much for all the effort you’ve put in. It’s most appreciated!