The color of cron messages does not reflect the message content. They are all green.

CommentFileSizeAuthor
#6 cron_message_2.patch780 bytesPasqualle
#3 cron_message_1.patch780 bytesPasqualle
cron_messages.patch1.79 KBPasqualle
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Needs work

i think the first fix is correct but the seond should not even be a warning. this is a new install and we want a clean experience.

webernet’s picture

webernet CreditAttribution: webernet commented

Yes, the second message is green 'by design'.

Pasqualle’s picture

Pasqualle
Location 🇭🇺 Budapest
CreditAttribution: Pasqualle commented
Status: Needs work » Needs review
FileSize
cron_message_1.patch780 bytes

ok, rerolled the first fix only

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Reviewed & tested by the community
webernet’s picture

webernet CreditAttribution: webernet commented
Status: Reviewed & tested by the community » Needs work

Shouldn't that be...
+ drupal_set_message(t('Cron run failed.'), 'error');

Pasqualle’s picture

Pasqualle
Location 🇭🇺 Budapest
CreditAttribution: Pasqualle commented
Status: Needs work » Needs review
FileSize
cron_message_2.patch780 bytes

yes, sorry

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Reviewed & tested by the community
Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

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

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