colopl / laravel-spanner

Laravel database driver for Google Cloud Spanner
Apache License 2.0
97 stars 16 forks source link

Allow pretending for DDL statements #162

Closed matthewjumpsoffbuildings closed 10 months ago

matthewjumpsoffbuildings commented 11 months ago

If the Connection is pretending, the DDL statements should not run

Since affectingStatement already handles pretending, its fine to just pass through to that method which will return 0 as normal, allowing the generated SQL to be captured

taka-oyama commented 10 months ago

Fixed in https://github.com/colopl/laravel-spanner/pull/170.