revolution icon indicating copy to clipboard operation
revolution copied to clipboard

[3.x] Include Flysystem FTP adapter in composer.json

Open smg6511 opened this issue 3 years ago • 0 comments

What does it do?

Adds missing base FTP adapter package to require block.

Why is it needed?

MODx 3 has a custom FTP adapter in the codebase and makes FTP an option as a type of media source. However, if you attempt to create an FTP source a fatal error will occur because the base package is missing.

How to test

  1. Run composer update in your base revolution directory.
  2. Create an FTP media source, verifying it works as expected. (I created one using an open public FTP resource to keep it simple for testing.)
  3. To verify the fatal error, remove the change I made to composer.json, re-run composer update, and attempt to access the manager.

Related issue(s)/PR(s)

No related issues.

smg6511 avatar Jun 04 '22 04:06 smg6511