Malin
37cf714058
✅ Fixed bulk conversion getting stuck on missing files ✅ Added robust error handling and timeout protection ✅ Improved JavaScript response parsing ✅ Added file existence validation ✅ Fixed missing PHP class imports ✅ Added comprehensive try-catch error recovery 🔧 Key fixes: - File existence checks before conversion attempts - 30-second timeout protection per file - Graceful handling of 500 errors and JSON parsing issues - Automatic continuation to next file on failures - Cache busting for JavaScript updates 🎯 Result: Bulk conversion now completes successfully even with missing files 🚀 Generated with Claude Code (https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
78 lines
2.7 KiB
PHP
78 lines
2.7 KiB
PHP
<?php
|
|
|
|
if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
|
|
|
|
function webPExpressAlterHtml($content) {
|
|
// Don't do anything with the RSS feed.
|
|
if (is_feed()) {
|
|
return $content;
|
|
}
|
|
|
|
if (get_option('webp-express-alter-html-replacement') == 'picture') {
|
|
if(function_exists('is_amp_endpoint') && is_amp_endpoint()) {
|
|
//for AMP pages the <picture> tag is not allowed
|
|
return $content;
|
|
}
|
|
/*if (is_admin() ) {
|
|
return $content;
|
|
}*/
|
|
require_once __DIR__ . '/classes/AlterHtmlPicture.php';
|
|
return \WebPExpress\AlterHtmlPicture::alter($content);
|
|
} else {
|
|
require_once __DIR__ . '/classes/AlterHtmlImageUrls.php';
|
|
return \WebPExpress\AlterHtmlImageUrls::alter($content);
|
|
|
|
}
|
|
}
|
|
|
|
function webpExpressOutputBuffer() {
|
|
if (!is_admin() || (function_exists("wp_doing_ajax") && wp_doing_ajax()) || (defined( 'DOING_AJAX' ) && DOING_AJAX)) {
|
|
ob_start('webPExpressAlterHtml');
|
|
}
|
|
}
|
|
|
|
function webpExpressAddPictureJs() {
|
|
// Don't do anything with the RSS feed.
|
|
// - and no need for PictureJs in the admin
|
|
if ( is_feed() || is_admin() ) { return; }
|
|
|
|
echo '<script>'
|
|
. 'document.createElement( "picture" );'
|
|
. 'if(!window.HTMLPictureElement && document.addEventListener) {'
|
|
. 'window.addEventListener("DOMContentLoaded", function() {'
|
|
. 'var s = document.createElement("script");'
|
|
. 's.src = "' . plugins_url('/js/picturefill.min.js', __FILE__) . '";'
|
|
. 'document.body.appendChild(s);'
|
|
. '});'
|
|
. '}'
|
|
. '</script>';
|
|
}
|
|
|
|
|
|
if (get_option('webp-express-alter-html-replacement') == 'picture') {
|
|
add_action( 'wp_head', 'webpExpressAddPictureJs');
|
|
}
|
|
|
|
if (get_option('webp-express-alter-html-hooks', 'ob') == 'ob') {
|
|
/* TODO:
|
|
Which hook should we use, and should we make it optional?
|
|
- Cache enabler uses 'template_redirect'
|
|
- ShortPixes uses 'init'
|
|
|
|
We go with template_redirect now, because it is the "innermost".
|
|
This lowers the risk of problems with plugins used rewriting URLs to point to CDN.
|
|
(We need to process the output *before* the other plugin has rewritten the URLs,
|
|
if the "Only for webps that exists" feature is enabled)
|
|
*/
|
|
add_action( 'init', 'webpExpressOutputBuffer', 1 );
|
|
//add_action( 'template_redirect', 'webpExpressOutputBuffer', 1 );
|
|
|
|
} else {
|
|
add_filter( 'the_content', 'webPExpressAlterHtml', 10000 ); // priority big, so it will be executed last
|
|
add_filter( 'the_excerpt', 'webPExpressAlterHtml', 10000 );
|
|
add_filter( 'post_thumbnail_html', 'webPExpressAlterHtml');
|
|
}
|
|
|
|
//echo wp_doing_ajax() ? 'ajax' : 'no ajax'; exit;
|
|
//echo is_feed() ? 'feed' : 'no feed'; exit;
|