Merge pull request laravel#772 from loic-sharma/patch-3

daylerees · daylerees · commit a0fd22f6f1c4 · 2012-07-31T08:42:47.000-07:00
Fixed bug where the profiler did not correctly put quotes around bindings
diff --git a/laravel/profiling/profiler.php b/laravel/profiling/profiler.php
@@ -5,6 +5,7 @@
 use Laravel\Event;
 use Laravel\Config;
 use Laravel\Request;
+use Laravel\Database;
 
 class Profiler {
 
@@ -57,6 +58,8 @@ public static function query($sql, $bindings, $time)
 	{
 		foreach ($bindings as $binding)
 		{
+			$binding = Database::connection()->pdo->quote($binding);
+
 			$sql = preg_replace('/\?/', $binding, $sql, 1);
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@`
`5`	`5`	`use Laravel\Event;`
`6`	`6`	`use Laravel\Config;`
`7`	`7`	`use Laravel\Request;`
	`8`	`+use Laravel\Database;`
`8`	`9`
`9`	`10`	`class Profiler {`
`10`	`11`
`@@ -57,6 +58,8 @@ public static function query($sql, $bindings, $time)`
`57`	`58`	`{`
`58`	`59`	`foreach ($bindings as $binding)`
`59`	`60`	`{`
	`61`	`+ $binding = Database::connection()->pdo->quote($binding);`
	`62`	`+`
`60`	`63`	`$sql = preg_replace('/\?/', $binding, $sql, 1);`
`61`	`64`	`}`
`62`	`65`