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.

Sign up for GitHub

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

Jump to bottom

Upcoming deprecation of mt_rand() #6275

Open

poodle123 opened this issue Mar 25, 2024 · 0 comments

poodle123 commented Mar 25, 2024

In order to future-proof CI3 mt_rand() should be replaced with random_int(PHP_INT_MIN, PHP_INT_MAX) for PHP versions starting with 7.0.0.

This affects the following system files:

core\Common.php
core\Security.php
helpers\captcha_helper.php
helpers\form_helper.php
helpers\string_helper.php
libraties\Upload.php

If one doesnt need support for PHP lower than 7, then its just a simple replacement otherwise a check of the PHP version would be required.

mckaygerhard added a commit to codeigniterpower/codeigniterpower that referenced this issue


          Fix upcoming deprecation of mt_rand on newer php versions

3c9fcc8

* provide a workaround function in code/Common like is_php does
* closes bcit-ci/CodeIgniter#6275
* related to pocketarc/codeigniter#3 (comment)

mckaygerhard added a commit to venenux/codeigniterpower that referenced this issue


          Fix upcoming deprecation of mt_rand on newer php versions

67f7eb9

* provide a workaround function in code/Common like is_php does
* closes bcit-ci/CodeIgniter#6275
* related to pocketarc/codeigniter#3 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment