Austin Ginder

I spent some time troubleshooting on the staging site, here is what I found:

• Bug only happens when updating to WordPress 5.6.
• Bug is a conflict with Popup Maker v1.13.1 and Search & Filter Pro v2.5.2.
• Both plugins are up to date and running the latest versions.
• Only happens when a popup is shown. If you’ve already dismissed then you won’t see the bug.

As a workaround I enabled an option within Search & Filter Pro called Lazy Load JavaScript. That means Search & Filter JavaScript files are only loaded on the pages that contain search forms which helps reduce the conflict with the popups. Will start a ticket with authors of Search & Filter Pro with my findings. Just wanted to post these details in case anyone else was running into the same conflict. I’ll consider my original request solved as it doesn’t seem to be problem with Popup Maker itself.

The plugin author says the following:

Everyone,
The WordPress 5.6 release included major changes. We’ve found number of issues that affected Responsive Lightbox & Gallery and working on them one by one to prepare a complete bugfix release.
If you can, please hold on couple of days before a 5.6 upgrade. We plan to release an updated RLG by the end of the week.
Sorry for any inconveniences – we’re doing our best.

https://dfactory.eu/support/topic/responsive-lightbox-new-conflict-with-wordpress-menus/#post-42477

Thanks for the followup. I can confirm that I was able to update to v3.10.4 and it’s working!

Will do. Thanks!

I did some troubleshooting on the staging site. I’m thinking it’s a conflict with my theme WpResidence v3.3.2 and WPBakery Page Builder v6.3.0. I tried FlexMLS IDX on v3.10.3, still getting PHP errors as shown below. Downgrading FlexMLS IDX to v3.9.1 seems to resolve.

An error of type E_ERROR was caused in line 317 of the file /wp-content/plugins/flexmls-idx/components/search-results.php. Error message: Uncaught Error: Only variables can be passed by reference in /wp-content/plugins/flexmls-idx/components/search-results.php:317
Stack trace:
#0 /wp-content/plugins/flexmls-idx/integration/wpbakery/components/VCE_component.php(27): fmcSearchResults->integration_view_vars()
#1 /wp-includes/class-wp-hook.php(287): VCE_component->integrateWithVC('')
#2 /wp-includes/class-wp-hook.php(311): WP_Hook->apply_filters(NULL, Array)
#3 /wp-includes/plugin.php(478): WP_Hook->do_action(Array)
#4 /wp-settings.php(546): do_action('init')
#5 /wp-config.php(39): require_once('/...')
#6 /wp-load.php(37): require_once('/...')
#7 /wp-admin/admin.php(34): require_once('/...')

I’ll have to play around with my site to see if I can pinpoint a pattern. Thanks for the followup!

Great, thanks for the followup. I can confirm it’s working better since the update however it still appears to be double sending ever time I click schedule. At least now it’s not sending more than 2 copies.

Same error when updating. It broke the backend of the website. Downgrade to v3.3.2 resolved the issue.

I am, WordPress v5.2 and BNFW v1.7.3.

Ah okay, sounds good. Yes a patch fix for the repeating emails would be great!

The second. I’m writing a post and as soon as I hit schedule multiple emails go out. Sometimes it’s 2 duplicates sometimes 3 or 4. On a development copy of the site I’ve narrowed it down to the new block editor. I’ve confirmed this behavior goes away if I install the classic editor plugin.

Likewise I’m receiving the following error with v3.3. Downgrading works as a workaround.

PHP Parse error: syntax error, unexpected 'private' (T_PRIVATE) in my-custom-css/pages/myphp/Edit.php on line 257" while reading response header from upstream

• This reply was modified 5 years, 10 months ago by Austin Ginder.

I likewise specifically used MonsterInsights with manual UA codes. It looks like they removed that GUI with their update. I may switch to use a performance plugin named Perfmatters. The main advantage is Perfmatters will serve analytics.js locally thus removing an extra lookup as explained here: https://perfmatters.io/docs/local-analytics/. As a bonus Perfmatters actually integrates with MonsterInsights.

I’m using a child theme so this was safely placed within it’s function.php file. Thanks for checking.

Doesn’t looks like using the <style> tag is recommended for emails. I think the css needs to be inline in order to work across most email clients as shown here: https://stackoverflow.com/questions/29391855/how-to-make-an-image-responsive-in-html-email-regardless-of-image-size.

After playing around with some of the filters I think I got bnfw_notification_message working as shown here.

/**
 * Responsive html email images for BNFW.
 */
function my_bnfw_responsive_html_images( $message, $setting ) {
	$message = str_replace( "<img src=\"", "<img style=\"width: 100% !important;\" src=\"", $message );
	return $message;
}

add_filter( 'bnfw_notification_message', 'my_bnfw_responsive_html_images', 10, 2 );

• This reply was modified 6 years, 3 months ago by Austin Ginder.