Fix PHP 8.4 Deprecations

[TheVitik]

Summary

This PR fixes PHP 8.4 deprecation warnings related to fputcsv().

Starting with PHP 8.4, calling fputcsv() without explicitly passing the $escape parameter triggers the following deprecation warning:

The $escape parameter must be passed when calling fputcsv() as its default value will change in a future PHP version.

Changes

• Updated both occurrences of fputcsv() where the $escape parameter was omitted.
• Explicitly passed the current default escape character ('\\') to preserve the existing behavior and eliminate the deprecation warning.

Example:

fputcsv( $csv, $row, ',', '"', '\\' );

Impact

This change is fully backward compatible and prepares the codebase for PHP 8.4 by removing the deprecation warnings without changing the existing CSV output behavior.

[PatelUtkarsh]

Thanks for the fix! The approach looks good and is backward compatible with PHP 7.2.

One issue: the changed lines use spaces instead of tabs, so phpcs fails. Running vendor/bin/phpcbf classes/class-cli.php exporters/class-exporter-csv.php will fix it.

[TheVitik]

Hi @PatelUtkarsh , thank you for the review.

Could you please check again?

When is it possible to release these changes?

[PatelUtkarsh]

Looks good!

[PatelUtkarsh]

Hello @TheVitik,

We can include it in next release; however next release isn't planned yet; I will let you know as soon as I confirm with team.