Posted by slinc on November 26, 2008 at 9:40am
| Project: | User Points Top Contributors |
| Version: | 6.x-1.4 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
Hi there,
I'm having problems with showing the current points of users in this module, as it only shows the lifetime points of each user.
The problem with this, is that spammers who rapidly spam hundreds of messages, will be seen as a top contributor EVEN if all their points were subtracted.
Moreover, users can abuse this system through using the user2userpoints function, and transferring all their points to and fro another account, causing them to have virtually unlimited points.
Is there a way to rank the current points instead of the lifetime points of users with this module?
Thanks for your help!
Comments
#1
The problem is basically that the top contributors module is receiving all its data from the LIFETIME POINTS, it should in fact receive the data from the CURRENT POINTS.
I would greatly appreciate it if someone could tell me how to modify the code to do so.
Thanks!
#2
True! It's easy to tweak the module by modifying only a few lines of code, but anyway, I've spotted the same issue.
My logic says that Top Contributors block should display CURRENT POINTS ranking.
#3
Changing category to feature request.
Changing priority to normal.
#4
Hi, there!
I faced the same problem as you and have made the following patch but for D6. If you apply this patch, the results on both the top-contributors page and block will be sorted by the current points instead of by lifetime points.
On the top contributors block, both current and lifetime points will be displayed, but the sort order will be by current points. I have also added some css-classes to the td's of the block, so that the table will be easier to style using css.
#5
Hi:
I tried to apply this patch to v6.x1.4 and I thought I did it correctly but I got this error message:
Macintosh:userpoints_top_contributors1.4patched username$ patch < userpoints_top_contributors.patch
(Stripping trailing CRs from patch.)
patching file userpoints_top_contributors.module
Hunk #1 FAILED at 110.
Hunk #2 FAILED at 170.
2 out of 2 hunks FAILED -- saving rejects to file userpoints_top_contributors.module.rej
Has anyone applied this patch successfully? I may well be doing something wrong so please help if you have any ideas.
#6
Hi, try with this one, I forgot to change line endings.
#7
I tried this and got exactly the same error codes.