API page: http://api.drupal.org/api/drupal/modules--user--user.api.php/function/ho...

Describe the problem you have found:

CommentFileSizeAuthor
#3 user.api_.php_.patch613 bytesTR
user.api_.php_.patch462 bytesTR
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
CreditAttribution: mr.baileys commented
Status: Needs review » Reviewed & tested by the community

Good catch, and the patch is RTBC.

One additional change I'd consider making is changing the query in the example to SELECT uid, foo FROM {my_table} WHERE uid IN (:uids) so it's clearer where foo comes from.

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
CreditAttribution: mr.baileys commented
Issue tags: +Needs backport to D7
TR’s picture

TR CreditAttribution: TR commented
Status: Reviewed & tested by the community » Needs review
FileSize
user.api_.php_.patch613 bytes

Re: #1
I thought the same thing, so let's just do it. Here's a revised patch.

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
CreditAttribution: mr.baileys commented
Status: Needs review » Reviewed & tested by the community

Thanks!

Dries’s picture

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

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

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