When enabling this module I receive 500 error with Drupal 7.21 (as in when viewing module list, the rest of the site is unaffected). The problem is found in drupal/modules/system/system.module line 109-110:

if (module_exists('update')) {
  if (update_manager_access()) {

This module is identified at "update", and Drupal expects to find the function update_manager_access(). This may be resolved by adding this function in the module:

function update_manager_access() {
  return FALSE;
}

If this is not a suitable solution, the system module should add a test:

if (module_exists('update') && function_exists('update_manager_access')) {
  if (update_manager_access()) {
CommentFileSizeAuthor
#3 1936032_3.patch216 byteschx
#2 meet-expectations-1936032-2.patch237 bytesklakegg

Comments

cdale’s picture

I think the solution here, is to do the following:

First, remove this module completely. Delete it from your installation.
Second, disable and uninstall the core update module! (This should probably go on the project page somewhere as a step to do before adding this module).
Third, once "update" is disabled, you can safely add this module back in, and just never turn it on. The warnings in the status report will be gone by doing this also. This module does not actually need to be enabled, and probably should not be enabled. It just has to exist.

You might also be able to do this by using drush, or updating the system table directly to disable the update module, but, the above method is probably better, as it will remove update data and tables added by the core update module first.

klakegg’s picture

Status: Active » Needs review
StatusFileSize
new237 bytes

This bug is not fixed by removing a core module, this is a problem in core.

I have attached a patch to fix the bug.

chx’s picture

Title: Module is not meeting expectations of Drupal » Fatal error on modules page after enabling this module
Priority: Normal » Critical
Issue summary: View changes
StatusFileSize
new216 bytes

This was added in #602490: Fix help text and action links to find the update manager pages quite long ago although I bet most users of this module never visit the modules page and just use drush. Still, fatal errors are not nice. A few small notes: you do not need to return anything and the space is no longer blank.

cweagans’s picture

Status: Needs review » Reviewed & tested by the community

LGTM

  • japerry committed 4421482 on 7.x-1.x
    Issue #1936032 by chx: Make 7.x not fatal.
    
japerry’s picture

Status: Reviewed & tested by the community » Fixed

Fixed!

Status: Fixed » Closed (fixed)

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