xSpy v0.4 Review

[flash.tato]

Quote:There is a 15 second window when you initially start "spying", which may not be terribly useful for smaller boards.  I guess the purpose of this plugin isn't really to provide a history, so perhaps this is desired behaviour, though, maybe, it might be a good idea if this time interval was configurable.

Did you mean 1.5 second? Perhaps i should make the interval of the timer configurable. by adding it as a variable in the JS part.

Quote:Maybe I'm wrong on this, but I couldn't find anything which automatically clears old entries from the page.  Probably not a big issue however.

Doesn't "clear" button do the trick or maybe you meant that i should clear old entries when there are news one, or you refer on the database side?

PHP Code:

<button onclick="$('xSpy_log').update('')">{$lang->xspy_clear}</button>

---

Quote:Maybe draw a line in the list when the user starts/stops "spying", so they know where new/old entries are

Great idea!

Quote:The plugin doesn't really do extensive permissions checking - it checks whether the user can view the forum or not, but doesn't check stuff like whether the thread is unapproved; arguably not a big issue, but may not be desirable

I will be adding new checking measures, this is a serious problem, i didn't consider the approved/unapproved threads.

Quote:Static data which is logged into the xspyevents table is static, thus if changed, could be inaccurate (eg thread being moved to a different forum); arguably not a big issue, but may not be desirable

I think i will make it modular so everyone could implement new things to be spyed, and yes now i will implement builtin some moderation actives to be spied.