-
Notifications
You must be signed in to change notification settings - Fork 259
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
Call to undefined function GuzzleHttp\Psr7\stream_for() #470
Comments
this is also fixed via #469 |
Quickfix: Goto: \vendor\zendesk\zendesk_api_client_php\src\Zendesk\API\HTTP.php Line 70 and replace //Corrected cause of error //Original code |
I don't know if Zendesk is willing to merge this.. A quick & MacGyver solution, instead of modifying the core files. Create a directory in your project
And paste in the code below
|
Merge please @zendesk 😇 |
@thekindofme, @gkatechis, @gmponos, @nogates sorry to ping you guys but would it be possible to merge one of the fixes (like #469)? The requirement is not correct and it's easy to mess-up. The current workaround is to add a function like @kevinirlen proposes or to require explicitly and old version of Thanks in advance |
if you want backward compatibility with guzzle 6.x... then i think the cleanest solution for now would be to require @thekindofme ping |
Huge problem, this just occurred on production for me. A workaround is to do what @simonmaass mentioned. |
As it might be helpful to some - I didn't want to mess with my directory structure, thus I've saved @kevinirlen into a file within my usual workspace (called functions.php) and used composer to autoload this file: composer.json:
|
Any news on a timeframe for a merge? Thanks for posting the workarounds guys |
Do you folks need a cross-compatible version of the fix that would comply with multiple versions of Guzzle, maybe?
A cross-compatible version should be easy, a simple I would've created a PR, but since there's lots of them already, I didn't want to pollute the current state of the project. However, I'm one of the several people whose production broke because of this, and I had to use a dirty quickfix for that which I don't like at all, so as others, I wish this could be fixed and released soon 😉 |
It seems that packagist is not updated automatically, the last ^v2 is still v2.2.11 (https://packagist.org/packages/zendesk/zendesk_api_client_php). You can use "dev-master" or the specific commit as a workaround. |
dev-master actually doesn't work directly as it isn't updated on packagist as well. You'll have to add this repository in your composer.json before being able to pull for dev-master or v2.2.11 |
You are right |
It looks like it is on packagist now. |
I have |
Can you release a new version that allows guzzle psr7 v2 to be installed? |
Hi, thank you for your request, |
Closing due to inactivity. Re-open if needed. |
Error: Call to undefined function GuzzleHttp\Psr7\stream_for() #7 zendesk/zendesk_api_client_php/src/Zendesk/API/Http.php(69): Zendesk\API\Http::send #6 zendesk/zendesk_api_client_php/src/Zendesk/API/HttpClient.php(494): Zendesk\API\HttpClient::post #5 zendesk/zendesk_api_client_php/src/Zendesk/API/Resources/Core/Tickets.php(164): Zendesk\API\Resources\Core\Tickets::create
The text was updated successfully, but these errors were encountered: