From d7fc5adc22c71261dd3f4544a3a5a1816b8d724e Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 12:38:37 +1000 Subject: [PATCH] Performance: Use only one INSERT INTO (with multiple rows) for tracked cites --- post.php | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/post.php b/post.php index 21d294cd..7ca9c1a9 100644 --- a/post.php +++ b/post.php @@ -682,15 +682,14 @@ if (isset($_POST['delete'])) { incrementSpamHash($post['antispam_hash']); } - if (isset($post['tracked_cites'])) { + if (isset($post['tracked_cites']) && count($post['tracked_cites'])) { + $insert_rows = array(); foreach ($post['tracked_cites'] as $cite) { - $query = prepare('INSERT INTO ``cites`` VALUES (:board, :post, :target_board, :target)'); - $query->bindValue(':board', $board['uri']); - $query->bindValue(':post', $id, PDO::PARAM_INT); - $query->bindValue(':target_board',$cite[0]); - $query->bindValue(':target', $cite[1], PDO::PARAM_INT); - $query->execute() or error(db_error($query)); + $insert_rows[] = '(' . + $pdo->quote($board['uri']) . ', ' . (int)$id . ', ' . + $pdo->quote($cite[0]) . ', ' . (int)$cite[1] . ')'; } + query('INSERT INTO ``cites`` VALUES ' . implode(', ', $insert_rows)) or error(db_error());; } if (!$post['op'] && strtolower($post['email']) != 'sage' && !$thread['sage'] && ($config['reply_limit'] == 0 || $numposts['replies']+1 < $config['reply_limit'])) {