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

Comments

moshe weitzman’s picture

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 commented

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

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Needs work » Needs review
StatusFileSize
new cron_message_1.patch780 bytes

ok, rerolled the first fix only

moshe weitzman’s picture

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

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
commented
Status: Needs work » Needs review
StatusFileSize
new cron_message_2.patch780 bytes

yes, sorry

moshe weitzman’s picture

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

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

Committed to CVS HEAD. Thanks.

Anonymous’s picture

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

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