Needs review
Project:
Token authentication
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
8 Dec 2012 at 13:20 UTC
Updated:
14 Dec 2012 at 23:49 UTC
Use dynamic queries instead of hardcoded sql.
| Comment | File | Size | Author |
|---|---|---|---|
| tokenauth_user_delete.patch | 570 bytes | lund.mikkel |
Comments
Comment #1
Grayside commentedA bug in the delete code was already fixed with #1744844: Argument 2 passed to db_query() must be an array null given, called in tokenauth.module on line 186.
Are you arguing that Tokenauth should use PDO code in general to achieve more database agnosticism? I've been waiting for practical use cases before taking on that performance overhead for what are relatively straightforward queries.