Handle null paths

[lazysoundsystem]

A sticking plaster for the error in #43

[jonpugh]

Can we boost this fix?

The only way I can get composer install working on my project is to roll back composer to 2.5.5 or use this fork.

[JonathanH-UK]

I've hit this too.. +1 for merging the PR please

[colinstillwell]

Same problem for me, would love to see this come in and become part of a stable release version.

[sandykadam]

Please merge this and release this asap! PHP8 upgrade is stuck because of this error.

[ivangrynenko]

Please merge this PR and create an official release.
+1

[gargsuchi]

Please merge this PR and create an official release.
+1

[vladdancer]

Same thing for me. +1 for mergin PR.

[lisotton]

Same for me, +1 for merge this PR.

[mxr576]

Null paths can be and probably should be handled at the source: \DrupalComposer\PreservePaths\PluginWrapper::getInstallPathsFromPackages()

diff --git a/src/PluginWrapper.php b/src/PluginWrapper.php
--- a/src/PluginWrapper.php	
+++ b/src/PluginWrapper.php	(date 1710258587092)
@@ -136,6 +136,8 @@
             $paths[] = $installationManager->getInstallPath($package);
         }
 
+        $paths = array_filter($paths);
+
         return $this->absolutePaths($paths);
     }

[niallfleming]

mxr576

Is there an example of how this should be achieved? What if I have no control of the source, but don't wish to fork it?

[sandykadam]

You can update and use mine fixed repo - composer require sandykadam/preserve-paths
https://github.com/sandykadam/preserve-paths