- 🇬🇷Greece vensires
Since we are updating the README file, it might good to write down somewhere that in PHP 8.1 and above, we get the following error when using the common setup of
$settings['redis.connection']['port'] = NULL;
:
Deprecated function: Redis::connect(): Passing null to parameter #2 ($port) of type int is deprecated in Drupal\redis\Client\PhpRedis->getClient() (line 32 of /XXX/modules/contrib/redis/src/Client/PhpRedis.php).I checked how Matomo handled this and it seems that the suggested way is to use the following instead:
$settings['redis.connection']['port'] = 0;
- First commit to issue fork.
- Assigned to Anybody
- Status changed to Needs work
7 months ago 8:16am 17 April 2024 - 🇩🇪Germany Anybody Porta Westfalica
- Issue was unassigned.
- Status changed to Needs review
7 months ago 8:48am 17 April 2024 - 🇩🇪Germany Anybody Porta Westfalica
- Status changed to RTBC
7 months ago 9:10am 17 April 2024 - 🇬🇷Greece vensires
I totally agree that we shouldn't go on with any changes in the code. Just document. And your MR succeeds in this.
So, I set it as RTBC ;) - Status changed to Fixed
3 months ago 7:59pm 16 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.