Memberlist Display

[Rich]

My memberlist is hundreds of pages long because it shows members from 9 years ago who meet the post count criteria. I am hoping that a simple plugin could change that so that it shows members who meet the post criteria but have also been active over the past year. This would drastically cut down on the number of people shown. I realize an advanced search can be done to refine the search criteria but I am optimistic that this could be easily achieved as a default setting via a plugin. Does anyone know how to achieve this? I ran several searches here, on vb.com as well as google and came up with nothing.

Thanks for your time.

[Christos Teriakis]

Just a "dirty" solution which means to manually edit the memberlist.php (repeating it on any vB upgrade). With a text editor (like Notepad) open the file memberlist.php. On line 375 you'll see:

$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'];

just above enter:

$timecondition = time() - (365 * 24 * 60 *60);

then replace the line:

$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'];

with:

$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'] .' AND lastactivity >= $timecondition';

If you want you can change lastactivity with lastvisit or lastpost. This restriction will limit the memberlist to members who have an activity during last 365 days.
Finally upload memberlist.php to your vb installation.

Chris

[Rich]

It shows an error Chris.

Invalid SQL:

SELECT COUNT(*) AS users
FROM 1user AS user


WHERE 1=1 AND posts >= 1 AND lastactivity >= $timecondition
AND (user.usergroupid IN (-1,22,39,28,15,40,42,6,7,32,2,37,5));

MySQL Error : Unknown column '$timecondition' in 'where clause'

I then put " around the timecondition variable and it got rid of the error but it still showed every listing. I even tried changing the >= to <= but that displays a completely empty memberlist. lol

[Christos Teriakis]

Originally Posted by Rich

It shows an error Chris.



I then put " around the timecondition variable and it got rid of the error but it still showed every listing. I even tried changing the >= to <= but that displays a completely empty memberlist. lol

Let me check it please.

--------------- Added [DATE]1358271095[/DATE] at [TIME]1358271095[/TIME] ---------------

How you added '' ? Must be '.$timecondition.'

[mokujin]

The las line should be:
$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'] .' AND lastactivity >=' $timecondition;

???

[Christos Teriakis]

Rich,

You also need to edit line 174.
Replace:

$condition = '1=1';

With:

$timecondition = time() - (365 * 24 * 60 *60);
$condition = '1=1 AND lastactivity >= '.$timecondition;

--------------- Added 15 Jan 2013 at 19:47 ---------------

Originally Posted by mokujin

The las line should be:
$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'] .' AND lastactivity >=' $timecondition;

???

When there is a variable (like $timecondition) must always exist a . after '. Both ways are correct:

$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'] .' AND lastactivity >='. $timecondition;
...or...

$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'] .' AND lastactivity >=' .$timecondition.'';

[Rich]

Thank you Chris. That did the trick!

Btw, this is what I did before I posted the issue:

$condition .= ' AND posts >= ' . $vbulletin->options['memberlistposts'] .' AND lastactivity >= "$timecondition"';

I added this: "$timecondition".