ElkArte Community Modern, Free, Powerful
Community building forum software

I was looking around and I found these two functions.

They are pretty much the same (sort of), but one is used cached for BoardIndex, the other is used in action=recent.
One grab a bunch of previously filtered id_msg, the other relies on its list of JOINs.

I was thinking it may be worth merging the two, move the "pre query" (still) done in the controller to the function and use it if $_REQUEST['start'] > 0.

What do you think?

I'm generally in favor of consolidating things like that ... poor "recent" is like the step child, can't remember how many times I've had to go back and add in the same things we did for display etc (or fix the same bugs)

lol

Okay, can be done even after beta 2 I think (it's not such a big issue and should not create big problems).