This API function intends to return all of a given user's flags but does not because we don't key the array by content ID. Thus, each flag overwrites the previous one in the loop and we always return one flag. One line patch attached.

CommentFileSizeAuthor
mw.patch946 bytesmoshe weitzman

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

Thanks Moshe!

mooffie’s picture

mooffie commented

(If you have time, I'd like to hear what you're using this function for. Is this for "Give me all the node IDs this user has flagged"?)

moshe weitzman’s picture

moshe weitzman commented

Yes, thats it, though Economist is flagging comments - not nodes.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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