From 61151cc5c2e77f29668834154bf9281cdd92fa53 Mon Sep 17 00:00:00 2001
From: sakaguchi <sakaguchi@catchup.co.jp>
Date: Tue, 2 Jun 2026 22:13:38 +0900
Subject: [PATCH] =?UTF-8?q?fix=20#4001=20=E3=83=96=E3=83=AD=E3=82=B0?=
 =?UTF-8?q?=E3=82=B3=E3=83=A1=E3=83=B3=E3=83=88=E7=AE=A1=E7=90=86=EF=BC=9A?=
 =?UTF-8?q?=E6=96=87=E5=AD=97=E8=89=B2=E3=81=AE=E7=9B=B8=E9=81=95=E3=81=8C?=
 =?UTF-8?q?=E3=81=82=E3=82=8B=E4=BB=B6=E3=82=92=E4=BF=AE=E6=AD=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/View/Helper/BcTextHelper.php          | 51 +++++++++++++++++++
 .../TestCase/View/Helper/BcTextHelperTest.php | 18 +++++++
 2 files changed, 69 insertions(+)

diff --git a/plugins/baser-core/src/View/Helper/BcTextHelper.php b/plugins/baser-core/src/View/Helper/BcTextHelper.php
index f8c91ee050..eb3182017e 100644
--- a/plugins/baser-core/src/View/Helper/BcTextHelper.php
+++ b/plugins/baser-core/src/View/Helper/BcTextHelper.php
@@ -475,4 +475,55 @@ public function booleanStatus($value)
     }
 // <<<
 
+    /**
+     * テキスト内のURLを自動でリンク化
+     *
+     * CakePHP本体の autoLinkUrls() では、ドメインのみURLの末尾スラッシュが
+     * リンク外に出るケースがあるため、リンク直後のスラッシュだけ補正する。
+     *
+     * @param string $text テキスト
+     * @param array<string, mixed> $options リンクオプション
+     * @return string リンク化後テキスト
+     * @checked
+     * @noTodo
+     * @unitTest
+     */
+    public function autoLinkUrls(string $text, array $options = []): string
+    {
+        $linkedText = parent::autoLinkUrls($text, $options);
+
+        return $this->fixTrailingSlashOutsideAnchor($linkedText);
+    }
+
+    /**
+     * ドメインのみURLで末尾スラッシュがリンク外に出る表示崩れを補正
+     *
+     * @param string $text リンク化後テキスト
+     * @return string 補正後テキスト
+     */
+    protected function fixTrailingSlashOutsideAnchor(string $text): string
+    {
+        // 壊れたパターンだけを対象に補正する。
+        // 例: <a href="https://example.com">https://example.com</a>/
+        //
+        // (?<before>) : href 属性より前の属性群（class, target など）
+        // (?<href>)   : href 値（スキーム + ドメインのみ。末尾スラッシュなし）
+        // (?<after>)  : href 属性より後の属性群
+        // (?<label>)  : リンク表示文字列（スキーム + ドメインのみ）
+        //
+        // 末尾の / は </a> の外側にあるものだけを拾い、直後が空白・タグ開始・行末の場合に限定する。
+        // パス付きURLや通常のリンクには触れず、副作用を避ける。
+        $result = preg_replace_callback(
+            '/<a\\b(?<before>[^>]*?)href="(?<href>(?:https?|ftp|nntp):\\/\\/[^"\\/\\s<]+)"(?<after>[^>]*)>(?<label>(?:https?|ftp|nntp):\\/\\/[^"\\/\\s<]+)<\\/a>\\/(?=[\\s<]|$)/u',
+            static function (array $matches): string {
+                $href = $matches['href'] . '/';
+                $label = $matches['label'] . '/';
+
+                return '<a' . $matches['before'] . 'href="' . $href . '"' . $matches['after'] . '>' . $label . '</a>';
+            },
+            $text
+        );
+
+        return $result ?? $text;
+    }
 }
diff --git a/plugins/baser-core/tests/TestCase/View/Helper/BcTextHelperTest.php b/plugins/baser-core/tests/TestCase/View/Helper/BcTextHelperTest.php
index 6cecf8afc1..820aabdb72 100644
--- a/plugins/baser-core/tests/TestCase/View/Helper/BcTextHelperTest.php
+++ b/plugins/baser-core/tests/TestCase/View/Helper/BcTextHelperTest.php
@@ -488,4 +488,22 @@ public function testBooleanStatus()
 
     }
 
+    /**
+     * autoLinkUrls の末尾スラッシュ補正のテスト
+     */
+    public function testAutoLinkUrlsTrailingSlash()
+    {
+        $result = $this->Helper->autoLinkUrls('https://example.com/');
+        $this->assertMatchesRegularExpression('/href=[\'\"]https:\/\/example\\.com\/[\'\"]/', $result);
+        $this->assertStringContainsString('>https://example.com/</a>', $result);
+        $this->assertStringNotContainsString('</a>/', $result);
+
+        $resultWithPath = $this->Helper->autoLinkUrls('https://example.com/path/');
+        $this->assertStringContainsString('>https://example.com/path/</a>', $resultWithPath);
+
+        $resultWithText = $this->Helper->autoLinkUrls('URL: https://example.com/ end');
+        $this->assertStringContainsString('>https://example.com/</a> end', $resultWithText);
+        $this->assertStringNotContainsString('</a>/ end', $resultWithText);
+    }
+
 }