Reven
Reven

Reputation: 173

A regex that converts text lists to html in PHP

I'm trying to code a regexp to convert a block of text:

* List item
* Another list item

to html:

<ul>
    <li>List item</li>
    <li>Another list item</li>
</ul>

I know there are snippets or classes to do this (Markdown, Textile, etc) but I think it's overkill: I really just want some basic functionality. So far I'm trying with:

$text = preg_replace("/\*+(.*)?/i","<li>$1</li>",$text);

But I don't know how to wrap everything in <ul> tags without using a separate replace, like so:

$text = preg_replace("/(\<li\>(.*)\<\/li\>\n*)+/is","<ul>\n$1\n</ul>\n",$text);

This interferes with other code, for example ordered lists. There must be a better way.

Thanks.

Upvotes: 7

Views: 3880

Answers (4)

BrandonS
BrandonS

Reputation: 932

On this question, if you where talking about the fact that the code you used would wrap multiple sets of li tags in one ul tag even if there was suppose to be a break in there like so:

* line 1
* line 1
* line 1
this is not part of a list
* line 1
* line 1
* line 1

Would become:

<ul>
<li>line 1</li>
<li>line 1</li>
<li>line 1</li>
this is not part a the list
<li>line 1</li>
<li>line 1</li>
</ul>

Then I have a solution for you. You had 90% of it there, here is a solution I came up with (but I am sure you already solved it anyway):

$text = preg_replace("/\*+(.*)?/i","<ul><li>$1</li></ul>",$text);
$text = preg_replace("/(\<\/ul\>\n(.*)\<ul\>*)+/","",$text);

The solution does not mess with lists of any kind already on the page in the text or whatever and makes sure to separate multiple lists. Reason is that every match it finds where an asterisk was used to create a text list item it surrounds that with a ul and li then the 2nd line finds all of the back to back closing and opening ul tags and removes them.

Upvotes: 14

Ignacio Vazquez-Abrams
Ignacio Vazquez-Abrams

Reputation: 798746

Perhaps you may find PHP Markdown useful.

Upvotes: 0

DCC
DCC

Reputation: 249

Why don't you store the first regex in an array with preg_match_all, and glue it like this:

$list='<ul><li>';
$list .= implode('</li><li>',$arr_regex);
$list .= '</li></ul>';

Upvotes: 1

Paulo Santos
Paulo Santos

Reputation: 11567

Well, you could simply do

$text = "<ul>" . preg_replace("/\*+(.*)?/i","<li>$1</li>",$text) . "</ul>";

or, if you really want to use preg_replace

$text = preg_replace("/(\<li\>(.*?)\<\/li\>\n*)+/is","<ul>\n$1\n</ul>\n",$text);

Upvotes: 0

Related Questions