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

CSV: Explicitly pass escape argument (PHP 8.4) #8495

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

mjansenDatabay
Copy link
Contributor

See: https://www.php.net/manual/en/migration84.deprecated.php

Using the default value for the escape parameter for the SplFileObject::setCsvControl(), SplFileObject::fputcsv(), and SplFileObject::fgetcsv() is now deprecated. It must be passed explicitly either positionally or via named arguments. This does not apply to SplFileObject::fputcsv() and SplFileObject::fgetcsv() if SplFileObject::setCsvControl() was used to set a new default value.

See: https://www.php.net/manual/en/migration84.deprecated.php

Using the default value for the escape parameter for the SplFileObject::setCsvControl(), SplFileObject::fputcsv(), and SplFileObject::fgetcsv() is now deprecated. It must be passed explicitly either positionally or via named arguments. This does not apply to SplFileObject::fputcsv() and SplFileObject::fgetcsv() if SplFileObject::setCsvControl() was used to set a new default value.
@mjansenDatabay
Copy link
Contributor Author

@qualitus-dahme The CI pipeline only fails because of the "wrong" copyright header in the SCORM-related PHP files. I suggest ignoring these errors in the context of this PR.

@dsstrassner dsstrassner assigned thojou and unassigned pascalseeland Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants