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:

Version 1
  • Version 1
  • aeef78b
  • 79efcd8
  • Group/Member Profile Visit counter patch

Showing aeef78b-79efcd8

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

How to apply this merge request to your repository