Reviewing merge request #1486: Group/Member Profile Visit counter patch
Improvements to support visitor counter on member profile pages and group profile pages.
Commits that would be merged:
Comments
i don’t believe this is necessarily in the core. Also, absolute unique visitors is probably more interesting than a simple visitor counter (like refreshing a billion times). Either way, I don’t see how this indicator is useful to the user.
Perhaps it should be a plugin.
For the time being, I’ll keep this merge request open for more comments.
I agree that it shouldn’t go in the core, but afaik the current pluginization hooks is not enough to rewrite it as a plugin atm.
Last, this is what i'm going to use atm in my statusnet install so i thought i should share :)
P.S: I agree that counting only unique visitors is the way to go. I just avoid it because of the extra db table & the extra overhead.
Cheers,
zmf


Add a new comment:
Login or create an account to post a comment