`pre_unzip_file` and `unzip_file` filters in `_unzip_file_pclzip()` do not function as documented

[sanchothefat]

The filters in _unzip_file_pclzip() have a bug, caused by the use of the variable name $file in 2 foreach loops within the function that overwrite the original argument passed to the function.

trunk/src/wp-admin/includes/file.php#L1963​

trunk/src/wp-admin/includes/file.php#L1990​

The foreach loops set the $file variable, overriding the one passed into the function on each iteration, so the filter does not work as documented for _unzip_file_pclzip()

Uses a different variable name in the foreach loops to preserve the $file variable passed to the filter.

Trac ticket: [](https://core.trac.wordpress.org/ticket/64398)

@roborourke looks like the changes broke a related unit test.

[SergeyBiryukov]

In 61374:

Filesystem API: Pass correct $file value to pre_unzip_file and unzip_file filters.

This commit ensures that the original $file argument passed to the function is not unintentionally overwritten by the use of the same variable name in two foreach loops.

Follow-up to [56689].

Props sanchothefat, westonruter, mukesh27, SergeyBiryukov.
Fixes #64398.

[westonruter]

In 61375:

Coding Standards: Remove whitespace at end of line.

Fixes PHPCS warning: Squiz.WhiteSpace.SuperfluousWhitespace.EndLine.

Follow-up to [61374].

See #64398.