Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes php error Stdlib\DispatchableInterface not found #32

Merged
merged 1 commit into from
Jan 19, 2020

Conversation

rodde177
Copy link
Contributor

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

Description

Tell us about why this change is necessary:

  • Are you fixing a bug or providing a failing unit test to demonstrate a bug?

on php 7.3 i'm getting fatal error interface not found in Client.php
doing proper import of interface

  • How do you reproduce it?
    error actually happened with old zend http, but it said dont accept new contributions,
    and i see code looks same in laminas

@Xerkus
Copy link
Member

Xerkus commented Jan 19, 2020

Not a bug. Check your local environment for issues. Explicit composer dump-autoload to recreate autoloading files fixes such issues in most cases.

Check if you have some laminas packages installed (composer show | grep "laminas") then make sure you use latest composer (1.9.2) as there were issues reported with older versions not registering autoloading properly on package rewrite by plugin. If you have laminas packages installed then makes sure to do composer update after you updated composer

I will accept this as QA but remove the test as redundant.

@rodde177
Copy link
Contributor Author

I just upgraded php version, so yeah might have forgot to dump-autoload.

@Xerkus Xerkus merged commit dee6425 into laminas:master Jan 19, 2020
Xerkus added a commit that referenced this pull request Jan 19, 2020
@rodde177 rodde177 deleted the interface-not-found-fix branch January 19, 2020 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants