From 31b4508b26bd1f97154a78d1c8086f46c3cc4214 Mon Sep 17 00:00:00 2001
From: Osman Bozdag <obozdag@gmail.com>
Date: Mon, 2 Sep 2024 06:46:15 +0200
Subject: [PATCH 1/9] docs: Fix some function definitions in
 helpers/html_helper.rst

Fix some function definitions in helpers/html_helper.rst
---
 user_guide_src/source/helpers/html_helper.rst | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/user_guide_src/source/helpers/html_helper.rst b/user_guide_src/source/helpers/html_helper.rst
index b16bba0eb933..6573f48469b9 100644
--- a/user_guide_src/source/helpers/html_helper.rst
+++ b/user_guide_src/source/helpers/html_helper.rst
@@ -53,7 +53,7 @@ The following functions are available:
 
     .. literalinclude:: html_helper/004.php
 
-.. php:function:: img_data([$src = ''[, $indexPage = false[, $attributes = '']]])
+.. php:function:: img_data($path[, $mime = null])
 
     :param string $path: Path to the image file
     :param string|null $mime: MIME type to use, or null to guess
@@ -261,7 +261,7 @@ The following functions are available:
 
     Identical to :php:func:`video()`, only it produces ``<audio>`` element instead of ``<video>``.
 
-.. php:function:: source($src = ''[, $type = false[, $attributes = '']])
+.. php:function:: source($src[, $type = false[, $attributes = '']])
 
     :param  string  $src:        The path of the media resource
     :param  bool    $type:       The MIME-type of the resource with optional codecs parameters
@@ -288,7 +288,7 @@ The following functions are available:
 
     .. literalinclude:: html_helper/016.php
 
-.. php:function:: object($data = ''[, $type = false[, $attributes = '']])
+.. php:function:: object($data[, $type = 'unknown'[, $attributes = ''[, $params = [][, $indexPage = false]]]])
 
     :param  string  $data:       A resource URL
     :param  bool    $type:       Content-type of the resource
@@ -313,7 +313,7 @@ The following functions are available:
           <param name="hello" type="ref" value="world" class="test" />
         </object>
 
-.. php:function:: param($name = ''[, $type = false[, $attributes = '']])
+.. php:function:: param($name = ''[, $value = ''[, $type = 'ref'[, $attributes = '']]])
 
     :param  string  $name:       The name of the parameter
     :param  string  $value:      The value of the parameter

From de35cf487eab64e53d5e268504ee3b15da3a7921 Mon Sep 17 00:00:00 2001
From: Osman Bozdag <obozdag@gmail.com>
Date: Mon, 2 Sep 2024 06:59:31 +0200
Subject: [PATCH 2/9] docs: Fix track function definition in
 helpers/html_helper.rst

Fix track function definition in helpers/html_helper.rst
---
 user_guide_src/source/helpers/html_helper.rst | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/user_guide_src/source/helpers/html_helper.rst b/user_guide_src/source/helpers/html_helper.rst
index b16bba0eb933..473a446ff42d 100644
--- a/user_guide_src/source/helpers/html_helper.rst
+++ b/user_guide_src/source/helpers/html_helper.rst
@@ -326,11 +326,12 @@ The following functions are available:
 
     .. literalinclude:: html_helper/018.php
 
-.. php:function:: track($name = ''[, $type = false[, $attributes = '']])
+.. php:function:: track($src, $kind, $srcLanguage, $label)
 
-    :param  string  $name:       The name of the parameter
-    :param  string  $value:      The value of the parameter
-    :param  array   $attributes: HTML attributes
+    :param  string $src:         The path of the media resource
+    :param  string $kind:        The kind of timed track
+    :param  string $srcLanguage: The language of the timed track
+    :param  string $label:       A user-readable title for the timed track
     :returns:   An HTML track element
     :rtype: string
 

From 3d310cf0bff18d327fa60e60463964a5857bea39 Mon Sep 17 00:00:00 2001
From: kenjis <kenji.uui@gmail.com>
Date: Thu, 5 Sep 2024 16:40:00 +0900
Subject: [PATCH 3/9] test: add tests for auto_link()

---
 tests/system/Helpers/URLHelper/MiscUrlTest.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tests/system/Helpers/URLHelper/MiscUrlTest.php b/tests/system/Helpers/URLHelper/MiscUrlTest.php
index 125356e7d0ed..219b5322dbb5 100644
--- a/tests/system/Helpers/URLHelper/MiscUrlTest.php
+++ b/tests/system/Helpers/URLHelper/MiscUrlTest.php
@@ -533,6 +533,10 @@ public static function provideAutoLinkUrl(): iterable
                 'Visit www.example.com or email foo@bar.com',
                 'Visit <a href="http://www.example.com">www.example.com</a> or email foo@bar.com',
             ],
