Debug: Add logging to YouTube Facade filter to diagnose issue
This commit is contained in:
@@ -32,23 +32,35 @@ final class YoutubeFacadeContentFilter
|
|||||||
*/
|
*/
|
||||||
public function filter(string $content): string
|
public function filter(string $content): string
|
||||||
{
|
{
|
||||||
|
// DEBUG: Log that filter is being called
|
||||||
|
error_log('YoutubeFacade: filter() called, content length: ' . strlen($content));
|
||||||
|
|
||||||
// Check if content has YouTube embeds at all (quick check)
|
// Check if content has YouTube embeds at all (quick check)
|
||||||
if (strpos($content, 'youtube.com/embed/') === false && strpos($content, 'youtube-nocookie.com/embed/') === false) {
|
if (strpos($content, 'youtube.com/embed/') === false && strpos($content, 'youtube-nocookie.com/embed/') === false) {
|
||||||
|
error_log('YoutubeFacade: No YouTube embeds found in quick check');
|
||||||
return $content;
|
return $content;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
error_log('YoutubeFacade: YouTube embed detected, applying regex');
|
||||||
|
|
||||||
// Pattern to match YouTube iframes
|
// Pattern to match YouTube iframes
|
||||||
// Handles: youtube.com/embed/ and youtube-nocookie.com/embed/
|
// Handles: youtube.com/embed/ and youtube-nocookie.com/embed/
|
||||||
// More flexible: allows any attributes in any order, whitespace variations
|
// More flexible: allows any attributes in any order, whitespace variations
|
||||||
$pattern = '/<iframe\s+[^>]*src=["\']https?:\/\/(?:www\.)?(?:youtube\.com|youtube-nocookie\.com)\/embed\/([a-zA-Z0-9_-]+)[^"\']*["\'][^>]*>\s*<\/iframe>/is';
|
$pattern = '/<iframe\s+[^>]*src=["\']https?:\/\/(?:www\.)?(?:youtube\.com|youtube-nocookie\.com)\/embed\/([a-zA-Z0-9_-]+)[^"\']*["\'][^>]*>\s*<\/iframe>/is';
|
||||||
|
|
||||||
$result = preg_replace_callback($pattern, function ($matches) {
|
$matchCount = 0;
|
||||||
|
$result = preg_replace_callback($pattern, function ($matches) use (&$matchCount) {
|
||||||
|
$matchCount++;
|
||||||
$videoId = $matches[1];
|
$videoId = $matches[1];
|
||||||
|
error_log('YoutubeFacade: Matched video ID: ' . $videoId);
|
||||||
return $this->renderer->render($videoId);
|
return $this->renderer->render($videoId);
|
||||||
}, $content);
|
}, $content);
|
||||||
|
|
||||||
|
error_log('YoutubeFacade: Regex replaced ' . $matchCount . ' iframes');
|
||||||
|
|
||||||
// Return original content if regex failed
|
// Return original content if regex failed
|
||||||
if ($result === null) {
|
if ($result === null) {
|
||||||
|
error_log('YoutubeFacade: preg_replace_callback returned null (regex error)');
|
||||||
return $content;
|
return $content;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user