Merge with media_entity_remote_image?

Created on 13 November 2019, over 5 years ago
Updated 21 February 2024, about 1 year ago

I recently released https://www.drupal.org/project/media_entity_remote_image β†’ which seems to hit on the same need as this module, but with a focus on Media. It looks like this module hasn't been updated since 2016 so not sure if there's any chance of connecting with a maintainer, but if they see this then drop me a line.

If anybody sees features missing in Media Entity Remote Image that are present in this module, drop an issue in the queue and I'll see what I can do! Patches are welcome.

πŸ’¬ Support request
Status

Postponed: needs info

Version

1.0

Component

Miscellaneous

Created by

πŸ‡ΊπŸ‡ΈUnited States mrweiner

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡¬πŸ‡§United Kingdom 2dareis2do

    If you could add a media widget for this it would be good for site editors. After all this is Drupal CMS.

    I think the issue ihere s that media type is attached to media bundle. media_entity_remote_image does this but the problem is it is somewhat inflexible because you have to use media bundle name for this. media_entity_remote_image assumes that this bundle name 'remote_image' is available . However, if you already have remote_image media bundle defined it makes it difficult to use.

    This pattern is used in core as we see certain media bundles are defined there.

    Furthermore, media_entity_remote_image does not use a remote stream wrapper, unlike remote_stream_wrapper module that also does similar. I understand that one difference is remote_stream_wrapper does give you the option to download the remote image and use it as an image style. Also it makes no assumptions about the name of the media bundle. That, said, unlike media_entity_remote_image it lacks the ability to add via media entity reference field or ckeditor 5 media widget.

    One problem with remote_stream_wrapper is it uses the same schema as an image, and comes with the limitations of said module. e.g. I believe remote image urls are limited to 256 chars or something. url strings can and are much longer. 2,048 would probably be a better default here as this is the ie limit.

    See https://stackoverflow.com/questions/812925/what-is-the-maximum-possible-...

Production build 0.71.5 2024