freshdesk-php-sdk icon indicating copy to clipboard operation
freshdesk-php-sdk copied to clipboard

Update Guzzlehttp to v7

Open dhildreth opened this issue 4 years ago • 3 comments

Laravel released version 8 recently. It requires guzzlehttp/guzzle 7.0.1 (upgrade docs). It appears v3.1 of freshdesk-php-sdk requires >=4.0,<7.0:

$ composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: don't install guzzlehttp/guzzle 7.0.1
    - Conclusion: don't install guzzlehttp/guzzle 7.1.x-dev
    - mpclarkson/freshdesk-php-sdk 0.3.1 requires guzzlehttp/guzzle >=4.0,<7.0 -> satisfiable by guzzlehttp/guzzle[4.x-dev, 5.3.x-dev, 6.5.x-dev].
    - mpclarkson/freshdesk-php-sdk 0.3.1 requires guzzlehttp/guzzle >=4.0,<7.0 -> satisfiable by guzzlehttp/guzzle[4.x-dev, 5.3.x-dev, 6.5.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.0.x-dev, 5.3.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.0.x-dev, 6.5.x-dev].
    - Can only install one of: guzzlehttp/guzzle[7.0.x-dev, 4.x-dev].
    - Installation request for guzzlehttp/guzzle ^7.0.1 -> satisfiable by guzzlehttp/guzzle[7.0.1, 7.0.x-dev, 7.1.x-dev].
    - Installation request for mpclarkson/freshdesk-php-sdk ^0.3.1 -> satisfiable by mpclarkson/freshdesk-php-sdk[0.3.1].

Similar to issue #8 , can we get updates for guzzle 7.0.1 please?

dhildreth avatar Sep 09 '20 21:09 dhildreth

Hi,

I just change the composer.json and it works.

PR here

cheers

patrocle avatar Dec 08 '20 11:12 patrocle

Is there anything else we can do to help get guzzle > 7.0.1 working with this package?

vagrant@homestead:~/www.embeddedarm.com$ composer require mpclarkson/freshdesk-php-sdk
Using version ^0.3.1 for mpclarkson/freshdesk-php-sdk
./composer.json has been updated
Running composer update mpclarkson/freshdesk-php-sdk
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires mpclarkson/freshdesk-php-sdk ^0.3.1 -> satisfiable by mpclarkson/freshdesk-php-sdk[0.3.1].
    - mpclarkson/freshdesk-php-sdk 0.3.1 requires guzzlehttp/guzzle >=4.0,<7.0 -> found guzzlehttp/guzzle[4.0.0-rc.1, ..., 4.x-dev, 5.0.0, ..., 5.3.x-dev, 6.0.0, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

dhildreth avatar Jul 15 '21 02:07 dhildreth

It appears there are some forks that are more up to date. For example, celery-payroll/freshdesk-php-sdk appears to be a company-backed fork with support for Guzzle < 8.0. To use it, add the following to your composer.json file:

    "repositories": [{
        "type": "vcs",
        "url": "https://github.com/celery-payroll/freshdesk-php-sdk"
    }]

Then, run composer require mpclarkson/freshdesk-php-sdk. Yes, this looks strange, but it will actually install the correct version (in my case v1.0.1, which allows for Guzzle < 8.0.

dhildreth avatar Jul 15 '21 02:07 dhildreth