[feature] Added ->toSql() for raw wheres

[rennokki]

See this open PR i closed a while: #1887

It's good to make it work not only because of debugging (either it's not called SQL), but also you can cache query-by-query easier using packages like this: https://github.com/rennokki/laravel-eloquent-query-cache

Needs testing.

Actually it is safe, it detects arrays and automatically serializes them to string, which is exactly what's needed.

[rennokki]

Tests are failing due to Coveralls. This might be needed to PR before re-checking the tests: #1922

[jenssegers]

Maybe we can put this under a category called "debugging" or "inspecting queries".

If I remember correctly there was also a way to get the mongodb query. That could also fit under that category. Will also be nice to be able to send this link to people having issues.

[Smolevich]

Add else info about DB::getQueryLog()

[Smolevich]

It old feature but getQueryLog() return original query struncture to mongodb

[rennokki]

@jenssegers maybe fix this after #1917

[rennokki]

@Smolevich i will look into it as soon as i get home

[Smolevich]

@rennokki what about to add info about using DB::getQueryLog()
For example

DB::enableQueryLog();
$user = factory(User::class)->create();
$query = DB::connection()->table(config('auth.passwords.users.table'));
dd(DB::getQueryLog());

Example of element of array that function getQueryLog returned

0 => array:3 [
    "query" => "users.insertOne({"name":"Mr. Jayden Cremin Sr.","email":"[email protected]","email_verified_at":{"$date":{"$numberLong":"1579992655000"}},"password":"$2y$10$92IXUNpkjO0rOQ5byMi.Ye4oKoEa3Ro9llC\/.og\/at2.uheWG\/igi","remember_token":"r6HicH5d78","updated_at":{"$date":{"$numberLong":"1579992655901"}},"created_at":{"$date":{"$numberLong":"1579992655901"}}})"
    "bindings" => []
    "time" => 48.52
  ]
1 => array:3 [
    "query" => "users.find({"email":"[email protected]"},{"limit":1,"typeMap":{"root":"array","document":"array"}})"
    "bindings" => []
    "time" => 4.49
  ]

[rennokki]

The scope of this PR was to be able to get the query string before the actual query was executed, so i can be able to add cache query-by-query.

I can add the query log thing in the readme, because that's useful.

[rennokki]

@Smolevich @Giacomo92

So far, the single issue here is that the basic get() and the aggregate and groups are treated differently. This means that the final query is split into two methods rather than a single get() one.

I repeat - the purpose of this PR was to get the string for each query, so I can be able to query the cache query-by-query.

[Smolevich]

@rennokki, do you see this method https://github.com/jenssegers/laravel-mongodb/blob/a8918acadb79f4f90d42eaddf0e996054ddd8184/src/Jenssegers/Mongodb/Query/Builder.php#L413?

[rennokki]

@Smolevich I think $options and $pipeline has to be attributes in the class and be passed to the generateCacheKey method.

[Smolevich]

Add param doc for argument $query and for return type please

[Smolevich]

@rennokki please resolve conflcts

[divine]

@rennokki this has been open for almost a month, will you take care?

Looks like no as you've deleted forked repo while at the same trying to reinvent a wheel.

Good luck, closing.