Open Bug 483318 Opened 16 years ago Updated 2 years ago

mozIStorageStatementParams doesn't let you bind an array

Categories

(Toolkit :: Storage, defect, P3)

defect

Tracking

()

People

(Reporter: sdwilsh, Unassigned)

References

(Blocks 3 open bugs, )

Details

And we really should be able to bind an array of ints, which is how we return binary blobs anyway.
Flags: in-testsuite?
bindBlobParameter() works fine, but i notice it's deprecated..
(In reply to comment #1)
> bindBlobParameter() works fine, but i notice it's deprecated..
bindBlobByName or bindBlobByIndex have replaced it.
Priority: -- → P3
Depends on: 1336944
We could use https://www.sqlite.org/carray.html but we should be careful, the provided query should never allow an user provided argument, we should only support it through Storage API. Note: I'm not sure which part of the carray syntax, if any, can be bound, I'm assuming its arguments can, so the idea currently is:
parse queries for "carray" and throw  if the any of its 2 arguments is not bound.
Sqlite.jsm may also allow something like "IN :myarray" or "IN ?" and just do a string replacement internally, while cpp API may only support a straight carray(?,?) or carray(:myarray, :mylen).
Blocks: 1336943
Blocks: 1047818
Severity: normal → S3
Blocks: 1339390
You need to log in before you can comment on or make changes to this bug.