Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compatibility problems with PHP 8. #570

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

k4bek4be
Copy link

Tested with PHP 8.1.20. Includes fixes from misterludvigsen and ColdSphinX.

Tested with PHP 8.1.20. Includes fixes from misterludvigsen and ColdSphinX.
@sjau
Copy link

sjau commented Mar 27, 2024

Unfortunately this isn't working for PHP 8.3 anymore.

I get tons of errors like this:

A PHP Error was encountered
Severity: 8192
Message: Creation of dynamic property CI_URI::$config is deprecated
Filename: core/URI.php
Line Number: 101
Backtrace:
File: ...
Line: 284
Function: require_once

A PHP Error was encountered
Severity: 8192
Message: Creation of dynamic property CI_Router::$uri is deprecated
Filename: core/Router.php
Line Number: 127
Backtrace:
File: ....
Line: 284
Function: require_once

@PJanisio
Copy link
Contributor

PJanisio commented Jun 3, 2024

Unfortunately this isn't working for PHP 8.3 anymore.

I get tons of errors like this:

A PHP Error was encountered
Severity: 8192
Message: Creation of dynamic property CI_URI::$config is deprecated
Filename: core/URI.php
Line Number: 101
Backtrace:
File: ...
Line: 284
Function: require_once

A PHP Error was encountered
Severity: 8192
Message: Creation of dynamic property CI_Router::$uri is deprecated
Filename: core/Router.php
Line Number: 127
Backtrace:
File: ....
Line: 284
Function: require_once

Solution has been already implemented in codeigniter here:

https://github.com/bcit-ci/CodeIgniter/pull/6173/files

@Tangol
Copy link

Tangol commented Sep 18, 2024

I found this: https://github.com/MercanoG/SNIP
Seems to be working fine on PHP 8.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants