The function user_status_user() returns 'false' when operations aren't 'delete' and 'update'. This is not as dictated by the API, and can result in undesired behavior of other modules. For example a module that listens to the operation 'view', which should return a meaningful array.

Attached is a patch that will fix this minor issue.

CommentFileSizeAuthor
user_status_hookuser.patch487 bytesmaartenvg

Comments

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Yup, good point. That was cruft left in this module since revision 1.1, which I never noticed or removed in my various attempts to clean it up. ;) Committed to HEAD and DRUPAL-5.

maartenvg’s picture

maartenvg commented

Thanks :)

Anonymous’s picture

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

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