Closed (fixed)
Project:
Outline Designer
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Mar 2008 at 19:00 UTC
Updated:
15 Apr 2008 at 13:51 UTC
to support DB prefixes, etc. style should be '{node_type}' instead of 'node_type' and good practice is not to use the '.' string concatenation in db strings but instead %s/%d use. In D6, this is required, but no reason not to here.
(PS, as promised at drupalcon, I said I'd review your work, and submit patches issues. here i am :-)
| Comment | File | Size | Author |
|---|---|---|---|
| db_abstraction.patch | 2.11 KB | schuyler1d |
Comments
Comment #1
btopro commentedI'll test 'er out tomorrow though I'm sure this'll be fine. Thanks for the review, these comments are exactly the kinda things I need to clean this stuff up for the masses. I'll implement it tomorrow.
Comment #2
btopro commentedthanks for the patch, went back through and applied this issue to my ajax code too
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.