When one disables the core update module, a requirement error is displayed on the status report and on the admin page. This error may hide other problems (schema updates, cron failures etc) on admin as the notification will be disregarded for crying wolf all the time.

Having the update module disabled is not a direct threat to one's site and may in fact be a concious decision when one is subscribed to the security announcement list and does not care about mere updates.

CommentFileSizeAuthor
update_warning.patch1.08 KBHeine
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

Version: 6.x-dev » 7.x-dev

Too late for D6 I think. I agree that it's not good to desensitise people to other warnings with big flashing red lights.

There is this option though: http://drupal.org/project/update_notifications_disable

catch’s picture

Status: Needs review » Reviewed & tested by the community

Still applies, still a good change. Marking RTBC.

moshe weitzman’s picture

This is extremely annoying. update.module makes all sorts of http requests to fetch its data. in some evironments, those are not able to complete and they actually block the web page from completeing. lets not hassle admins like this. they actively chose to disable this during install.

swentel’s picture

I like this one too, patch still works, although with an offset of 3 lines.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD and DRUPAL-6 as per the request of the commenters.

Freso’s picture

Awesome. My test installs thank you. =)

Anonymous’s picture

Status: Fixed » Closed (fixed)

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