A message on the Status Report page refers to the old "update manager module" instead of "update status module."

Message on the "Status Report" page:

----------------------------------------------------------
Update notifications Not enabled

Update notifications are not enabled. It is highly recommended that you enable the update status module from the module administration page in order to stay up-to-date on new releases. For more information, Update status handbook page.

-----------------------------------------------------------

But if the user looks on the module admin page, he won't see an "update status module." He'll see an "Update manager" module.

So the text should be revised to say: ". . . recommended that you enable the update manager module. . ."

CommentFileSizeAuthor
#3 999024.patch1.22 KBbfroehle
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Obviously the first line in the message above is backwards. Should say: "refers to the old 'update status module' instead of 'update manager module.' "

bfroehle’s picture

Subscribing. I'll see if I can write a patch in the next few days.

bfroehle’s picture

Version: 7.0-rc2 » 7.x-dev
Status: Active » Needs review
Issue tags: +String freeze
FileSize
1.22 KB

New text is:

Update notifications Not enabled

Update notifications are not enabled. It is highly recommended that you enable the update manger module from the module administration page in order to stay up-to-date on new releases. For more information, Update status handbook page.

danillonunes’s picture

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

Subscribe.

But I thought this can't be fixed on 7.x for now, until http://groups.drupal.org/node/154304 moves on.

Also, should Update Manager's name use caps?

geerlingguy’s picture

Status: Needs review » Reviewed & tested by the community

Also, should Update Manager's name use caps?

Since it's in the middle of a sentence, I could go either way, but the convention seems to be lowercase, so I'm happy with that.

I'm checking in here and RTBCing, simply because a few times now, I've seen that message, gone to the module page, and searched for 'Update status', only to realize that it's 'Update manager' now.

yoroy’s picture

Nice catch. We even have a report from the field by geerlingguy confirming this is one sneaky little bug. Lets re-connect these dots with some consistency.

webchick’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +Needs backport to D7

Since this is a legitimate bug fix, I committed and pushed to both 8.x and 7.x.

Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -String freeze, -Needs backport to D7

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