Laurent
Forum Replies Created
-
Hello,
First of all, thank you for NS Cloner — it’s a very solid plugin, and we’ve been using it extensively in a WordPress multisite context. Overall, the architecture and the cloning logic are really well thought out.
I’m reaching out regarding a debug notice that has become very visible since WordPress 6.7, especially on multisite installations with logging enabled.
Function _load_textdomain_just_in_time was called incorrectly.
Translation loading for thens-cloner-site-copierdomain was triggered too early.I’ve seen your previous response mentioning that you plan to move things to the
inithook, but that you were unable to reproduce the notice. After some in-depth debugging, I think I understand why it’s hard to replicate, and I wanted to share the findings.Context where the issue appears- WordPress multisite
- NS Cloner activated at network level
- WP_DEBUG_LOG enabled
- Regular navigation in network admin
In this setup, the notice is emitted multiple times per page load (typically 2 to 4 times), which makes the debug log grow very quickly. Even though it’s not functionally blocking, it becomes quite noisy in production logs.What seems to trigger the notice
Leaving aside whether the plugin is intended to be network-only or also usable from sub-sites (I understand there may be Pro features relying on that), the key issue appears to be this:
- Some files required very early by
ns-cloner.php(for example utility files) contain translation calls (__(), etc.) at global scope - These files are loaded before
init, and before WordPress considers translations “safe” to load - On WP 6.7+, this causes WordPress to fall back to just-in-time translation loading, which now logs a notice
Because this happens very early, moving
load_plugin_textdomain()toinitalone does not fully prevent the notice.Why it’s easy to miss- On single-site installs, the notice often doesn’t appear
- Without WP_DEBUG_LOG, it stays invisible
- On multisite, the same early code path may be executed more than once per request, amplifying the effect
Why this matters
Even though the notice is technically harmless, in a multisite / SaaS-like environment it can result in dozens of log entries per minute, simply from normal admin navigation.
At scale, this becomes a maintenance concern rather than a cosmetic one.
I’m sharing this mainly as feedback, not as a complaint — the plugin works well, but WP 6.7 has made these early translation patterns much more visible. It may be worth reviewing global-scope translation calls in early-loaded files to reduce log noise going forward.
On our side, this has also prompted us to reflect on whether a more infrastructure-driven cloning approach (WP-CLI + scripted batches) might be more appropriate long term, but that’s more an architectural consideration than a criticism of NS Cloner itself.
Thanks again for your work on the plugin, and feel free to reach out if you’d like more concrete pointers or examples.
Best regards,
LaurentHi Noruzzaman,
Thanks for the follow-up — and good news: after running new tests this morning, the accessibility warning has completely disappeared. It was probably a cache issue on my side.
However, I’ve noticed a small visual behavior that might be worth checking. When the page first loads, the slider navigation arrows briefly appear for a fraction of a second before disappearing. It’s very quick but visible. (block : base)
This might be related to CSS/JS minification, caching, or the initialization timing of Swiper, but I wanted to mention it just in case it’s something handled by the plugin.
You can see it here (WordPress multisite installation):
👉 https://pageslocales.fr/pl-elecmarseille/Thanks again — the plugin works great overall!
Best,
LaurentSubject: Accessibility issue still present after update (v1.0.10)
Hello,
First, thank you for updating the plugin — I really appreciate your work.
I’ve just updated to version 1.0.10, cleared all caches (object cache, verification cache, and page cache), and re-tested the site with Google Lighthouse / PageSpeed Insights.However, I still see the same accessibility warning:
Elements use prohibited ARIA attributes <div class="swiper-button-prev" aria-label="Previous slide"> <div class="swiper-button-next" aria-label="Next slide">It looks like the navigation arrows are still using
<div>elements witharia-labelinstead of<button>(or<div role="button" tabindex="0">).Maybe I did something wrong in my testing process, but on my side, the issue still appears exactly as before.
Could you please double-check if the fix was included in version 1.0.10?
Thank you again for your great plugin — it’s a really useful and well-designed tool.
Best regards,
Laurent