The way bbcode handles new line breaks

I don't know if this is really a bbcode issue, but I think the way bbcode works, it is in bbcode where it is the easier to change. Because usually with filtered html people do not use lists, except with bbcode, and this problems happens mostly with lists...

This is a bbcode issue, I tested without bbcode and I have not this issue.
http://drupal.org/node/51126

I need to do some testing, but until now (I haven't tested last cvs yet):

imagine we have two paragraphs, with the text : paragraph1 for the first, and the text paragraph2 for the second.

bbcode created a code like this:

<p><p>paragraph1</p><p>paragraph2</p></p>

I think it is nonsense to enclose the whole article into a paragraph by itself, because a text is not always a paragraph.

Still does not work.
Look, I'll give you two examples (with the 1.21 version of bbcode).

I'm doing something like:

• ligne 1
• ligne 2

BBcode gives me a code like this:

<p><p><ul class="bb-list"><li>ligne 1<br />
</li><li>ligne 2<br />

</li></ul></p>
</p>

Which is not xhtml compliant because the (ul) tag is enclosed into a (p) tag.

Without bbcode, drupal gives me a code like this:

<ul>
<li>ligne 1</li>
<li>ligne 2

</ul>

Which is easier, and which is xhtml compliant.

It is really nonsense to enclose the whole article into a (p) tag, and really, there shouldn't be (p) tags surrounding the (ul) one.

mmmm... the only other filter I have is the integrated drupal filter which will goes to the line automatically. Should I have this one enabled or disabled?

I have activated this one and the bbcode intelligent paragraph breaking.

And I have setup html filter to escape all tags.

What should I put on and off to make it work?

Until then, I have still modified the .inc file this way, by adding some lines at the end :

    '#<p><p>#si'                            => '<p>',
    '#<\/p>\n<\/p>#si'                      => '</p>',

    '#<p>(?:.*?)<ol(.*?)>#si'               => '<ol\\1>',
    '#<\/ol(.*?)>(?:.*?)<\/p>#si'           => '</ol\\1>',

    '#<p>(?:.*?)<ul(.*?)>#si'               => '<ul\\1>',
    '#<\/ul(.*?)>(?:.*?)<\/p>#si'           => '</ul\\1>',

This way I remove double (p) and (/p) tags, and I remove the (p) before the (ul) and (ol), and the (/p) after...

I did... it works, thanks ;)

Now there's another bug which goes with this issue.
So I disabled the drupal new line break.

But now, except for paragraphs, there is no line break, you see what I mean?

If, in my article, I skip two lines, then bbcode puts me (p) code, that's fine.

But if I just go to the line, bbcode should put the (br /) tag, but it does not.

That is why I had activated the drupal line break, but it is not compatible with bbcode it seems. What is exactly the issue? You succeeded in replacing two line breaks by a (p) tag, but not just one line break?

Perhaps you should look at the way drupal handles this. For now I have no idea on how to do this either...

The problem with the [br] tag is that when you'll find the issue to change new lines to (br /), then the br tag added with your issue will create too many line breaks :) So I hope we'll find a way to deal with it before my site goes into production ;)