+            'trailing slash' => [
+                'Trailing slash: https://codeigniter.com/ fubar',
+                'Trailing slash: <a href="https://codeigniter.com/">https://codeigniter.com/</a> fubar',
+            ],
         ];
     }
 
@@ -629,6 +633,10 @@ public static function provideAutolinkBoth(): iterable
                 'Visit www.example.com or email foo@bar.com',
                 "Visit <a href=\"http://www.example.com\">www.example.com</a> or email <script>var l=new Array();l[0] = '>';l[1] = 'a';l[2] = '/';l[3] = '<';l[4] = '|109';l[5] = '|111';l[6] = '|99';l[7] = '|46';l[8] = '|114';l[9] = '|97';l[10] = '|98';l[11] = '|64';l[12] = '|111';l[13] = '|111';l[14] = '|102';l[15] = '>';l[16] = '\"';l[17] = '|109';l[18] = '|111';l[19] = '|99';l[20] = '|46';l[21] = '|114';l[22] = '|97';l[23] = '|98';l[24] = '|64';l[25] = '|111';l[26] = '|111';l[27] = '|102';l[28] = ':';l[29] = 'o';l[30] = 't';l[31] = 'l';l[32] = 'i';l[33] = 'a';l[34] = 'm';l[35] = '\"';l[36] = '=';l[37] = 'f';l[38] = 'e';l[39] = 'r';l[40] = 'h';l[41] = ' ';l[42] = 'a';l[43] = '<';for (var i = l.length-1; i >= 0; i=i-1) {if (l[i].substring(0, 1) === '|') document.write(\"&#\"+unescape(l[i].substring(1))+\";\");else document.write(unescape(l[i]));}</script>",
             ],
+            'email starting with "www."' => [
+                'this is some text that includes www.email@domain.com which is causing an issue',
+                'this is some text that includes ' . safe_mailto('www.email@domain.com') . ' which is causing an issue',
+            ],
         ];
     }
 

From d7da0b8cef90a11a61dd62f248042f445bbc3b38 Mon Sep 17 00:00:00 2001
From: kenjis <kenji.uui@gmail.com>
Date: Thu, 5 Sep 2024 16:40:33 +0900
Subject: [PATCH 4/9] fix: auto_link() regexp

From CI3.
See https://github.com/bcit-ci/CodeIgniter/blob/2042929bab8ffc14faf6193de0107b7f95abbdaf/system/helpers/url_helper.php#L397
---
 system/Helpers/url_helper.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/system/Helpers/url_helper.php b/system/Helpers/url_helper.php
