Wrong model count

[evan108108]

Wrong model count

[evan108108]

The reason we don't use "count()" is that query results can be limited so
you will not get the total count. Imagine there are 100,000 results, in
this case we may return 100, if we use "count" the result count would be
100 and not 100,000.

If you are manipulating the findAll just make sure you do the same with the
count event. Remember that inside event callbacks that $this context is
your controller, so you can easily access any abstracted logic and apply to
both count and findAll events.

If you have noticed any specific instances (where you have not manipulated
the findall) and counts are incorrect, I would love to know about them as
that would imply a bug. Also I am open to suggestion if you have a better
way to do the count.

-Evan

On Wednesday, October 16, 2013, blitzjs wrote:

I've noticed in several instances where the model count is not correct,
especially if you modify the find.all events.

In the render event why not just use count($models) instead of performing
a stat query? The count should really reflect what is being passed as $data
and not gathered from another source (although in theory they should
match). Plus, doing a simple count() call will save another query execution.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/53
.