On user_load(), the parameters documentation reports the function accepts an array, but it doesn't report the function also accept a user ID.

Files: 
CommentFileSizeAuthor
#11 872374_4.patch1.67 KBsender
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374_4.patch.
[ View ]
#9 872374_3.patch1.66 KBsender
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374_3.patch.
[ View ]
#8 872374_2.patch1.55 KBsender
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374_2.patch.
[ View ]
#3 872374.patch633 bytessender
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374.patch.
[ View ]

Comments

Issue tags:+Novice

That's true - and it's a Drupal 6 issue only. Thanks for the report.

This is a good project for a novice Drupal contributor...

Assigned:Unassigned» sender

StatusFileSize
new633 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374.patch.
[ View ]

Here's a small patch.

Status:Active» Needs review

Status:Needs review» Reviewed & tested by the community

That looks good, thanks!

Status:Reviewed & tested by the community» Needs work

Is it still good then to name the argument $array?

Well, that's a good point. I agree, we should change the argument name to something else besides $array. Perhaps $info or $attributes? That would still constitute a doc patch, as long as only the argument name is changed.

Status:Needs work» Needs review
StatusFileSize
new1.55 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374_2.patch.
[ View ]

Good point indeed. Here's a new version with a search and replace on $array with $user_info. Credits for the name: jhodgdon ;)

StatusFileSize
new1.66 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374_3.patch.
[ View ]

Improved with better wording and a list describing the different possibilities on how to use $user_info.

Status:Needs review» Needs work

One of the documentation lines needs to be wrapped at 80 characters.

Otherwise it looks OK.

Status:Needs work» Needs review
StatusFileSize
new1.67 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 872374_4.patch.
[ View ]

Fixed.

+ *   Information about the user to load, consisting of one of the following:

Should not the phrase changed with a different one?

What's wrong with that phrase?

I'm ready to mark #13 as RTBC, but not until I hear what kiamlaluno has to say about why that line needs changing. :)

What I read is not a sentence. I would rather remove that line, as the following comment lines describe already what the argument is.

@kiamlaluno: tried that before, but would that be valid Doxygen? If the line is needed, what do you suggest that would be a sentence in your view?

@jhodgdon: is this line really necessary? Can't find anything about it at https://drupal.org/node/1354#lists (or I might be blind).

Typically we don't just put a list in without a line (ending in :) right before it, explaining what the list means. In this case, I think the line needs to explain that it's either/or. You could shorten it to "One of these alternatives:" or something like that.

Status:Needs review» Reviewed & tested by the community

Status:Reviewed & tested by the community» Needs work

The last submitted patch, 872374_4.patch, failed testing.

Status:Needs work» Reviewed & tested by the community

The D6 test bot is choking on every patch. This patch is still fine.

Status:Reviewed & tested by the community» Fixed

Thanks, committed.

Status:Fixed» Closed (fixed)
Issue tags:-Novice

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