Skip to content

Conversation

@settermjd
Copy link

@settermjd settermjd commented Aug 11, 2025

Q A
Documentation no
Bugfix no
BC Break no
New Feature no
RFC no
QA no

Description

While not strictly necessary for laminas-filter, I've created this change to support the latest version of laminas-servicemanager that contains the removes duplicate delegators fix. By including this change, I can then update laminas-inputfilter to support the same version of laminas-servicemanager as well, as it depends on laminas-filter.

As some background, the reason for the version bump is that a project I'm working on depends on laminas-servicemanager and laminas-inputfilter, but needs to have the above fix, as without it when loaded into a Mezzio project, its ConfigProvider classes, which loads a set of custom routes, will cause a duplicate route exception if the existing project's ConfigProvider class also loads routes into the applications routing table.

While not strictly necessary, I've created this change to support the
latest version of laminas-servicemanager that contains the removes
duplicate delegators fix.

See laminas/laminas-servicemanager#251.

By including this change, I can then update laminas-inputfilter to
support the same version of laminas-servicemanager as well.

Signed-off-by: Matthew Setter <[email protected]>
@settermjd settermjd force-pushed the support-laminas-servicemanager-4.4.x branch from aff03b8 to a86a147 Compare August 11, 2025 08:06
@gsteel
Copy link
Member

gsteel commented Aug 11, 2025

@settermjd It'll be a while before input-filter is released with SMv4 support as it's a pretty big change.

It will likely support ^4.x and so it will be unnecessary to bump to a specific version - any project running composer update will get latest deps and all the mentioned fixes.

To rephrase, I don't see the point - it's not the role of filter or input-filter to fix mezzio's / other projects deps, particularly when the current dependency set would not prevent an update to 4.whatever unless I'm missing something?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants