Page 1 of 1

Disabling nl2br?

Posted: Tue May 04, 2010 4:49 pm
by balu
Hi,

up to now I've used the nl2br plugin for my entries, but I'd like to switch that, because I want use a WYSIWYG editor.

If I just disable the plugin for the entries all older posts are formatted without any paragraphs, which looks rather bad ;).

I guess I need to update the entries in the database to use <p> instead of newlines.

Can anyone give me a hint on how to do so?

Regards
Balu

Re: Disabling nl2br?

Posted: Wed May 05, 2010 5:39 am
by Don Chambers
I cannot think of any easy/automatic way to do this. I would personally edit the entries using a database utility such as phpmyadmin, and not edit the entries via the serendipity editor.

Hopefully Garvin or others have a better idea.

Re: Disabling nl2br?

Posted: Wed May 05, 2010 12:02 pm
by garvinhicking
Hi!

Hm, there's no automatic logic for this.

Something that could work is a script like this:

Code: Select all

<?php

include 'serendipity_config.inc.php';

function autop($pee, $br = 1) {

	if ( trim($pee) === '' )
		return '';
	$pee = $pee . "\n"; // just to make things a little easier, pad the end
	$pee = preg_replace('|<br />\s*<br />|', "\n\n", $pee);
	// Space things out a little
	$allblocks = '(?:table|thead|tfoot|caption|col|colgroup|tbody|tr|td|th|div|dl|dd|dt|ul|ol|li|pre|select|form|map|area|blockquote|address|math|style|input|p|h[1-6]|hr|fieldset|legend)';
	$pee = preg_replace('!(<' . $allblocks . '[^>]*>)!', "\n$1", $pee);
	$pee = preg_replace('!(</' . $allblocks . '>)!', "$1\n\n", $pee);
	$pee = str_replace(array("\r\n", "\r"), "\n", $pee); // cross-platform newlines
	if ( strpos($pee, '<object') !== false ) {
		$pee = preg_replace('|\s*<param([^>]*)>\s*|', "<param$1>", $pee); // no pee inside object/embed
		$pee = preg_replace('|\s*</embed>\s*|', '</embed>', $pee);
	}
	$pee = preg_replace("/\n\n+/", "\n\n", $pee); // take care of duplicates
	// make paragraphs, including one at the end
	$pees = preg_split('/\n\s*\n/', $pee, -1, PREG_SPLIT_NO_EMPTY);
	$pee = '';
	foreach ( $pees as $tinkle )
		$pee .= '<p>' . trim($tinkle, "\n") . "</p>\n";
	$pee = preg_replace('|<p>\s*</p>|', '', $pee); // under certain strange conditions it could create a P of entirely whitespace
	$pee = preg_replace('!<p>([^<]+)</(div|address|form)>!', "<p>$1</p></$2>", $pee);
	$pee = preg_replace('!<p>\s*(</?' . $allblocks . '[^>]*>)\s*</p>!', "$1", $pee); // don't pee all over a tag
	$pee = preg_replace("|<p>(<li.+?)</p>|", "$1", $pee); // problem with nested lists
	$pee = preg_replace('|<p><blockquote([^>]*)>|i', "<blockquote$1><p>", $pee);
	$pee = str_replace('</blockquote></p>', '</p></blockquote>', $pee);
	$pee = preg_replace('!<p>\s*(</?' . $allblocks . '[^>]*>)!', "$1", $pee);
	$pee = preg_replace('!(</?' . $allblocks . '[^>]*>)\s*</p>!', "$1", $pee);
	if ($br) {
		$pee = preg_replace_callback('/<(script|style).*?<\/\\1>/s', create_function('$matches', 'return str_replace("\n", "<WPPreserveNewline />", $matches[0]);'), $pee);
		$pee = preg_replace('|(?<!<br />)\s*\n|', "<br />\n", $pee); // optionally make line breaks
		$pee = str_replace('<WPPreserveNewline />', "\n", $pee);
	}
	$pee = preg_replace('!(</?' . $allblocks . '[^>]*>)\s*<br />!', "$1", $pee);
	$pee = preg_replace('!<br />(\s*</?(?:p|li|div|dl|dd|dt|th|pre|td|ul|ol)[^>]*>)!', '$1', $pee);
	if (strpos($pee, '<pre') !== false)
		$pee = preg_replace_callback('!(<pre[^>]*>)(.*?)</pre>!is', 'clean_pre', $pee );
	$pee = preg_replace( "|\n</p>$|", '</p>', $pee );

	return $pee;
}

$r = serendipity_db_query("SELECT id, body, extended FROM {$serendipity['dbPrefix']}entries");
foreach($r as $entry) {
    $entry['body'] = autop($entry['body']);
    $entry['extended'] = autop($entry['extended']);
    serendipity_db_query("UPDATE {$serendipity['dbPrefix']}entries 
                             SET body = '" . serendipity_db_escape_string($entry['body']) . "',
                                 extended = '" . serendipity_db_escape_string($entry['extended']) . "'
                           WHERE id = " . $entry['id']);
}
OF COURSE MAKE A BACKUP BEFORE YOU EVEN CONSIDER USING THIS! :-)

(And don't ask me from where I stole the formatting function ;) )

Regards,
Garvi

Re: Disabling nl2br?

Posted: Thu May 06, 2010 4:03 pm
by balu
Hey Garvin,

at first I need to say the following: Backups are for cowards. ;-)

At first glance it looks like everything went alright. It's not the same formatting as the module though. The original one had some classes and stuff inside those <p>aragraphs.

Thanks for your help,

Balu