WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 3 years ago

#6644 closed defect (bug) (fixed)

Prepare MySQL Queries

Reported by: filosofo Owned by: ryan
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: General Keywords: prepare wpdb has-patch
Focuses: Cc:

Description

Misc. wpdb prepares throughout trunk. I've tried to avoid prepares where the data is already escaped.

Attachments (13)

prepared_queries1.diff (2.7 KB) - added by filosofo 6 years ago.
prepared_queries2.diff (2.8 KB) - added by filosofo 6 years ago.
prepared_queries3.diff (6.9 KB) - added by filosofo 6 years ago.
prepared_queries4.diff (3.1 KB) - added by filosofo 6 years ago.
prepared_queries5.diff (3.4 KB) - added by filosofo 6 years ago.
prepared_queries6.diff (3.1 KB) - added by filosofo 6 years ago.
prepared_queries7.diff (2.8 KB) - added by filosofo 6 years ago.
prepared_queries8.diff (5.0 KB) - added by filosofo 6 years ago.
prepared_queries9.diff (4.9 KB) - added by filosofo 6 years ago.
prepared_queries10.diff (12.6 KB) - added by filosofo 6 years ago.
prepared_queries11.diff (3.3 KB) - added by filosofo 6 years ago.
prepared_queries12.diff (2.9 KB) - added by filosofo 6 years ago.
prepared_queries13.diff (1.3 KB) - added by filosofo 6 years ago.

Download all attachments as: .zip

Change History (21)

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

filosofo6 years ago

comment:1 ryan6 years ago

Looks good to me. I want to run it by MarkJaquith too since he is most familiar with prepare.

comment:2 ryan6 years ago

(In [7645]) Prepare DB queries in more places. Props filosofo. see #6644

comment:3 ryan6 years ago

(In [7657]) Add missing comma. see #6644

comment:4 ryan6 years ago

(In [7658]) Typo fixes. see #6644

comment:5 ryan6 years ago

(In [7693]) Strip slashes before preparing link for insert. see #6644

comment:6 ryan6 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.