Fix WPML clearing all languages cache

[wordpressfan]

Description

Here we introduced a new filter that runs after clearing cache and used in preload as in the issue's grooming.

Fixes #6214

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue).
• Enhancement (non-breaking change which improves an existing functionality).

Is the solution different from the one proposed during the grooming?

Exactly as it's groomed.

Checklists

Generic development checklist

• My code follows the style guidelines of this project, with adapted comments and without new warnings.
• I have added unit and integration tests that prove my fix is effective or that my feature works.
• The CI passes locally with my changes (including unit tests, integration tests, linter).
• Any dependent changes have been merged and published in downstream modules.
• If applicable, I have made corresponding changes to the documentation. Provide a link to the documentation.

Test summary

• I triggered all changed lines of code at least once without new errors/warnings/notices.
• I validated all Acceptance Criteria of the related issues. (If applicable, provide proof).
• I validated all test plan the QA Review asked me to.

If not, detail what you could not test.

Please describe any additional tests you performed.

[nicomollet]

The doc to fix?

[Tabrisrp]

Did you check if there was other instances of callbacks on after_rocket_clean_domain that could be replaced to optimize this?

[wordpressfan]

Did you check if there was other instances of callbacks on after_rocket_clean_domain that could be replaced to optimize this?

That's a good point, I saw some cases but I wasn't sure if we need to do this here and overload this PR or not, but I believe u r correct, let me check them and see.

[Mai-Saad]

@wordpressfan Thanks for the PR. A request is done once now when changing settings, full cache clear from admin, clearing used CSS, changing permalinks (assuming that all will trigger this function

wp-rocket/inc/Engine/Preload/Subscriber.php

Line 285 in b67069f

set_transient( 'wpr_preload_running', true );

). However, when we switch theme can see many calls to the same function, can you please check 🙏

[wordpressfan]

When we switch theme we are hooking rocket_clean_domain callback with three different hooks in the following locations:

First location:

wp-rocket/inc/common/purge.php

Line 16 in afef3b6

add_action( 'update_option_theme_mods_' . get_option( 'stylesheet' ), 'rocket_clean_domain' ); // When location of a menu is updated.

Second location:

wp-rocket/inc/common/purge.php

Line 8 in afef3b6

add_action( 'update_option_sidebars_widgets', 'rocket_clean_domain' ); // When you change the order of widgets.

Third and main location:

wp-rocket/inc/common/purge.php

Line 6 in afef3b6

add_action( 'switch_theme', 'rocket_clean_domain' ); // When user change theme.

I believe this is another issue.

cc @piotrbak

[piotrbak]

@Mai-Saad @wordpressfan Yes, this is different issue. Let's go with separate issue for this one

[Mai-Saad]

Working as expected then. (Note, the behavior when switch theme is the same even without WPML)

[wordpressfan]

The other issue is created here: #6276