From 0bb254fdb37990958b17c1c72b59f747eab2e3f0 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 12:30:16 +1000 Subject: [PATCH 01/15] Critical bugfix: SQL typo --- templates/posts.sql | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/templates/posts.sql b/templates/posts.sql index 1e092985..e41a7f38 100644 --- a/templates/posts.sql +++ b/templates/posts.sql @@ -29,6 +29,6 @@ CREATE TABLE IF NOT EXISTS ``posts_{{ board }}`` ( KEY `thread_id` (`thread`,`id`), KEY `time` (`time`), FULLTEXT KEY `body` (`body`), - KEY `ip` (`ip`), + KEY `ip` (`ip`) ) ENGINE=MyISAM DEFAULT CHARSET=utf8mb4 AUTO_INCREMENT=1 ; \ No newline at end of file From d7fc5adc22c71261dd3f4544a3a5a1816b8d724e Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 12:38:37 +1000 Subject: [PATCH 02/15] 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'])) { From 577a8b991d10b898e0025f5f45bb38cafff5080e Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 12:52:31 +1000 Subject: [PATCH 03/15] Performance: Use only one query for validating >>X links in posts --- inc/functions.php | 32 +++++++++++++++++++++++--------- 1 file changed, 23 insertions(+), 9 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index 1bd185a9..321c65d0 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1576,29 +1576,41 @@ function markup(&$body, $track_cites = false) { $skip_chars = 0; $body_tmp = $body; - + + $search_cites = array(); + foreach ($cites as $matches) { + $search_cites[] = '`id` = ' . $matches[2][0]; + } + $search_cites = array_unique($search_cites); + + $query = query(sprintf('SELECT `thread`, `id` FROM ``posts_%s`` WHERE ' . + implode(' OR ', $search_cites), $board['uri'])) or error(db_error()); + + $cited_posts = array(); + while ($cited = $query->fetch(PDO::FETCH_ASSOC)) { + $cited_posts[$cited['id']] = $cited['thread'] ? $cited['thread'] : false; + } + foreach ($cites as $matches) { $cite = $matches[2][0]; - $query = prepare(sprintf("SELECT `thread`,`id` FROM ``posts_%s`` WHERE `id` = :id LIMIT 1", $board['uri'])); - $query->bindValue(':id', $cite); - $query->execute() or error(db_error($query)); // preg_match_all is not multibyte-safe foreach ($matches as &$match) { $match[1] = mb_strlen(substr($body_tmp, 0, $match[1])); } - if ($post = $query->fetch(PDO::FETCH_ASSOC)) { + if (isset($cited_posts[$cite])) { $replacement = '' . - '>>' . $cite . - ''; + $config['root'] . $board['dir'] . $config['dir']['res'] . + ($cited_posts[$cite] ? $cited_posts[$cite] : $cite) . '.html#' . $cite . '">' . + '>>' . $cite . + ''; $body = mb_substr_replace($body, $matches[1][0] . $replacement . $matches[3][0], $matches[0][1] + $skip_chars, mb_strlen($matches[0][0])); $skip_chars += mb_strlen($matches[1][0] . $replacement . $matches[3][0]) - mb_strlen($matches[0][0]); if ($track_cites && $config['track_cites']) - $tracked_cites[] = array($board['uri'], $post['id']); + $tracked_cites[] = array($board['uri'], $cite); } } } @@ -1612,6 +1624,8 @@ function markup(&$body, $track_cites = false) { $skip_chars = 0; $body_tmp = $body; + // TODO: Make this use fewer queries, similar to local board cites. + foreach ($cites as $matches) { $_board = $matches[2][0]; $cite = @$matches[3][0]; From 3545e2406edc441123d0796ffff12fc871bf7ec1 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 13:31:02 +1000 Subject: [PATCH 04/15] Perforamnce: Much more efficient >>X and >>>/board/X --- inc/functions.php | 111 ++++++++++++++++++++++++++++++++++------------ 1 file changed, 82 insertions(+), 29 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index 321c65d0..f260a0ce 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1623,8 +1623,66 @@ function markup(&$body, $track_cites = false) { $skip_chars = 0; $body_tmp = $body; - - // TODO: Make this use fewer queries, similar to local board cites. + + if (isset($cited_posts)) { + // Carry found posts from local board >>X links + foreach ($cited_posts as $cite => $thread) { + $cited_posts[$cite] = $config['root'] . $board['dir'] . $config['dir']['res'] . + ($thread ? $thread : $cite) . '.html#' . $cite; + } + + $cited_posts = array( + $board['uri'] => $cited_posts + ); + } else + $cited_posts = array(); + + $crossboard_indexes = array(); + $search_cites_boards = array(); + + foreach ($cites as $matches) { + $_board = $matches[2][0]; + $cite = @$matches[3][0]; + + if (!isset($search_cites_boards[$_board])) + $search_cites_boards[$_board] = array(); + $search_cites_boards[$_board][] = $cite; + } + + $tmp_board = $board['uri']; + + foreach ($search_cites_boards as $_board => $search_cites) { + $clauses = array(); + foreach ($search_cites as $cite) { + if (!$cite || isset($cited_posts[$_board][$cite])) + continue; + $clauses[] = '`id` = ' . $cite; + } + $clauses = array_unique($clauses); + + if ($board['uri'] != $_board) { + if (!openBoard($_board)) + continue; // Unknown board + } + + if (!empty($clauses)) { + $cited_posts[$_board] = array(); + + $query = query(sprintf('SELECT `thread`, `id` FROM ``posts_%s`` WHERE ' . + implode(' OR ', $clauses), $board['uri'])) or error(db_error()); + + while ($cite = $query->fetch(PDO::FETCH_ASSOC)) { + $cited_posts[$_board][$cite['id']] = $config['root'] . $board['dir'] . $config['dir']['res'] . + ($cite['thread'] ? $cite['thread'] : $cite['id']) . '.html#' . $cite['id']; + } + } + + $crossboard_indexes[$_board] = $config['root'] . $board['dir'] . $config['file_index']; + } + + // Restore old board + if ($board['uri'] != $tmp_board) + openBoard($tmp_board); foreach ($cites as $matches) { $_board = $matches[2][0]; @@ -1638,39 +1696,34 @@ function markup(&$body, $track_cites = false) { // Temporarily store board information because it will be overwritten $tmp_board = $board['uri']; - // Check if the board exists, and load settings - if (openBoard($_board)) { - if ($cite) { - $query = prepare(sprintf("SELECT `thread`,`id` FROM ``posts_%s`` WHERE `id` = :id LIMIT 1", $board['uri'])); - $query->bindValue(':id', $cite); - $query->execute() or error(db_error($query)); - - if ($post = $query->fetch(PDO::FETCH_ASSOC)) { - $replacement = '' . - '>>>/' . $_board . '/' . $cite . - ''; - - $body = mb_substr_replace($body, $matches[1][0] . $replacement . $matches[4][0], $matches[0][1] + $skip_chars, mb_strlen($matches[0][0])); - $skip_chars += mb_strlen($matches[1][0] . $replacement . $matches[4][0]) - mb_strlen($matches[0][0]); - - if ($track_cites && $config['track_cites']) - $tracked_cites[] = array($board['uri'], $post['id']); - } - } else { - $replacement = '' . - '>>>/' . $_board . '/' . - ''; + if ($cite) { + if (isset($cited_posts[$_board][$cite])) { + $link = $cited_posts[$_board][$cite]; + + $replacement = '' . + '>>>/' . $_board . '/' . $cite . + ''; + $body = mb_substr_replace($body, $matches[1][0] . $replacement . $matches[4][0], $matches[0][1] + $skip_chars, mb_strlen($matches[0][0])); $skip_chars += mb_strlen($matches[1][0] . $replacement . $matches[4][0]) - mb_strlen($matches[0][0]); + + if ($track_cites && $config['track_cites']) + $tracked_cites[] = array($_board, $cite); } + } elseif(isset($crossboard_indexes[$_board])) { + $replacement = '' . + '>>>/' . $_board . '/' . + ''; + $body = mb_substr_replace($body, $matches[1][0] . $replacement . $matches[4][0], $matches[0][1] + $skip_chars, mb_strlen($matches[0][0])); + $skip_chars += mb_strlen($matches[1][0] . $replacement . $matches[4][0]) - mb_strlen($matches[0][0]); } - - // Restore main board settings - openBoard($tmp_board); } } + + $tracked_cites = array_unique($tracked_cites, SORT_REGULAR); $body = preg_replace("/^\s*>.*$/m", '$0', $body); From 334e69b69f2c0191bfcfb9ff1ef3d7d8ffeb8a9e Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 13:32:55 +1000 Subject: [PATCH 05/15] Remove old code --- inc/functions.php | 3 --- 1 file changed, 3 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index f260a0ce..d4a5766a 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1693,9 +1693,6 @@ function markup(&$body, $track_cites = false) { $match[1] = mb_strlen(substr($body_tmp, 0, $match[1])); } - // Temporarily store board information because it will be overwritten - $tmp_board = $board['uri']; - if ($cite) { if (isset($cited_posts[$_board][$cite])) { $link = $cited_posts[$_board][$cite]; From 5f977ee5935f28090c73ee2afc9a277af3671140 Mon Sep 17 00:00:00 2001 From: ctrlcctrlv Date: Thu, 29 Aug 2013 00:38:39 +0000 Subject: [PATCH 06/15] Moving threads wasn't working with the catalog theme enabled Conflicts: inc/mod/pages.php --- inc/mod/pages.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/inc/mod/pages.php b/inc/mod/pages.php index bba16d6d..e782b09b 100644 --- a/inc/mod/pages.php +++ b/inc/mod/pages.php @@ -1137,7 +1137,7 @@ function mod_move($originBoard, $postID) { buildIndex(); // trigger themes - rebuildThemes('post'); + rebuildThemes('post', $targetBoard); // return to original board openBoard($originBoard); From abd013d6e423b316d52c1900711f2c8bd7dbe964 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 15:28:45 +1000 Subject: [PATCH 07/15] Add $config['always_regenerate_markup'] (99.9% of Tinyboard users should ignore this) --- inc/config.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/inc/config.php b/inc/config.php index 5d572010..9f68d04b 100644 --- a/inc/config.php +++ b/inc/config.php @@ -472,6 +472,11 @@ // } // ); + // Always regenerate markup. This isn't recommended and should only be used for debugging; by default, + // Tinyboard only parses post markup when it needs to, and keeps post-markup HTML in the database. This + // will significantly impact performance when enabled. + $config['always_regenerate_markup'] = false; + /* * ==================== * Image settings From 3b63cad71fe15f98dc84f3d6451817251e8f89f7 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 15:29:04 +1000 Subject: [PATCH 08/15] $config['always_regenerate_markup'] --- inc/display.php | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/inc/display.php b/inc/display.php index 81f8c53d..4639e86e 100644 --- a/inc/display.php +++ b/inc/display.php @@ -316,6 +316,11 @@ class Post { $this->modifiers = extract_modifiers($this->body_nomarkup); + if ($config['always_regenerate_markup']) { + $this->body = $this->body_nomarkup; + markup($this->body); + } + if ($this->mod) // Fix internal links // Very complicated regex @@ -411,6 +416,11 @@ class Thread { $this->modifiers = extract_modifiers($this->body_nomarkup); + if ($config['always_regenerate_markup']) { + $this->body = $this->body_nomarkup; + markup($this->body); + } + if ($this->mod) // Fix internal links // Very complicated regex From 7b817eea1119a9bd243497d68ba27059533ba5d8 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 15:56:36 +1000 Subject: [PATCH 09/15] Fix markup again. And add the option to repair fucked up nesting (and more) with HTML Tidy ($config['markup_repair_tidy']) --- inc/config.php | 12 ++++++++---- inc/functions.php | 11 ++++++++++- 2 files changed, 18 insertions(+), 5 deletions(-) diff --git a/inc/config.php b/inc/config.php index 9f68d04b..f9dc9912 100644 --- a/inc/config.php +++ b/inc/config.php @@ -459,10 +459,10 @@ */ // "Wiki" markup syntax ($config['wiki_markup'] in pervious versions): - $config['markup'][] = array("/'''([^<]+?)'''/", "\$1"); - $config['markup'][] = array("/''([^<]+?)''/", "\$1"); - $config['markup'][] = array("/\*\*([^<]+?)\*\*/", "\$1"); - // $config['markup'][] = array("/^[ |\t]*==([^<]+?)==[ |\t]*$/m", "\$1"); + $config['markup'][] = array("/'''(.+?)'''/", "\$1"); + $config['markup'][] = array("/''(.+?)''/", "\$1"); + $config['markup'][] = array("/\*\*(.+?)\*\*/", "\$1"); + // $config['markup'][] = array("/^[ |\t]*==(.+?)==[ |\t]*$/m", "\$1"); // Highlight PHP code wrapped in tags (PHP 5.3+) // $config['markup'][] = array( @@ -472,6 +472,10 @@ // } // ); + // Repair markup with HTML Tidy. This may be slower, but most if the time it solves nesting mistakes. + // Tinyboad, at the time of writing this, can not prevent out-of-order markup tags (eg. "**''test**''). + $config['markup_repair_tidy'] = false; + // Always regenerate markup. This isn't recommended and should only be used for debugging; by default, // Tinyboard only parses post markup when it needs to, and keeps post-markup HTML in the database. This // will significantly impact performance when enabled. diff --git a/inc/functions.php b/inc/functions.php index d4a5766a..d006d0ad 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1728,7 +1728,16 @@ function markup(&$body, $track_cites = false) { $body = preg_replace('/\s+$/', '', $body); $body = preg_replace("/\n/", '
', $body); - + + if ($config['markup_repair_tidy']) { + $tidy = new tidy(); + $body = $tidy->repairString($body, array( + 'doctype' => 'omit' + )); + $body = str_replace("\n", '', $body); + $body = preg_replace('@^.+|.+$@', '', $body); + } + return $tracked_cites; } From a8e37543757a6f5e6e841a3745cdb10b2247a269 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 18:55:25 +1000 Subject: [PATCH 10/15] Minor $config['try_smarter'] work --- inc/functions.php | 5 ++++- inc/template.php | 4 +++- post.php | 2 +- 3 files changed, 8 insertions(+), 3 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index d006d0ad..8b338063 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1306,7 +1306,7 @@ function buildIndex() { for ($page = 1; $page <= $config['max_pages']; $page++) { $filename = $board['dir'] . ($page == 1 ? $config['file_index'] : sprintf($config['file_page'], $page)); - if ($config['try_smarter'] && isset($build_pages) && count($build_pages) + if ($config['try_smarter'] && isset($build_pages) && !empty($build_pages) && !in_array($page, $build_pages) && is_file($filename)) continue; $content = index($page); @@ -1352,6 +1352,9 @@ function buildIndex() { $jsonFilename = $board['dir'] . 'catalog.json'; file_write($jsonFilename, $json); } + + if ($config['try_smarter']) + $build_pages = array(); } function buildJavascript() { diff --git a/inc/template.php b/inc/template.php index 6f85e78b..6d417631 100644 --- a/inc/template.php +++ b/inc/template.php @@ -35,7 +35,7 @@ function load_twig() { } function Element($templateFile, array $options) { - global $config, $debug, $twig; + global $config, $debug, $twig, $build_pages; if (!$twig) load_twig(); @@ -51,6 +51,8 @@ function Element($templateFile, array $options) { unset($debug['start']); unset($debug['start_debug']); } + if ($config['try_smarter'] && isset($build_pages) && !empty($build_pages)) + $debug['build_pages'] = $build_pages; $debug['included'] = get_included_files(); $debug['memory'] = round(memory_get_usage(true) / (1024 * 1024), 2) . ' MiB'; $options['body'] .= diff --git a/post.php b/post.php index 7ca9c1a9..de4bc07d 100644 --- a/post.php +++ b/post.php @@ -682,7 +682,7 @@ if (isset($_POST['delete'])) { incrementSpamHash($post['antispam_hash']); } - if (isset($post['tracked_cites']) && count($post['tracked_cites'])) { + if (isset($post['tracked_cites']) && !empty($post['tracked_cites'])) { $insert_rows = array(); foreach ($post['tracked_cites'] as $cite) { $insert_rows[] = '(' . From eb7cb42e938f4f21ac860e08724699424569a2c4 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 18:59:36 +1000 Subject: [PATCH 11/15] $config['markup_repair_tidy']: Better comment --- inc/config.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/inc/config.php b/inc/config.php index f9dc9912..86addf9b 100644 --- a/inc/config.php +++ b/inc/config.php @@ -472,8 +472,9 @@ // } // ); - // Repair markup with HTML Tidy. This may be slower, but most if the time it solves nesting mistakes. - // Tinyboad, at the time of writing this, can not prevent out-of-order markup tags (eg. "**''test**''). + // Repair markup with HTML Tidy. This may be slower, but it solves nesting mistakes. Tinyboad, at the + // time of writing this, can not prevent out-of-order markup tags (eg. "**''test**'') without help from + // HTML Tidy. $config['markup_repair_tidy'] = false; // Always regenerate markup. This isn't recommended and should only be used for debugging; by default, From 3f26aa5ac3dc3f438a9ce875d355d5040b684054 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 20:05:24 +1000 Subject: [PATCH 12/15] . --- inc/config.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/inc/config.php b/inc/config.php index 86addf9b..02cdbabf 100644 --- a/inc/config.php +++ b/inc/config.php @@ -611,7 +611,7 @@ // Display the aspect ratio of uploaded files. $config['show_ratio'] = false; // Display the file's original filename. - $config['show_filename']= true; + $config['show_filename'] = true; // Display image identification links using regex.info/exif, TinEye and Google Images. $config['image_identification'] = false; From 8144e517e9f3913ef024a45d702f0185a825df39 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 20:40:39 +1000 Subject: [PATCH 13/15] Don't set arbitrary cache timeouts for DNS stuff --- inc/functions.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index 8b338063..f99713f7 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -2003,7 +2003,7 @@ function rDNS($ip_addr) { } if ($config['cache']['enabled']) - cache::set('rdns_' . $ip_addr, $host, 3600); + cache::set('rdns_' . $ip_addr, $host); return $host; } @@ -2028,7 +2028,7 @@ function DNS($host) { } if ($config['cache']['enabled']) - cache::set('dns_' . $host, $ip_addr !== false ? $ip_addr : '?', 3600); + cache::set('dns_' . $host, $ip_addr !== false ? $ip_addr : '?'); return $ip_addr; } From 471525a58bc59399066eaffb6e593b432ec854a8 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 21:05:03 +1000 Subject: [PATCH 14/15] HTML Tidy fixes: UTF-8, preserving whitespace, keep attributes literal, don't wrap text --- inc/functions.php | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index f99713f7..22a6d9ff 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1731,16 +1731,22 @@ function markup(&$body, $track_cites = false) { $body = preg_replace('/\s+$/', '', $body); $body = preg_replace("/\n/", '
', $body); - + if ($config['markup_repair_tidy']) { $tidy = new tidy(); - $body = $tidy->repairString($body, array( - 'doctype' => 'omit' - )); + $body = $tidy->repairString(str_replace(' ', ' ', $body), array( + 'doctype' => 'omit', + 'bare' => true, + 'literal-attributes' => true, + 'quote-nbsp' => true, + 'indent' => false, + 'show-body-only' => true, + 'wrap' => 0, + 'output-bom' => false, + ), 'utf8'); $body = str_replace("\n", '', $body); - $body = preg_replace('@^.+|.+$@', '', $body); } - + return $tracked_cites; } From 738179c766953fa45670e451875210c6874a41f2 Mon Sep 17 00:00:00 2001 From: Michael Foster Date: Thu, 29 Aug 2013 21:24:38 +1000 Subject: [PATCH 15/15] HTML Tidy bug fix: broken attributes, and preserve whitespace properly --- inc/functions.php | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/inc/functions.php b/inc/functions.php index 22a6d9ff..e634b87e 100644 --- a/inc/functions.php +++ b/inc/functions.php @@ -1555,6 +1555,9 @@ function markup(&$body, $track_cites = false) { if ($num_links > $config['max_links']) error($config['error']['toomanylinks']); } + + if ($config['markup_repair_tidy']) + $body = str_replace(' ', '  ', $body); if ($config['auto_unicode']) { $body = unicodify($body); @@ -1731,18 +1734,21 @@ function markup(&$body, $track_cites = false) { $body = preg_replace('/\s+$/', '', $body); $body = preg_replace("/\n/", '
', $body); - + if ($config['markup_repair_tidy']) { $tidy = new tidy(); - $body = $tidy->repairString(str_replace(' ', ' ', $body), array( + $body = $tidy->repairString($body, array( 'doctype' => 'omit', 'bare' => true, 'literal-attributes' => true, - 'quote-nbsp' => true, 'indent' => false, 'show-body-only' => true, 'wrap' => 0, 'output-bom' => false, + 'output-html' => true, + 'newline' => 'LF', + 'quiet' => true, + ), 'utf8'); $body = str_replace("\n", '', $body); }