Skip to content

Fix: composer.json - clue/stream-filter version #127

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

Merged
merged 1 commit into from
May 13, 2020

Conversation

tuhin18003
Copy link
Contributor

@tuhin18003 tuhin18003 commented May 13, 2020

clue/stream-filter version should update to 1.4.1.

Problem: if you use "php-http/message" several time in a package, it will call "clue/stream-filter" several time. On their previous version, before 1.4.1, they didn't have the duplicate function checking. So, it will create same namespace several time.

Real Life Example: "php-http/message" has been used in - infusionsoft/php-sdk
if you create several wordpress plugins with - infusionsoft/php-sdk and install all the plugins in same website then "php-http/message" will be called several time so the "clue/stream-filter". Which create error for using same namespace.

* clue/stream-filter version should update to 1.4.1. 

*Problem:* if you use "php-http/message" several time on a package it will call "clue/stream-filter" several time. On their previous version before 1.4.1 they didn't have the duplicate function checking. So, it will create same namespace several time. 

*Real Life Example:*  "php-http/message" has been used in - (infusionsoft/php-sdk)[https://github.com/infusionsoft/infusionsoft-php] 
if you create several wordpress plugins with - infusionsoft/php-sdk and install all the plugins in same website then "php-http/message" will be called several time so the "clue/stream-filter". Which create error for using same namespace.
@dbu dbu merged commit f7fa350 into php-http:master May 13, 2020
@dbu
Copy link
Contributor

dbu commented May 13, 2020

thanks! glad that you found whats wrong, it would have been quite puzzling to figure that out...

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.

2 participants