SQLLiteral with Data interpolation

[bikram990]

What did you do?

I created a SQLLiteral with one of the interpolation as Binary Data.

What did you expect to happen?

I expected my Data to bind as whole.

What happened instead?

Bindings where created for each byte in the data and final sql ended up having to many bindings.

Environment

GRDB flavor(s): GRDB
GRDB version: 4.1.0
Installation method: CocoaPods
Xcode version: 10.2.1
Swift version: 5
Platform(s) running GRDB: iOS
macOS version running Xcode: 10.14.6

Demo SQL

try db.execute(literal: """ INSERT INTO tableName (identifier, data) VALUES (\(item.identifier), \(item.data)) """)

resulted in

INSERT INTO tableName (identifier, data) VALUES (?, (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?))

I know my expectations might be wrong here, as GRDB uses string interpolation and according to that I'm supposed to pass a string convertible item instead of Data. But I see that GRDB is able to recognize Data as a special type and has inserted () to contain its value. If GRDB is able to recognize this then why not support Data in SQLLiteral.

[groue]

Hello @bikram990,

What is the exact type of item.data?

[bikram990]

Hi @groue ,
item.data is of Data type and the column is of blob type.

[groue]

Oops, sorry for the useless question. I could reproduce the unexpected interpolation.

The reason is that the Data conformance to the Sequence protocol kicks in when it should not: sql interpolation sees the data as a sequence of bytes instead of a whole data.

This will be fixed in the next GRDB version. Meanwhile, you can use this workaround:

// temp workaround for https://github.com/groue/GRDB.swift/issues/591
// Hide the Sequence in Data with item.data.databaseValue
"INSERT ... VALUES (..., \(item.data.databaseValue))"

[groue]

Fixed in v4.2.1