pagarme / pagarme-php

76%

master: 73%

LAST BUILD ON BRANCH v3.7.8

branch: v3.7.8

CHANGE BRANCH

x

Reset

• v3.7.8
• 3.0.2
• V3
• add_circleci
• add_coveralls
• add_cvv_on_transaction
• add_sentry
• change_subscription_cancel
• deprecated/partial-refund-boleto-amount
• enable-external-id
• feature/abstraction-for-transactions-can-be-paid-on-installments
• feature/add-customer-id-to-transaction-and-subscription-payload
• feature/add-cvv-to-card-payload
• feature/add-metadata-when-capture-transaction
• feature/add-pending-review-status-const
• feature/add-reference-key
• feature/add-softdescriptor-to-boleto
• feature/boleto-async
• feature/boleto-instructions
• feature/capture-by-token
• feature/capture-with-split-rules
• feature/invoice-reminder-to-plan
• feature/remove-git-hook
• feature_balance
• feature_company
• feature_event_transactions
• feature_impl_default_timeout_option
• fix-building-bulk-anticipation
• fix/acceptance-tests
• fix/add-optional-address-data-to-transction
• fix/api-version
• fix/balance-constructor
• fix/bank-account-object-on-transfer-creation
• fix/boleto-refund
• fix/bulk-anticipation-tests
• fix/code-style
• fix/guzzle-timeout
• fix/insight-minor
• fix/issue-248
• fix/object-manipulation
• fix/object-types
• fix/recipient_operations_test
• fix/request-timeout
• fix/search-handler-params
• fix/update-guzzle
• fixParameterNameSubscriptionUpdate
• fix_postbackUrl_null
• fix_properties_case
• fix_scrutinizer_issues
• fix_subscrition_by_card_hash
• improvement/contributing
• increase_coverage
• issue_pr_template
• other_test
• postback_validity
• remove_key_dump
• revert-107-add_sentry
• story/customize-headers
• test
• test/coverage
• test/one
• thamara
• updateSubscroptionRoute
• update_readme
• v3.0
• v3.0.1
• v3.0.2
• v3.0.3
• v3.1.0
• v3.1.1
• v3.1.2
• v3.1.3
• v3.3.1
• v3.3.2
• v3.4.0
• v3.5.0
• v3.6.1
• v3.7.4
• v3.7.5
• v3.7.6
• v3.7.9

Build # 1572

Build Type

push

travis-ci

Committed by leonampd

Commit Message

client: fixes legacy guzzle request objects instantiation

Theres a big difference between `5.*` and `6.*` guzzle majors. So, in the `PagarMe\Sdk\Client::buildRequest` method the sdk should instantiate the correct request object accordingly its contracts, in this case twice: `\GuzzleHttp\Message\Request` and `\GuzzleHttp\Psr7\Request`.

If the user define request options, in guzzle `5.*` this options should be within the request object. If the major is `6.*` the options should be informed on `\GuzzleHttp\Client::send()` method as a second argument.

This fixes the described behavior above informing the request options on the right places.

Run Details

1243 of 1634 relevant lines covered (76.07%)

3.82 hits per line