-
-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Latest versions not working with AWS media source #111
Comments
I currently have no external media source available. This code tries to get the image size with img.onload, since the MODX way has issues with images larger than 800x600. With 2.9.1 I did not use the MODX way anymore and switched to the img.onload: ImagePlus/source/js/mgr/imageplus.panel.input.js Lines 311 to 363 in 2d7dfad
There has to be some switch for external media sources. |
Ah, on external media i stopped grabbing the image dimensions with the media source because it's huge request killer when there are more than a handful of files in a folder. |
nevermind, i left the fake 400x300 value in there so it's not missing entirely https://github.com/modxcms/aws-s3-media-source/blob/6a0d42265a66d30a05b698a2a65decb03df696cc/core/components/awss3mediasource/model/awss3mediasource/awss3mediasource.class.php#L471 |
The issue is the onload requirement. An external media source has to retrieve the image different. If you are able to send me some code loading the image from an exernal media source I will be very happy. |
Upgrading from 2.8.5 to 2.9.1 caused it to stop working with AWS media sources. When I select the image from an AWS Source it says image could not be found. It looks like it is trying to read the path and not the full URL.
The text was updated successfully, but these errors were encountered: