Problem/Motivation

We removed our dependency on laminas/laminas-zendframework-bridge some time ago (when we moved aggregator to contrib).
However we still have code for it in one of our composer scripts

see \Drupal\Core\Composer\Composer::preAutoloadDump

if ($repository->findPackage('laminas/laminas-zendframework-bridge', $constraint)) {
      $autoload['classmap'] = array_merge($autoload['classmap'], [
        $vendor_dir . '/laminas/laminas-zendframework-bridge/src/Autoloader.php',
        $vendor_dir . '/laminas/laminas-zendframework-bridge/src/RewriteRules.php',
      ]);
    }

Steps to reproduce

Proposed resolution

Remove it

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3366314-2.patch781 bytesprem suthar

Comments

larowlan created an issue. See original summary.

prem suthar’s picture

StatusFileSize
new781 bytes

Removed the Code of laminas/laminas-zendframework-bridge Form Composer.php

prem suthar’s picture

Status: Active » Needs review
borisson_’s picture

Will this removal make it harder to install that library again from the contrib module? If it doesn't, rtbc++

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Removing it didn't seem to cause any test failures.

#4 would imagine you could still add it normally though from contrib.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x, thanks!

  • catch committed 16d3fc11 on 11.x
    Issue #3366314 by Prem Suthar, larowlan: Remove composer optimisations...
marvil07’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.