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

Comments

catch’s picture

catch
he/him
Primary language English
commented
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

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Still applies, still a good change. Marking RTBC.

moshe weitzman’s picture

moshe weitzman commented

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

swentel commented

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

dries’s picture

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

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

Freso’s picture

Freso commented

Awesome. My test installs thank you. =)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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