With entity cache enabled, I am not able to view node revisions. The following warnings show up in watchdog:

Warning: array_filter() expects parameter 1 to be array, string given in EntityCacheControllerHelper::entityCacheCleanIds() (line 215 of \sites\all\modules\entitycache\includes\entitycache.entitycachecontrollerhelper.inc).

Warning: array_map(): Argument #2 should be an array in EntityCacheControllerHelper::entityCacheCleanIds() (line 216 of \sites\all\modules\entitycache\includes\entitycache.entitycachecontrollerhelper.inc).

CommentFileSizeAuthor
#7 clean-revision-id-2816171-7.patch1.33 KBgreggles
#4 clean-revision-id-2816171-3.patch1.26 KBjordanrussellsmith
#2 clean-revision-id-2816171-2.patch1.38 KBjordanrussellsmith
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jordanrussellsmith created an issue. See original summary.

jordanrussellsmith’s picture

jordanrussellsmith CreditAttribution: jordanrussellsmith commented
Status: Active » Needs review
Related issues: +#2732621: PDOException: SQLSTATE[22018]: Conversion failed when converting the varchar value 'configure' to data type int
FileSize
clean-revision-id-2816171-2.patch1.38 KB

This patch is derived from the patch which fixes the same issue in core.

Core Patch: PDOException: SQLSTATE[22018]: Conversion failed when converting the varchar value 'configure' to data type int

Status: Needs review » Needs work

The last submitted patch, 2: clean-revision-id-2816171-2.patch, failed testing.

jordanrussellsmith’s picture

jordanrussellsmith CreditAttribution: jordanrussellsmith commented
Status: Needs work » Needs review
FileSize
clean-revision-id-2816171-3.patch1.26 KB

Fixed the path in the patch. Forgot the --relative flag.

jordanrussellsmith’s picture

jordanrussellsmith CreditAttribution: jordanrussellsmith commented
Status: Needs review » Reviewed & tested by the community

This patch is running on our live site and fixes the issue. Can we get this committed?

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

Super minor whitespace issue in the patch, otherwise it applies without issue.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles at CARD.com commented
FileSize
clean-revision-id-2816171-7.patch1.33 KB

Here's a reroll just to fix the tabs problem. Leaving at RTBC since it should be the same content. I didn't review myself.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 7: clean-revision-id-2816171-7.patch, failed testing. View results

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

Bot marked this as 'Needs Work' but it was RTBC. What does this need?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles at CARD.com commented
Status: Needs work » Needs review

The bot was broken for a bit. I think it's better now, so moving back to needs review to confirm it passes tests and then it can be moved back to RTBC.