Error: Class 'ZipArchive' not found in Drupal\Core\Archiver\Zip->__construct() (line 30 of core/lib/Drupal/Core/Archiver/Zip.php).

Hm, It's actually a core bug.

The Drupal Core doesn't require ext-zip in composer.json file. This leads to this problem.

I believe the System requirements page also needs to be updated to explicitly tells that Drupal needs PHP Zip extension.

Things going a bit complicated.

PHP 7.4.8, Zip and ZLib extensions enabled. Still got this error. So this is not the only requirements missing problem.

There are two approaches here:

- either the ArchiverInterface plugin system is extended to allow plugins to declare their required extensions and this is used when creating an instance

- OR -

- we make core require the zip extension, this would require patching both system_requirements and composer.json.

At least patch to prevent this

+++ b/core/modules/system/system.module
@@ -1475,3 +1475,13 @@ function system_path_alias_presave(PathAliasInterface $path_alias) {
+  if (!class_exists('\ZipArchive')) {
+    // PHP Zip extension could be missing.
+    unset($info['zip']);

I was wrong

11 is the valid patch

Since php 5.5 better to use full class name

Last patch failed to apply for 9.0.x , uploading new patch for 9.0.x kindly review.

We could add a suggest entry to composer.json as well?

Can we update the IS? Would be nice to know what the resolution is and what are the remaining tasks.

And #16 needs to be addressed.

Addressed #16.
Added suggest in composer.json

Updated hash.

Still needs an issue summary update. In order to review the patch and check that it is doing what is planned the IS needs to have, at least, a proposed resolution section completed.

@guilhermevp, thank you very much.

I read the issue and found that the question in #16 has not been answered. Setting to NW for that and the fact that the patch needs a reroll.

Re-rolled patch #23

Not quite sure why this issue is still stuck on 9.1.x, updated Version to 9.3.x.

Since only the creator of a MR can change the branch-to-merge-to I'm opening a new MR based on 9.3.x.

Made 4 changes:

1) Moved the suggest bit from composer.json to core/composer.json. This because it seemed to me that all the requiring of PHP extensions is done over there.
2) There's no core/composer.lock, but even if there was, adding a suggest doesn't show up in the corresponding composer.lock file.
2) Did a composer update drupal/core to pull in our changed core/composer.json.
3) The correct capitalisation in the system_archiver_info_alter-hook is Zip.
4) Reworded the suggest bit in core/composer.json.

Updated the IS with what I believe is going on.

#16 is now addressed and the solution here is one of the two suggested in #10. Unfortunately, the MR needs an update.

Thanks @quietone.

• Merged HEAD of 9.3.x into the MR.
• Did a composer update drupal/core to pull in our changed core/composer.json.

Test currently running.

Rerolled against the latest 9.3.x

It makes sense to improve as drupal-CI runners mentions in every CI-run

As there is no 'unzip' nor '7z' command installed zip files are being unpacked using the PHP zip extension.
This may cause invalid reports of corrupted archives. Besides, any UNIX permissions (e.g. executable) defined in the archives will be lost.
Installing 'unzip' or '7z' (21.01+) may remediate them.

need a reroll

thanks :)
Feeling luck so RTBC

Makes sense to not add this if it's not available, and to add the extension to suggestions, but the patch doesn't apply on 10.1

#47 applied to 10.1.x, and #48 to 10.0.x; committed to both branches. Thanks!