docs: modify the upgrade docs.

codeigniter4 · Oct 24, 2023 · 60f7e69 · 60f7e69
1 parent b413205
commit 60f7e69
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 8 deletions.
diff --git a/user_guide_src/source/changelogs/v4.5.0.rst b/user_guide_src/source/changelogs/v4.5.0.rst
@@ -18,8 +18,6 @@ Highlights
 BREAKING
 ********
 
-- **AutoRouting Legacy:** Add a condition to check if the controller exists.
-
 Behavior Changes
 ================
 
@@ -39,6 +37,8 @@ See :ref:`Upgrading Guide <upgrade-450-nested-route-groups-and-options>` for det
 Others
 ------
 
+- **AutoRouting Legacy:** Changed so that a ``PageNotFoundException`` is thrown
+  if the controller corresponding to the request URI does not exist.
 - **Logger:** The :php:func:`log_message()` function and the logger methods in
   ``CodeIgniter\Log\Logger`` now do not return ``bool`` values. The return types
   have been fixed to ``void`` to follow the PSR-3 interface.

diff --git a/user_guide_src/source/incoming/controllers.rst b/user_guide_src/source/incoming/controllers.rst
@@ -412,7 +412,9 @@ without route definitions. The auto-routing is disabled by default.
 
 .. important:: Auto Routing (Legacy) routes a HTTP request with **any** HTTP method to a controller method.
 
-.. important:: If Auto Routing (Legacy) doesn't find the controller, it will throw page not found exception before the filter executes in version **4.5**.
+.. important:: Since v4.5.0, if Auto Routing (Legacy) doesn't find the controller,
+    it will throw ``PageNotFoundException`` exception before the Controller Filters
+    execute.
 
 Consider this URI::
 

diff --git a/user_guide_src/source/installation/upgrade_450.rst b/user_guide_src/source/installation/upgrade_450.rst
@@ -119,17 +119,24 @@ has been removed.
 
 If you extneds ``BaseModel``, implement the ``getIdValue()`` method in the child class.
 
+Auto Routing (Legacy)
+=====================
+
+In previous versions, the controller filters might be executed even when the
+corresponding controller was not found.
+
+This bug has been fixed and now a ``PageNotFoundException`` will be thrown and
+the filters will not be executed if the controller is not found.
+
+If you have code that depends on this bug, for example if you expect global filters
+to be executed even for non-existent pages, please add the necessary routes.
+
 Removed Deprecated Items
 ========================
 
 Some deprecated items have been removed. If you extend these classes and are
 using them, upgrade your code. See :ref:`ChangeLog <v450-removed-deprecated-items>` for details.
 
-Auto Routing (Legacy) 
-=======================
-
-If the auto routing cannot found the controller, it will throw page not found exception before the filter executed.
-
 Breaking Enhancements
 *********************