Attempt to read property "id" on null in JobController::actionPush()

[xcopy]

yii2-queue-monitor/src/controllers/JobController.php

Lines 164 to 169 in fa5246b

	$uid = $record->getSender()->push($record->createJob());
	$newRecord = PushRecord::find()->byJob($record->sender_name, $uid)->one();
	return $this
	->success(Module::t('notice', 'The job is pushed again.'))
	->redirect(['view', 'id' => $newRecord->id]);

The $uid variable will be null in case if beforePush event has been handled (i.e. $event->handled set to true in the particular behavior).

Here is the proposed solution:

if ($uid = $record->getSender()->push($record->createJob())) {
    $newRecord = PushRecord::find()->byJob($record->sender_name, $uid)->one();

    $this->success(Module::t('notice', 'The job is pushed again.'));
    $id = $newRecord->id;
} else {
    $this->error(Module::t('notice', 'The job cannot be pushed again because the "beforePush" event has already been handled.'));
    $id = $record->id;
}

return $this->redirect(['view', 'id' => $id]);

Not sure about the error message.

What you say guys?