Warning: in_array() expects parameter 2 to be array, null given in workbench_scheduler_manage_schedules_access_check()

Warning is appearing on every node page.

CommentFileSizeAuthor
#4 workbench_scheduler-access_callback_fix-2538586-4.patch549 bytesbotanic_spark
#1 workbench_scheduler-access_callback_fix-2538586-1.patch797 bytesbotanic_spark

Comments

botanic_spark’s picture

botanic_spark commented
StatusFileSize
new workbench_scheduler-access_callback_fix-2538586-1.patch797 bytes

Patch is just checking if variable $types exists.

botanic_spark’s picture

botanic_spark commented
Status: Active » Needs review
mikemiles86’s picture

mikemiles86
he/him
commented
Status: Needs review » Needs work

@botanic_spark

You need to update the reference paths in your patch, to just be relative to the module. Currently they point from your local docroot.

diff --git a/docroot/profiles/olympics/modules/contrib/workbench_scheduler/workbench_scheduler.module b/docroot/profiles/olympics/modules/contrib/workbench_scheduler/workbench_scheduler.module
index e9cd2a4..24f53f9 100644
--- a/docroot/profiles/olympics/modules/contrib/workbench_scheduler/workbench_scheduler.module
+++ b/docroot/profiles/olympics/modules/contrib/workbench_scheduler/workbench_scheduler.module
botanic_spark’s picture

botanic_spark commented
StatusFileSize
new workbench_scheduler-access_callback_fix-2538586-4.patch549 bytes

@mikemiles86 Thanks for noticing that. Here is update.

botanic_spark’s picture

botanic_spark commented
Status: Needs work » Needs review

mikemiles86’s picture

mikemiles86
he/him
commented
Status: Needs review » Fixed

@botanic_spark

Thanks for your updated patch. work looks good. I took it just a bit further to also make sure that the node was successfully loaded.

Fix will be out in the next release!

Status: Fixed » Closed (fixed)

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