search

BobRay / UpgradeMODX

A dashboard widget that detects upgrades and (optionally) installs them from within the MODX Manager
https://bobsguides.com/upgrade-modx-package.html
22 stars 14 forks source link

Do not require Guzzle7 extra for MODX 3 #85

Closed bezumkin closed 1 month ago

bezumkin commented 3 months ago

MODX 3 already has guzzlehttp/guzzle as composer dependency in core/vendor directory.

We don't need to require additional extra with different version of Guzzle.

BobRay commented 3 months ago

I'm working on a new version that won't use either Guzzle or cURL. It uses MODX internal functions to generate the cache files. It's much faster and better, but it still has some strange issues I have to work out. Let me know if you'd like to be a beta tester.

bezumkin commented 3 months ago

@BobRay Sure I would like to test the beta, but my point was to quickly release this update to help people to get rid of Guzzle7 in MODX 3.

Because it may cause conflicts.

BobRay commented 3 months ago

Thanks, I'll put it up for you when I have something worth testing.

BobRay commented 3 months ago

@BobRay Sure I would like to test the beta, but my point was to quickly release this update to help people to get rid of Guzzle7 in MODX 3.

Because it may cause conflicts.

I'm hoping it's about ready to release. The transport file for the 3.0.1-beta is in the dev branch HERE.

If you don't find any problems let me know and I'll release it. Thanks for the help!

Bob

Mark-H commented 2 months ago

Fwiw, the issue in guzzle7 is fixed in v7.8.1. See my comment here