EntityFieldQuery::execute documentation - return value example doesn't work if query result is empty [#1427098]

API page: http://api.drupal.org/api/drupal/includes--entity.inc/function/EntityFie...

Dear all,

The documentation of 'Return Value' states:

    $result = $query->execute();
    $entities = entity_load($my_type, array_keys($result[$my_type]));

It seems that this only works when the result is nonempty. Otherwise $result[$my_type] causes an error message Notice: Undefined index: model in .... To fix this, you must explicitly check for empty results, not so elegant, but necessary it seems.

Kaspar

Comment	File	Size	Author
#9	1427098-d7-version-2.patch	1.8 KB	chris.leversuch
#9
#7	1427098-d7-version.patch	1.8 KB	chris.leversuch
#7
#6	1427098-d7.patch	1.8 KB	chris.leversuch
#6
#3	1427098.patch	1.86 KB	chris.leversuch
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 5 February 2012 at 17:52

Title:	Documentation problem with EntityFieldQuery::execute	» EntityFieldQuery::execute documentation - return value example doesn't work if query result is empty
Version:	7.x-dev	» 8.x-dev
Issue tags:		+Novice, +Needs backport to D7

Thanks for reporting this! I think a novice patch contributor could probably fix this.
It appears that it needs fixing in Drupal 8 first:
http://api.drupal.org/api/drupal/core--modules--entity--entity.query.inc...

I'm not sure why the Drupal 8 and 7 versions of this method don't link to each other... probably because the file name has changed. Anyway, that's a separate issue, which I'll file in the API module.

Comment #2

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 5 February 2012 at 17:58

For anyone interested, here's the issue about the methods not linking to each other:
#1429314: Other versions linking between branches on methods pages seems to be broken

Comment #3

chris.leversuch CreditAttribution: chris.leversuch commented 18 February 2012 at 16:58

Status:

Active

» Needs review

File	Size
1427098.patch	1.86 KB

I've added "The entity type will only exist in the outer array if results were found." to the return description and an if statement has been added to the sample code.

I've also adjusted the line wrap on the first line as the text was wrapping too early - I can remove this change if it shouldn't be included.