Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

add support for limit() and skip() #172

Merged
merged 1 commit into from

2 participants

@jmikola
Owner

One question: does this lead to limit(0) always being appended to the query log? IIRC, the limit defaults to 0 and might always be set.

@scourgen

I did some test, When I don't set the limit manually(such like use findAll(),findById(),etc ), there's no $log['limit'] anyway.So It won't be a problem.

@jmikola jmikola merged commit c711aa0 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 28, 2013
  1. @scourgen
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 0 deletions.
  1. +4 −0 DataCollector/PrettyDataCollector.php
View
4 DataCollector/PrettyDataCollector.php
@@ -108,6 +108,10 @@ public function collect(Request $request, Response $response, \Exception $except
}
}
$query .= ')';
+ } elseif (isset($log['skip'])) {
+ $query .= '.skip('.$log['skipNum'].')';
+ } elseif (isset($log['limit'])) {
+ $query .= '.limit('.$log['limitNum'].')';
} elseif (isset($log['createCollection'])) {
$query .= '.createCollection()';
} elseif (isset($log['createDBRef'])) {
Something went wrong with that request. Please try again.