MySql gurus... is this query ok (it works) top referrer

[sarahk]

I'd add a check for the year as well, although I'd probably feed the current month and year into the sql rather than get MySQL to work it out.

From my experience with other databases I would expect MySQL to be working out the month(now()) and year(now()) for every row which is unnecessary processing. Far better to do it once in PHP and have it as a constant in the query.

AND YEAR( FROM_UNIXTIME( user.joindate ) ) = YEAR( now( ) )