pagarme / pagarme-php

76%

master: 73%

LAST BUILD ON BRANCH v3.7.8

branch: v3.7.8

CHANGE BRANCH

x

Reset

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

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