drush cron reports success even if the cron run fails (i.e. cron is already running or there is a stale semaphore)

CommentFileSizeAuthor
#5 drush-cron-fail.patch741 bytesjohn morahan
#2 drush-cron-fail.patch732 bytesjohn morahan
drush-cron-fail.patch707 bytesjohn morahan

Comments

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Needs work

Good catch. We should use drush_error() and drush_log() instead of drush_print()

john morahan’s picture

john morahan commented
Status: Needs work » Needs review
StatusFileSize
new drush-cron-fail.patch732 bytes

Ah... like this?

moshe weitzman’s picture

moshe weitzman commented

The success message should be drush_log(dt('Cron successfully run'), 'success') or somesuch.

john morahan’s picture

john morahan commented
Status: Needs review » Needs work

Oh, wait, you meant drush_log for success.

john morahan’s picture

john morahan commented
Status: Needs work » Needs review
StatusFileSize
new drush-cron-fail.patch741 bytes
moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

committed. thx.

Status: Fixed » Closed (fixed)

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