Skip to content

Conversation

@cedric-anne
Copy link
Member

Fixes the following error:

Deprecated: fgetcsv(): the $escape parameter must be provided as its default value will change in /var/www/glpi/vendor/glpi-project/inventory_format/lib/php/FilesToJSON.php on line 263

Call Stack:
    0.0003     412896   1. {main}() /var/www/glpi/vendor/bin/build_hw_jsons:0
    0.0004     414584   2. include('/var/www/glpi/vendor/glpi-project/inventory_format/bin/build_hw_jsons') /var/www/glpi/vendor/bin/build_hw_jsons:119
    0.0232    5683296   3. Glpi\Inventory\FilesToJSON->run() /var/www/glpi/vendor/glpi-project/inventory_format/bin/build_hw_jsons:14
    0.3098    5683416   4. Glpi\Inventory\FilesToJSON->convertIftypeFile() /var/www/glpi/vendor/glpi-project/inventory_format/lib/php/FilesToJSON.php:82
    0.3147    5871752   5. fgetcsv($stream = resource(72) of type (stream)) /var/www/glpi/vendor/glpi-project/inventory_format/lib/php/FilesToJSON.php:263

See https://php.watch/versions/8.4/csv-functions-escape-parameter

@cedric-anne cedric-anne requested a review from trasher March 19, 2025 13:41
@cedric-anne cedric-anne self-assigned this Mar 19, 2025
@trasher trasher merged commit 0e9bdcf into glpi-project:main Mar 19, 2025
6 checks passed
@cedric-anne cedric-anne deleted the fix/php-8.4 branch April 7, 2025 11:35
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.

2 participants