beyondcode /
laravel-query-detector
| @@ 8-48 (lines=41) @@ | ||
| 5 | use Illuminate\Support\Collection; |
|
| 6 | use Symfony\Component\HttpFoundation\Response; |
|
| 7 | ||
| 8 | class Alert implements Output |
|
| 9 | { |
|
| 10 | public function output(Collection $detectedQueries, Response $response) |
|
| 11 | { |
|
| 12 | if ($response->isRedirection()) { |
|
| 13 | return; |
|
| 14 | } |
|
| 15 | ||
| 16 | $content = $response->getContent(); |
|
| 17 | ||
| 18 | $outputContent = $this->getOutputContent($detectedQueries); |
|
| 19 | ||
| 20 | $pos = strripos($content, '</body>'); |
|
| 21 | ||
| 22 | if (false !== $pos) { |
|
| 23 | $content = substr($content, 0, $pos) . $outputContent . substr($content, $pos); |
|
| 24 | } else { |
|
| 25 | $content = $content . $outputContent; |
|
| 26 | } |
|
| 27 | ||
| 28 | // Update the new content and reset the content length |
|
| 29 | $response->setContent($content); |
|
| 30 | ||
| 31 | $response->headers->remove('Content-Length'); |
|
| 32 | } |
|
| 33 | ||
| 34 | protected function getOutputContent(Collection $detectedQueries) |
|
| 35 | { |
|
| 36 | $output = '<script type="text/javascript">'; |
|
| 37 | $output .= "alert('Found the following N+1 queries in this request:\\n\\n"; |
|
| 38 | foreach ($detectedQueries as $detectedQuery) { |
|
| 39 | $output .= "Model: ".addslashes($detectedQuery['model']). " => Relation: ".addslashes($detectedQuery['relation']); |
|
| 40 | $output .= " - You should add \"with(\'".$detectedQuery['relation']."\')\" to eager-load this relation."; |
|
| 41 | $output .= "\\n"; |
|
| 42 | } |
|
| 43 | $output .= "')"; |
|
| 44 | $output .= '</script>'; |
|
| 45 | ||
| 46 | return $output; |
|
| 47 | } |
|
| 48 | } |
|
| @@ 6-38 (lines=33) @@ | ||
| 3 | use Illuminate\Support\Collection; |
|
| 4 | use Symfony\Component\HttpFoundation\Response; |
|
| 5 | ||
| 6 | class Console implements Output |
|
| 7 | { |
|
| 8 | public function output(Collection $detectedQueries, Response $response) |
|
| 9 | { |
|
| 10 | if ($response->isRedirection()) { |
|
| 11 | return; |
|
| 12 | } |
|
| 13 | $content = $response->getContent(); |
|
| 14 | $outputContent = $this->getOutputContent($detectedQueries); |
|
| 15 | $pos = strripos($content, '</body>'); |
|
| 16 | if (false !== $pos) { |
|
| 17 | $content = substr($content, 0, $pos) . $outputContent . substr($content, $pos); |
|
| 18 | } else { |
|
| 19 | $content = $content . $outputContent; |
|
| 20 | } |
|
| 21 | // Update the new content and reset the content length |
|
| 22 | $response->setContent($content); |
|
| 23 | $response->headers->remove('Content-Length'); |
|
| 24 | } |
|
| 25 | protected function getOutputContent(Collection $detectedQueries) |
|
| 26 | { |
|
| 27 | $output = '<script type="text/javascript">'; |
|
| 28 | $output .= "console.warn('Found the following N+1 queries in this request:\\n\\n"; |
|
| 29 | foreach ($detectedQueries as $detectedQuery) { |
|
| 30 | $output .= "Model: ".addslashes($detectedQuery['model']). " => Relation: ".addslashes($detectedQuery['relation']); |
|
| 31 | $output .= " - You should add \"with(\'".$detectedQuery['relation']."\')\" to eager-load this relation."; |
|
| 32 | $output .= "\\n"; |
|
| 33 | } |
|
| 34 | $output .= "')"; |
|
| 35 | $output .= '</script>'; |
|
| 36 | return $output; |
|
| 37 | } |
|
| 38 | } |
|
| 39 | ||
