Update lib/Buzz/Client/AbstractCurl.php #92

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@eXsio

eXsio commented Jul 27, 2012

Hi

You didn't implement adding CURLOPT_POSTFIELDS properly - if $fields was an array, it didn't got sent. http_build_query() resolved this issue.

greets
eXsio

Update lib/Buzz/Client/AbstractCurl.php
Hi

You didn't implement adding CURLOPT_POSTFIELDS properly - if $fields was an array, it didn't got sent. http_build_query() resolved this issue.

greets
eXsio
@kriswallsmith

This comment has been minimized.

Show comment
Hide comment
@kriswallsmith

kriswallsmith Oct 31, 2012

Owner

Thank you. Would you mind writing a functional test to demonstrate this defect?

Owner

kriswallsmith commented Oct 31, 2012

Thank you. Would you mind writing a functional test to demonstrate this defect?

@Nyholm

This comment has been minimized.

Show comment
Hide comment
@Nyholm

Nyholm Mar 17, 2018

Collaborator

Closing because of inactivity.

Collaborator

Nyholm commented Mar 17, 2018

Closing because of inactivity.

@Nyholm Nyholm closed this Mar 17, 2018

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