The docs for sess_count() don't entirely match the implementation. The function either returns the number of anonymous sessions or the number of authenticated sessions, but never both.

CommentFileSizeAuthor
#2 20080812.sess_count_docs-2.patch1.01 KBbart jansens
20080810.sess_count_docs.patch989 bytesbart jansens

Comments

dries’s picture

dries commented

In the documentation, the parameter type is set as int, but it should probably be a boolean too? It is pretty minor but probably worth tacking onto this patch.

bart jansens’s picture

bart jansens commented
StatusFileSize
new 20080812.sess_count_docs-2.patch1.01 KB

Oops, missed that one. Here is an updated patch.

dries’s picture

dries commented
Status: Needs review » Fixed

Thanks Bart. I've committed your last patch to DRUPAL-6 and CVS HEAD. Thanks.

Anonymous’s picture

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

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