on line 950 in flag.module

its declared but never used.

plus global $user is delicate, we should use user_uid_optional_load() instead if its really necessary :)

CommentFileSizeAuthor
#1 flag-cleanup-global-user-1890520-1.patch368 bytesalexweber

Comments

alexweber’s picture

alexweber commented
Status: Active » Needs review
StatusFileSize
new flag-cleanup-global-user-1890520-1.patch368 bytes

silly little patch attached

joachim’s picture

joachim commented
Status: Needs review » Fixed

Thanks for the cleanup! Committed.

Status: Fixed » Closed (fixed)

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