API page: http://api.drupal.org/api/drupal/includes--entity.inc/class/EntityFieldQ...

The "add" in the following sentence should be "and".

Note that code that is simply creating and running a field query should still use the appropriate methods add conditions on the query.

CommentFileSizeAuthor
#2 1096842-EntityFieldQuery-documentation.patch727 bytesnanotube
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Issue tags: +Novice

I think it should be
add -> to add
actually...

Good project for a novice doc contributor...

nanotube’s picture

nanotube CreditAttribution: nanotube commented
Status: Active » Needs review
Issue tags: -Novice
FileSize
1096842-EntityFieldQuery-documentation.patch727 bytes

First contrib, I think i've done this correctly...

nanotube’s picture

nanotube CreditAttribution: nanotube commented
Version: 7.x-dev » 8.x-dev

moving to D8 first.

nonzero’s picture

nonzero CreditAttribution: nonzero commented

Shouldn't this patch be applied to 7.x-dev?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community

Thanks! Please apply to d8 and d7.

nonzero: Drupal 8 is the in-development version. So fixes need to be done there first, then ported to drupal 7 if appropriate (as in this case, where d8 and d7 both have this typo).

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks!

Status: Fixed » Closed (fixed)

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