Skip to content

Commit

Permalink
Merge pull request #8067 from kenjis/refactor-IncomingRequest-uri
Browse files Browse the repository at this point in the history
refactor: make IncomingRequest::$uri protected
  • Loading branch information
kenjis committed Oct 22, 2023
2 parents de282b9 + 9e05289 commit 365e4c1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
8 changes: 2 additions & 6 deletions system/HTTP/IncomingRequest.php
Original file line number Diff line number Diff line change
Expand Up @@ -67,11 +67,9 @@ class IncomingRequest extends Request
* AFTER the baseURL. So, if hosted in a sub-folder this will
* appear different than actual URI path. If you need that use getPath().
*
* @deprecated Will be protected. Use getUri() instead.
*
* @var URI
*/
public $uri;
protected $uri;

/**
* The detected URI path (relative to the baseURL).
Expand Down Expand Up @@ -125,10 +123,8 @@ class IncomingRequest extends Request
* Configuration settings.
*
* @var App
*
* @deprecated Will be protected.
*/
public $config;
protected $config;

/**
* Holds the old data from a redirect.
Expand Down
2 changes: 2 additions & 0 deletions user_guide_src/source/changelogs/v4.5.0.rst
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,8 @@ Test
Others
------

- **IncomingRequest::** The visibility of the deprecated properties ``$uri`` and
``$config`` has been changed to protected.
- **Config:** The deprecated ``CodeIgniter\Config\Config`` class has been removed.

Enhancements
Expand Down

0 comments on commit 365e4c1

Please sign in to comment.