There is edge case, when some module is already missing from the filesystem but still available in database. And now you want to call drupal_uninstall_modules to be sure it's cleaned from database. I know, it sounds weird but possible. So, I face the issue because of require_once in rabbit_hole_modules_uninstalled.

CommentFileSizeAuthor
#2 3061189-2.patch750 bytesApacheEx

Comments

ApacheEx created an issue. See original summary.

ApacheEx’s picture

StatusFileSize
new750 bytes

So, here is small patch which fixes it. I think we are safe here.

ApacheEx’s picture

Version: 7.x-2.25 » 7.x-2.x-dev
colinstillwell’s picture

Looks good to me!

  • Matroskeen committed 6a1a924 on 7.x-2.x authored by ApacheEx
    Issue #3061189 by ApacheEx: Replace require_once to module_load_include...
matroskeen’s picture

Status: Needs review » Fixed

Yeah, it looks good to me as well!
Committed to 7.x-2.x.

Thanks @ApacheEx,
See you around :)

Status: Fixed » Closed (fixed)

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