Activating Caching Plugin breaks page

Resolved steffiiiii
(@steffiiiii)

5 years, 9 months ago
Hi bizswoop team,

I am using:
WP 5.4.2
WooCommerce 4.0.0
Store Hours Manager for WooCommerce 4.0.10
Theme: dt-the7 9.1.0

The plugin works well, but as soon as I enable a caching plugin the site breaks when the cart is not empty.
I tried the following caching plugins: Autoptimize, WP Fastest Cache, WP Super Cache, WP-Optimize – Clean, Compress, Cache. With all plugins it’s the same problem.

By ‘the site breaks’ I mean the following:
The first 350 lines of code are missing in the source code. No <html> or <head> tag. No css or js includes. No error in php log / debug.log. Everything before this code is missing:
```
<style>
        .zhours_alertbutton {
            color: #ffffff;
            background-color: #fe6002;
            padding: 16px;
            font-size: 16px;
        }
    </style>
```
It seems to work fine, when Store Hours are open/enabled, but breaks if Store Hours are closed/disabled and the first item is added to the cart.

I tracked it down to the following line of code:
includes/Setup.php Line 62 to 67
```
\add_action('woocommerce_after_mini_cart', function () {
  if (!WC()->cart->is_empty()) :
    \ob_end_clean();
  get_alertbutton();
  endif;
});
```
If I comment out ob_end_clean(); it works as expected.
The caching plugins also use ob_start and ob_end_clean() for buffering. This seems to conflict.

What is ob_start and ob_end_clean() necessary for in your plugin? Can I safely delete it?
Could you fix this in your plugin so it is compatible to future updates?

Thanks in advance

Viewing 8 replies - 1 through 8 (of 8 total)

Plugin Author bizswoop
(@bizswoop)

5 years, 9 months ago

Unfortunately, we tried to reproduce the behavior you flagged but we couldn’t recreate the issue, as a result we can’t provide any fix at this time until we can reproduce and identify if any errors are occurring related to Cache plugin behavior.

We tried to reproduce the behavior using two of the Cache plugins you referenced: WP Fastest Cache and WP Super Cache.

We created a test environment and used Automattic’s cache plugin, we wanted to share this video to show our reproduction testing and verifying the behavior working correctly with WP Super Cache, as shown here:
https://www.loom.com/share/951940d124604057ad5ad6bb3bdc1cf2

A few recommendations, we noticed you are running older versions of WordPress and WooCommerce. All our reproduction testing is based upon the latest versions WordPress 5.5 and WooCommerce 4.4.1 and WP Super Cache 1.7.1.

Our first recommendation would be to update and verify you are running the latest versions of these plugins.

For the Store Hours Manager, we’d recommend you deactivate and delete the current version running, if you have identified code missing. Or if you have made code modifications. Download and activate a clean new version from ww.wp.xz.cn. We suspect running the latest versions and clean new install of Store Hours Manager should resolve this issue for you as shown in our reproduction testing efforts.

After you are running the latest versions and using a clean install of Store Hours Manager, if the issue continues, let’s isolate and try to reproduce the behavior using the WP Super Cache plugin. Take a video of the settings you have setup on each tab: Easy, Advance, CDN, Contents, Preload, Plugins and Debug. We will attempt to recreate an environment based upon the settings to understand if a specific setting is causing the issue.

Thread Starter steffiiiii
(@steffiiiii)

5 years, 9 months ago

Thanks for your fast reply and effort to take the video.

The problem seems to be the combination of:
Store Hours Manager
The 7 Theme
The 7 Theme – Microwidget ‘Cart’ in top header

If I disable the microwidget cart, everything works as expected.
So it seems there is an incompatibility between the microwidget cart of The 7 Theme and Store Hours Manager.
Thread Starter steffiiiii
(@steffiiiii)

5 years, 9 months ago
hi bizswoop,

due to the incompatibility I ended up removing those two functions from your plugin:
includes/Setup.php Line 59 to 67
```
\add_action('woocommerce_widget_shopping_cart_before_buttons', function () {
        \ob_start();
});
\add_action('woocommerce_after_mini_cart', function () {
        if (!WC()->cart->is_empty()) :
            \ob_end_clean();
        get_alertbutton();
        endif;
});
```
Any chance you could change those two functions from anonymous functions to named functions? Then I could remove the two actions in my child theme and your plugins stays untouched.

Thanks
Plugin Author bizswoop
(@bizswoop)

5 years, 9 months ago

Great to hear you isolated the conflict.

Yes, we can push a release for new handlers to help resolve the issue and avoid you having to modify the plugin code.

The new handlers:

\Zhours\render_widget_shopping_cart_before_buttons

\Zhours\render_after_mini_cart

Could you post and share on the forum post here the code developed on the child theme? This would allow other community members facing a similar conflict to solve as well. The new release 4.0.11 should post to ww.wp.xz.cn in the next 24 hours. Hopefully that helps!
Thread Starter steffiiiii
(@steffiiiii)

5 years, 9 months ago
Thanks for your quick plugin update!
Unfortunately I overlooked, that the new handlers are in a nested add_action, which is anonymous.
includes/Setup.php Line 39
\add_action('wp', function () {

So I’m afraid I still can’t remove the new actions.
For example, I can easily remove the alertbar in the footer, because it is a single (not nested) action:
remove_action('wp_footer', '\Zhours\get_alertbar');

The main action is still anonymous.
- This reply was modified 5 years, 9 months ago by steffiiiii.
Plugin Author bizswoop
(@bizswoop)

5 years, 9 months ago

You’re welcome. We’re happy to improve the plugin to make things easier for developers and themes.

Check out our new release we pushed version 4.0.12, we pushed a new handler for the action identified as anonymous.

\Zhours\init_checkout_actions

Hopefully this should allow you to manage actions as needed now.
Thread Starter steffiiiii
(@steffiiiii)

5 years, 9 months ago
Hi,

I just did a quick test and it works! I can now remove those two handlers:
```
\Zhours\render_widget_shopping_cart_before_buttons
\Zhours\render_after_mini_cart
```
within my child theme (functions.php) and the Store Hours Manager Plugin stays updateable!

Thank you!
- This reply was modified 5 years, 9 months ago by steffiiiii.
Plugin Author bizswoop
(@bizswoop)

5 years, 9 months ago

Awesome to hear this works now for you. Thanks for the update and sharing the solution with the community, we appreciate it!

If you have a few extra minutes, we’d love if you left a review for the plugin: https://ww.wp.xz.cn/plugins/order-hours-scheduler-for-woocommerce/#reviews

Reviews help motivate us to continue developing for the community knowing it’s value and also shares your experience to other community members evaluating the plugin to use as well! Thank you for the consideration.

Viewing 8 replies - 1 through 8 (of 8 total)

The topic ‘Activating Caching Plugin breaks page’ is closed to new replies.