From c6d09e52709decbdc9366cd3885c876dc3643a73 Mon Sep 17 00:00:00 2001
From: Grischa Brockhaus <github@brockha.us>
Date: Mon, 5 Dec 2022 12:46:18 +0100
Subject: [PATCH] Changes added because of good remarks by
 hypolite@friendica.mrpetovan.com

---
 fancybox/asset/fancybox/fancybox.config.js |  1 +
 fancybox/fancybox.php                      | 60 +++++++++-------------
 2 files changed, 26 insertions(+), 35 deletions(-)

diff --git a/fancybox/asset/fancybox/fancybox.config.js b/fancybox/asset/fancybox/fancybox.config.js
index 6ac5d9c5..2b9c4f41 100644
--- a/fancybox/asset/fancybox/fancybox.config.js
+++ b/fancybox/asset/fancybox/fancybox.config.js
@@ -1,4 +1,5 @@
 $(document).ready(function() {
     $.fancybox.defaults.loop = "true";
+    // this disables the colorbox hook found in frio/js/modal.js:34
     $("body").off("click", ".wall-item-body a img");
 });
\ No newline at end of file
diff --git a/fancybox/fancybox.php b/fancybox/fancybox.php
index 1400a75c..43219103 100644
--- a/fancybox/fancybox.php
+++ b/fancybox/fancybox.php
@@ -28,43 +28,33 @@ function fancybox_footer(App $a, string &$str)
 	DI::page()->registerFooterScript(__DIR__ . '/asset/fancybox/fancybox.config.js');
 }
 
-function fancybox_render($a, array &$b){
+function fancybox_render(App $a, array &$b){
 	$gallery = 'gallery-' . $b['item']['uri-id'] ?? random_int(1000000, 10000000);
 
-	// prevent urls in <div class="type-link"> to be replaced
-	$b['html'] = preg_replace_callback(
+	// performWithEscapedBlocks escapes block defined with 2nd par pattern that won't be processed.
+	// We don't want to touch images in class="type-link":
+	$b['html'] = \Friendica\Util\Strings::performWithEscapedBlocks(
+		$b['html'],
 		'#<div class="type-link">.*?</div>#s',
-		function ($matches) use ($gallery) {
-			return str_replace('<a href', '<a data-nofancybox="" href', $matches[0]);
-		},
-		$b['html']
-	);
-
-	// This processes images inlined in posts
-	// Frio / Vier hooks für lightbox are un-hooked in fancybox-config.js. So this works for them, too!
-	//if (!in_array($a->getCurrentTheme(),['vier','frio']))
-	{
-		// normal post inline linked images
-		$b['html'] = preg_replace_callback(
-			'#<a[^>]*href="([^"]*)"[^>]*>(<img[^>]*src="[^"]*"[^>]*>)</a>#',
-			function ($matches)  use ($gallery) {
-				// don't touch URLS marked as not "fancyable".. ;-)
-				if (preg_match('#data-nofancybox#', $matches[0]))
-				{
-					return $matches[0];
-				}
-				return '<a data-fancybox="' . $gallery . '" href="'. $matches[1] .'">' . $matches[2] .'</a>';
-			},
-			$b['html']
-		);
-	}
-
-	// Local content images attached:
-	$b['html'] = preg_replace_callback(
-		'#<div class="body-attach">.*?</div>#s',
-		function ($matches) use ($gallery) {
-			return str_replace('<a href', '<a data-fancybox="' . $gallery . '" href', $matches[0]);
-		},
-		$b['html']
+		function ($text) use ($gallery) {
+			// This processes images inlined in posts
+			// Frio / Vier hooks für lightbox are un-hooked in fancybox-config.js. So this works for them, too!
+			//if (!in_array($a->getCurrentTheme(),['vier','frio']))
+			$text = preg_replace(
+				'#<a[^>]*href="([^"]*)"[^>]*>(<img[^>]*src="[^"]*"[^>]*>)</a>#',
+				'<a data-fancybox="' . $gallery . '" href="$1">$2</a>',
+				$text);
+
+			// Local content images attached:
+			$text = preg_replace_callback(
+				'#<div class="body-attach">.*?</div>#s',
+				function ($matches) use ($gallery) {
+					return str_replace('<a href', '<a data-fancybox="' . $gallery . '" href', $matches[0]);
+				},
+				$text
+			);
+
+			return $text;
+		}
 	);
 }
-- 
2.39.5