media_entity_copy_icons is called in media_entity_document.install file which has been removed from the codebase. This is causing the installation of the module to fail.

CommentFileSizeAuthor
#2 2936985-2.patch1.24 KBhussainweb

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

Status: Active » Needs review
StatusFileSize
new1.24 KB

This patch should fix the problem. I know installation on Drupal 8.4+ is not a valid scenario but it is required in some scenarios. Anyway, it is a good idea to have the module working.

marcoscano’s picture

Thanks for reporting!

In principle you should only need (and are recommended to use) this module if you are using Media Entity 1.x (regardless of Drupal core version).

On that branch, the function media_entity_copy_icons() exists.

Could you please indicate what scenario you are getting the failure on?

If you are using the core Media module (8.4.x+), you don't need Media Entity Image anymore, once core has the same functionality.

hussainweb’s picture

@marcoscano, thanks for responding. I understand we don't need the module on Drupal 8.4, but Lightning still depends on it. I am not able to install a new site because of this and hence submitted the patch. This way, I can at least move ahead with my site setup (and it might help someone else).

I understand if this patch can't be committed, but it helped move things forward. :)