index 1d03132d82cf..d68fea2aad7d 100644
--- a/system/Helpers/url_helper.php
+++ b/system/Helpers/url_helper.php
@@ -351,7 +351,7 @@ function safe_mailto(string $email, string $title = '', $attributes = ''): strin
     function auto_link(string $str, string $type = 'both', bool $popup = false): string
     {
         // Find and replace any URLs.
-        if ($type !== 'email' && preg_match_all('#(\w*://|www\.)[^\s()<>;]+\w#i', $str, $matches, PREG_OFFSET_CAPTURE | PREG_SET_ORDER)) {
+        if ($type !== 'email' && preg_match_all('#(\w*://|www\.)[a-z0-9]+(-+[a-z0-9]+)*(\.[a-z0-9]+(-+[a-z0-9]+)*)+(/([^\s()<>;]+\w)?/?)?#i', $str, $matches, PREG_OFFSET_CAPTURE | PREG_SET_ORDER)) {
             // Set our target HTML if using popup links.
             $target = ($popup) ? ' target="_blank"' : '';
 

From fdba50299c35fe29a2a25a7521b009a8f094b17f Mon Sep 17 00:00:00 2001
From: kenjis <kenji.uui@gmail.com>
Date: Fri, 6 Sep 2024 10:33:06 +0900
Subject: [PATCH 5/9] docs: add changelog

---
 user_guide_src/source/changelogs/v4.5.5.rst | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/user_guide_src/source/changelogs/v4.5.5.rst b/user_guide_src/source/changelogs/v4.5.5.rst
index f6799e484ebf..aaea324399af 100644
--- a/user_guide_src/source/changelogs/v4.5.5.rst
+++ b/user_guide_src/source/changelogs/v4.5.5.rst
@@ -30,6 +30,10 @@ Deprecations
 Bugs Fixed
 **********
 
+- **URL Helper:** The bug where the regular expression in :php:func:`auto_link()`
+  was old has been fixed. As a result of this fix, the same regular expression as
+  CodeIgniter 3 is now used.
+
 See the repo's
 `CHANGELOG.md <https://github.com/codeigniter4/CodeIgniter4/blob/develop/CHANGELOG.md>`_
 for a complete list of bugs fixed.

From 9ac45a1d645d3d4868fd686534c6aa13abcc2451 Mon Sep 17 00:00:00 2001
From: Shayo Rowland <127336696+eightrix@users.noreply.github.com>
Date: Fri, 6 Sep 2024 07:05:20 +0100
Subject: [PATCH 6/9] Update validation.rst

Insert 'are' after 'that' on line 1053
---
 user_guide_src/source/libraries/validation.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/user_guide_src/source/libraries/validation.rst b/user_guide_src/source/libraries/validation.rst
index 048449580422..2d3b75297ce0 100644
--- a/user_guide_src/source/libraries/validation.rst
+++ b/user_guide_src/source/libraries/validation.rst
@@ -1050,7 +1050,7 @@ Rules for File Uploads
 When you validate uploaded files, you must use the rules specifically created for
 file validation.
 
-.. important:: Only rules that listed in the table below can be used to validate
+.. important:: Only rules that are listed in the table below can be used to validate
     files. Therefore, adding any general rules, like ``permit_empty``, to file
     validation rules array or string, the file validation will not work correctly.
 

From e81e8e7f0e7f76c5b94ca843e14bf570ce7bc020 Mon Sep 17 00:00:00 2001
From: Osman Bozdag <obozdag@gmail.com>
Date: Fri, 6 Sep 2024 09:51:33 +0200
Subject: [PATCH 7/9] docs: Fix track function src param definition in
 helpers/html_helper.rst

Fix track function src param definition in helpers/html_helper.rst
---
 user_guide_src/source/helpers/html_helper.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/user_guide_src/source/helpers/html_helper.rst b/user_guide_src/source/helpers/html_helper.rst
index 473a446ff42d..aa579307eba9 100644
--- a/user_guide_src/source/helpers/html_helper.rst
+++ b/user_guide_src/source/helpers/html_helper.rst
@@ -328,7 +328,7 @@ The following functions are available:
 
 .. php:function:: track($src, $kind, $srcLanguage, $label)
 
-    :param  string $src:         The path of the media resource
+    :param  string $src:         The path of the track (.vtt file)
     :param  string $kind:        The kind of timed track
     :param  string $srcLanguage: The language of the timed track
     :param  string $label:       A user-readable title for the timed track

From 1e868d035a4b5d101de7af0aaaa3d746f16f0c11 Mon Sep 17 00:00:00 2001
From: Osman Bozdag <obozdag@gmail.com>
Date: Fri, 6 Sep 2024 11:23:44 +0200
Subject: [PATCH 8/9] docs: Fix param function definition in
 helpers/html_helper.rst

Co-authored-by: kenjis <kenji.uui@gmail.com>
---
 user_guide_src/source/helpers/html_helper.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/user_guide_src/source/helpers/html_helper.rst b/user_guide_src/source/helpers/html_helper.rst
index 6573f48469b9..10d7fb89a7f4 100644
--- a/user_guide_src/source/helpers/html_helper.rst
+++ b/user_guide_src/source/helpers/html_helper.rst
@@ -313,7 +313,7 @@ The following functions are available:
           <param name="hello" type="ref" value="world" class="test" />
         </object>
 
-.. php:function:: param($name = ''[, $value = ''[, $type = 'ref'[, $attributes = '']]])
+.. php:function:: param($name, $value[, $type = 'ref'[, $attributes = '']])
 
     :param  string  $name:       The name of the parameter
     :param  string  $value:      The value of the parameter

From 8e09849c27988280b668f9f43dd58084d08aacc3 Mon Sep 17 00:00:00 2001
From: Osman Bozdag <obozdag@gmail.com>
Date: Fri, 6 Sep 2024 11:24:37 +0200
Subject: [PATCH 9/9] docs: Fix source function definition in
 helpers/html_helper.rst

Co-authored-by: kenjis <kenji.uui@gmail.com>
---
 user_guide_src/source/helpers/html_helper.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/user_guide_src/source/helpers/html_helper.rst b/user_guide_src/source/helpers/html_helper.rst
index 10d7fb89a7f4..ac781f5b8523 100644
--- a/user_guide_src/source/helpers/html_helper.rst
+++ b/user_guide_src/source/helpers/html_helper.rst
@@ -261,7 +261,7 @@ The following functions are available:
 
     Identical to :php:func:`video()`, only it produces ``<audio>`` element instead of ``<video>``.
 
-.. php:function:: source($src[, $type = false[, $attributes = '']])
+.. php:function:: source($src, $type = 'unknown', $attributes = '', $indexPage = false)
 
     :param  string  $src:        The path of the media resource
     :param  bool    $type:       The MIME-type of the resource with optional codecs parameters