[PHP 8.4] fgetcsv() deprecated warning in three more files

[addison74]

Related to PR #4297.

Three more files need to be updated.

PS - Credits to Copilot.

[Copilot]

Pull Request Overview

This PR addresses PHP 8.4 compatibility by updating fgetcsv() function calls to include the escape parameter, preventing deprecation warnings. The change affects three files that were missing from a previous similar update.

• Adds explicit escape parameter ('\') to fgetcsv() calls in three files
• Ensures consistency with PHP 8.4 requirements for the fgetcsv() function signature
• Completes the deprecation warning fixes started in PR #4297

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
lib/Varien/Io/File.php	Updated fgetcsv() call in streamReadCsv() method to include escape parameter
app/code/core/Mage/Paypal/Model/Report/Settlement.php	Updated fgetcsv() call in CSV parsing loop to include escape parameter
app/code/core/Mage/Dataflow/Model/Convert/Iterator/Http.php	Updated fgetcsv() call in file upload processing to include escape parameter

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[addison74]

We have to do one more step: ZF1-Future repository.

@sreichel can you do it into your fork? I will approve the PR immediately.

[sreichel]

I'd suggest using "vendor patches".

(and move all patches to this repo)

[addison74]

I'd suggest using "vendor patches".

(and move all patches to this repo)

It is a better approach.

[sreichel]

Fixed in #4891 .