PDOException:Invalid text representation when attempting to load an entity with a string ID

Marked #1795918: Can only flip STRING and INTEGER values! as a duplicate of this issue.

This patch worked for me.

eta: I take it back. It blew up some Views. Will investigate...

The patch in #0 no longer reflects the latest patch #1003788: PostgreSQL: PDOException:Invalid text representation when attempting to load an entity with a string or non-scalar ID.

The newest patch from #1003788 actually looks a lot more like #1795918: Can only flip STRING and INTEGER values! #13 which was marked duplicate.

I think the right solution for the time being may be to simply wrap the array_flip parameter with array_filter as #1795918 did, until core is solved.

#0 works for me so far... I don't see any regression like #2. Will keep you posted if issue.

Hi All,

In Entity Cache 7.x-1.1 I see the same problem when somebody go to the URL /node/sites or similar.

PDOException: SQLSTATE[22P02]: Invalid text representation: 7 ERROR: la sintaxis de entrada no es válida para integer: «sites» LINE 5: WHERE (base.nid IN ('sites')) ^: SELECT revision.vid AS vid, base.uid AS uid, revision.title AS title, revision.log AS log, revision.status AS status, revision.comment AS comment, revision.promote AS promote, revision.sticky AS sticky, base.nid AS nid, base.type AS type, base.language AS language, base.created AS created, base.changed AS changed, base.tnid AS tnid, base.translate AS translate, revision.timestamp AS revision_timestamp, revision.uid AS revision_uid FROM {node} base INNER JOIN {node_revision} revision ON revision.vid = base.vid WHERE (base.nid IN (:db_condition_placeholder_0)) ; Array ( [:db_condition_placeholder_0] => sites ) en EntityCacheControllerHelper::entityCacheLoad() (línea 95 de /var/www/html/sites/all/modules/entitycache/entitycache.module).

I think that if the node id is not a number the module must send a page not found message.

Thx by this great module.

Agreeing with #4: The patch in #0 works for me so far, no broken views. The proposal in #3 seems better, but at least some solution should make it into a release, no? ;)

For SEO reference, the error messages that the #0 patch fixed for me had been these – different from the original report, but similar to duplicate #1795918: Can only flip STRING and INTEGER values!:

Warning: array_flip(): Can only flip STRING and INTEGER values! in EntityCacheControllerHelper::entityCacheLoad() (line 69 of [...]/modules/contrib/entitycache/entitycache.module).
Warning: array_flip(): Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->cacheGet() (line 350 of [...]/includes/entity.inc).

Repeated several times, and appearing esp. in Organic Group pages of groups. I use entitycache 7.x-1.2 as part of Drupal Commons 7.x-3.9.

entitycache-numeric-ids.patch queued for re-testing.

#0 fixes the issue. RTBC.

This patch worked for me. Thanks.

#0 works and is critical for PostgreSQL sites. Can we have this in please?

Sorry to butt into an issue that is already marked RTBC, but D8 has this fix already and a D7 backport is RTBC in core after a more extensive discussion (#1003788: PostgreSQL: PDOException:Invalid text representation when attempting to load an entity with a string or non-scalar ID).

Attached is that same core patch copy-pasted into Entitycache, it has more or less the same logic as #0

@stefan.r that patch instantly gave me a WSOD.
And this on drush cc all

PHP Fatal error: Using $this when not in object context in sites/all/modules/contrib/entitycache/entitycache.module on line 95
Drush command terminated abnormally due to an unrecoverable error. [error]
Error: Using $this when not in object context in sites/all/modules/contrib/entitycache/entitycache.module, line 95

Followed by this on registry rebuild:
drush rr

PHP Fatal error: Using $this when not in object context in sites/all/modules/contrib/entitycache/entitycache.module on line 95
Drush command terminated abnormally due to an unrecoverable error. [error]
Error: Using $this when not in object context in sites/all/modules/contrib/entitycache/entitycache.module, line 95

Another issue with #11 is it uses a lamda function. Unfortunately, that isn't so cool in older versions of PHP (< 5.3).

I'll address this in #1003788: PostgreSQL: PDOException:Invalid text representation when attempting to load an entity with a string or non-scalar ID.. As soon as that patch goes in (if ever ;) it can be ported to Entity Cache.

#1003788: PostgreSQL: PDOException:Invalid text representation when attempting to load an entity with a string or non-scalar ID is in. So?

we can just copypaste the cleanIds() from there -- though we'd then have a minimum drupal version requirement for entity cache.

maybe we can only do the entity ID sanitization if people have drupal 7.36 and up?

just a note: conflicts with RTBC https://www.drupal.org/node/1851398

It may be good to log the error as it seems likely that there may be an error in code that is passing values up the stack. At least array_flip errors seem to be popping up like that for me;)

This patch is just copying code from the core entity controller -- it simulates MySQL behavior by filtering out invalid IDs. Other database systems throw an exception.

If we were to log this, shouldn't we be fixing that in core first?

Probably should be fixed in core, but 'first' is debatable, either case maybe it's a follow-up.

drush rr --fire-bazooka
Error: Using $this when not in object context in
modules/all/entitycache/entitycache.module, line 167

$entity_info = entity_get_info($this->entityType);

created a follow-up issue at #2516094: Log an error when attempting to load an entity with a non-numeric ID

@stefan.r, I tested the patch in #27. It didn't fix our issue, but it did cause us to realize that there was a function error causing our problems.

However, I did want to inform you of a change that's needed in the patch. When making a dependency on Drupal, the typical way to do so is as follows:

dependencies[] = system (>=7.36)

The word "drupal" is not seen as a module name, and actually will cause Entitycache (and dependent modules) to becomes disabled because Drupal is seen as "missing." Thanks.

Now that this is in 7.x, I'm happy to commit the same code to entitycache, although looks like #28 needs fixing.

reroll

Committed/pushed to 7.x-1.x, thanks!