|
Hi Chuck,
applying a LIMIT is certainly a got idea. I did not suggest it yet, because I haven't looked at the surounding code yet. Once I have, I'll let you know of the result. Not limiting the query results into performance problems. Problem is, with the current implementation, you need to get all for the paging and you have to check each photo if it's private or not. Using the "view all photos" link on a big site will bring the server down easily. On a busy site, the "14 days" can also be a real problem.
The bug I posted here is a little bit different, I think. The "limit=last14" means that only photos from the last 14 days should be displayed. So what's certainly missing is the WHERE condition "date < 14 days ago". As you can see, that's constructed in the second elseif (see code snippet above). But that elseif is not executed, because of cat=all.
|