[12.x] run prepareBindings before passing bindings to Grammar::substituteBindingsIntoRawSql

[taka-oyama]

All bindings passed to Grammar::substituteBindingsIntoRawSql(...) run through Connection::prepareBindings() except for this one.

I believe this is a bug.

---

Examples of other places that call Grammar::substituteBindingsIntoRawSql(...)

Builder::toRawSql()

framework/src/Illuminate/Database/Query/Builder.php

Line 3025 in 071a9d3

$this->toSql(), $this->connection->prepareBindings($this->getBindings())

QueryExecuted::toRawSql()

framework/src/Illuminate/Database/Events/QueryExecuted.php

Line 69 in 071a9d3

->substituteBindingsIntoRawSql($this->sql, $this->connection->prepareBindings($this->bindings));

Connection::getRawQueryLog()

framework/src/Illuminate/Database/Connection.php

Lines 1522 to 1525 in 071a9d3

	'raw_query' => $this->queryGrammar->substituteBindingsIntoRawSql(
	$log['query'],
	$this->prepareBindings($log['bindings'])
	),

[taylorotwell]

Are we able to write a test to confirm a fixed behavior?

[taka-oyama]

I've added test to assert the behavior.

[taylorotwell]

I personally wonder if this was intentional to prevent logging sensitive information.