From my prior patches to add configurable content types for post count I mentioned that for my current project I am using a shared user table with multiple sites and so needed to have post count per user per site.

Here is my patch to allow for selectable post count profile field.

CommentFileSizeAuthor
#4 user_stats_post_field.patch8.06 KBliam mcdermott
#4 user_stats_post_field.patch8.06 KBliam mcdermott
#3 user_stats.module.patch4.84 KBjaydub
user_stats.module.patch4.78 KBjaydub

Comments

liam mcdermott’s picture

liam mcdermott commented

Wow, this looks good. Hope you had a good holiday (or vacation, depending on what you call it)!

Am going to do a version 1.0 release of User stats before we head into this stuff. It's a pretty big change. I also want to get rid of the dependency on profile.module. Then port to Drupal 6.

For the moment though, am too busy to work on modules, hopefully in the next couple of weeks I'll get some free time. Thanks for this though, looks good.

liam mcdermott’s picture

liam mcdermott commented
Status: Needs review » Needs work

Needs a re-roll. Scratch what I said about getting this in after version one. Let's get it in now. :)

jaydub’s picture

jaydub commented
Status: Needs work » Needs review
StatusFileSize
new user_stats.module.patch4.84 KB

re-rolled

liam mcdermott’s picture

liam mcdermott commented
StatusFileSize
new user_stats_post_field.patch8.06 KB
new user_stats_post_field.patch8.06 KB

I've edited your patch a bit: the, installer created, user_post_count field is used by default as the post count field and the updater will variable_set('user_stats_postcount_profile_field', 'user_post_count'); (so you personally might have to set this again for your install Jaydub).

New patch attached, have a look and let me know what you think (applies from the modules directory). :)

liam mcdermott’s picture

liam mcdermott commented
Status: Needs review » Fixed

Am impatient, so committed the patch in #4. Feel free to re-open this if that gives you any trouble. As ever, thanks for the help Jaydub. :)

jaydub’s picture

jaydub commented

looks good! you did a better job of explaining things in your form field descriptions....

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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