UpgradeMODX icon indicating copy to clipboard operation
UpgradeMODX copied to clipboard

Do not require Guzzle7 extra for MODX 3

Open bezumkin opened this issue 1 year ago • 5 comments

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.

bezumkin avatar Mar 22 '24 07:03 bezumkin

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.

BobRay avatar Mar 23 '24 02:03 BobRay

@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.

bezumkin avatar Mar 23 '24 09:03 bezumkin

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

BobRay avatar Mar 24 '24 20:03 BobRay

@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

BobRay avatar Apr 05 '24 22:04 BobRay

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

Mark-H avatar Apr 24 '24 13:04 Mark-H