Hi

I stumbled upon the case where one of the queues didn't have a maximum execution time set (Queue Mail module). This is actually a quite trivial situation as the parameter is optional (defaults to 15s).

So to omit any error messages, the following should get incorporated (line 1616 elysia_cron.module):
Original:

  'description' => $queue_name . ' queue processing (Items count: <strong>' . $queue->numberOfItems() . '</strong>, worker max duration: <strong>' . $info['time'] . 's</strong>)',

Fixed:

  'description' => $queue_name . ' queue processing (Items count: <strong>' . $queue->numberOfItems() . '</strong>, worker max duration: <strong>' . (isset($info['time']) ?  $info['time'] : 15) . 's</strong>)',

Patch will follow.

-Propaganistas

CommentFileSizeAuthor
#1 elysia_cron-errorfix_QueueTimeParameter-1984392-1.patch960 bytesAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous’s picture

Status: Active » Needs review
gotheric’s picture

Assigned: Unassigned » gotheric
Status: Needs review » Fixed

Committed in dev with a little mod:
(isset($info['time']) ? $info['time'] . 's' : t('unspecified') . ' (15s)')

Thanks

Status: Fixed » Closed (fixed)

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