Sorry for my bad English. I'm from Ukraine.

Thanks for the nice module! I would like to propose changes in the work of function that display the user status .

1. Now, if the registered user after viewing the page forcibly ends the session (log out), other users will still be
see its status as online, until after the period of time specified in the settings of Online Users (User activity).

There is no test for the existence of a session by. It must be added.

2. Now if the user was not active more than 900 sec (or another time set in the user_stats module), and after to reload the page, he first sees his status as offline, and only after the second page reload
its status is changed to ONLINE. This is bad. Status should be changed immediately.

So you need to add a check for the user who is viewing the page. If the logged in user UID matches the UID of the user for which you want
status display - you need to always return status online.

Thanks.

CommentFileSizeAuthor
author_pane_status.patch1.01 KBbigferumdron
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bigferumdron’s picture

Sory its patch for 7.x-2.0-beta1 version

bigferumdron’s picture

Version: 7.x-2.x-dev » 7.x-2.0-beta1
Scyther’s picture

Thanks for the input. Have made some changes but used a bit of that code to.

Commited to dev version.
http://drupalcode.org/project/author_pane.git/commit/ae42d4d

Scyther’s picture

Issue summary: View changes
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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