Problem/Motivation

web/modules/contrib/virustotal/virustotal.module
Call to deprecated function module_load_install(). Deprecated in drupal:9.4.0 and is removed from drupal:10.0.0. Use Drupal::moduleHandler()->loadInclude($module, 'install') instead. Note, the replacement no longer allows including code from uninstalled modules.

web/modules/contrib/virustotal/virustotal.info.yml

Value of core_version_requirement: ^8 || ^9 is not compatible with the next major version of Drupal core. See https://drupal.org/node/3070687.

web/modules/contrib/virustotal/composer.json

Parse error in composer.json. Having a composer.json is not a requirement in general, but if there is one, it should be valid. See https://drupal.org/node/2514612.

CommentFileSizeAuthor
#6 after.png13.19 KBraveen_thakur51
#6 before.png43.33 KBraveen_thakur51
#3 virustotal-3365532-2.patch1.17 KByogeshsevak
Screenshot from 2023-06-08 12-30-29.png112.45 KByogeshsevak

Issue fork virustotal-3365532

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

yogeshsevak created an issue. See original summary.

sakthi_dev’s picture

sakthi_dev commented
Assigned: Unassigned » sakthi_dev
yogeshsevak’s picture

yogeshsevak commented
Assigned: sakthi_dev » yogeshsevak
Status: Active » Needs review
StatusFileSize
new virustotal-3365532-2.patch1.17 KB

Providing patch with fixes. Please review.

yogeshsevak’s picture

yogeshsevak commented
Assigned: yogeshsevak » Unassigned

sakthi_dev opened merge request !2

raveen_thakur51’s picture

raveen_thakur51 commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new before.png43.33 KB
new after.png13.19 KB

Hello Guys, I have installed the module version - 8.x-1.2 on Drupal 9.5 After installing the module The module was not compatible with D10. As you can see the issues are in the attached screenshot. But after applying patch #3, The module started showing in the compatible section. Please see attached screenshots for reference.
And Thank You.

ysamoylenko’s picture

ysamoylenko commented

Thank you for the patch implementation and review activities. The changes will be committed soon.

ysamoylenko’s picture

ysamoylenko commented
Status: Reviewed & tested by the community » Fixed

The changes have been committed.

ysamoylenko closed merge request !2

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.