BBcode class

[Gpower2]

A class that will contain the functionality needed for BBcodes:

• parsing
• writing
• fixing???

It should make heavy use of regular expressions in order to be as fast as possible.

[tolias63]

I have done the parsing code. What do you mean when you say “writing” and “fixing”?

[LelouchVII]

If I want to use the spoiler tag. it need to input at the correct place [spoiler]...[\spoiler]

[Gpower2]

Push the BBcode class that you've written so far and we'll see about the rest ;)

[tolias63]

I have 3 php files: text.php (textarea box), class.php (bbcode) and index.php (show the result).

Do you want me to push all of them?

[Gpower2]

Yes, just make a separate folder in order to have a clean file structure.

[tolias63]

Ok, I pushed them.

P.S.: Sorry for the delay, but I found some errors in the script and I fixed them.

[Gpower2]

I made some changes to the directory structure and the names of the classes. You should pull the changes locally and see them carefully.

Also, the bbcode transformations that you used don't match the ones we currently use. I guess you found them and used them as they were. I uploaded a test file in order to use it as a reference for the correct output.

[tolias63]

I didn't use them as they were, I selected the names of the tags. I will change them to match with the ones we use now.;)

Also, with the “bbcodeTest01.txt” that you uploaded, I found error in the quote code. When it is quote within quote it doesn’t work (I didn’t think to testing that). I will try to fix it and I will edit the code.

[Gpower2]

Made a few more changes, so please pull once again to have the latest version, thanks.

[tolias63]

I finished with the changes. I'll check the script again tomorrow and I will push it. ;)

[tolias63]

I have pushed the files. I uploaded and a new bbcodeTest02.txt. I think now we are ok, but test it yourself.;)

[Gpower2]

Better, but it still has a lot of issues:

• Inside [code][/code] there should be no bbcode parsing.
• we don't want inline css styles. we should either use style classes, or html markup like tags
• youtube shouldn't be in iframes. Check our embedded html (

[Gpower2]

And a simple use case to help you:

[quote] This is a quote: [quote=Gpower2] This is a named quote: [quote=tolias63] This is a simple list: [list] [] This is a malformed list: [list=] [] This is an ordered list: [list=1] [*] This is a code statement: [code] Testing simple code. [/code] This is a nested code statement: [code] First level: [code] Second level It contains a quote: [quote] That it contains a code: [code] Code inside a quote [/code] [/quote] [/code] [/code] [/list] [/list] [/list] [/quote] [/quote] [/quote]

[tolias63]

When you say "style classes" you mean something like this, right?

php file: (div class="centerbox")(/div) (div class="centerbox2")(p class="announcetitle" )Hallo(/p)(/div)

css file: div.centerbox {background-color: white; margin-left: 450px;} div.centerbox2 {position: relative; margin-left: 300px; margin-top: 100px;} p.announcetitle {color: black; font-weight: bold; font-size:36px;}

[Gpower2]

Yes, exactly like this. ;)

[tolias63]

Ok, i will do the changes. :D

[tolias63]

I have fixed the style classes, the youtube, and the javascript for spoiler.

I have also done the “code”, but when there is [code][code]...text...[/code][/code] doing parsing the first [code] with the first [/code], not the first [code] with the latest [/code].

I will try to fix it and I will push it.

[tolias63]

I pushed the files. I hope they are ok now.;)

[tolias63]

Btw, I have started to making and the buttons with javascript.;)

[Gpower2]

There are still some issues, I will make some changes and push them, in order to understand what I mean. ;)

[tolias63]

Do I have a lot of wrongs?

[Gpower2]

No, you will see and you will understand. ;)

[tolias63]

OK. Can I continue with the buttons?

[Gpower2]

I don't quite understand what you mean by "buttons". This issue is only about back end core code, not front end design. So you shouldn't concern yourself with the CSS and javascript for the time being. These are issues for later. ;)

[tolias63]

I mean, for example, when I press the "B" to add [b][/b] in the text.

[Gpower2]

No, that's entirely another issue and it is definitely for later. You can work on it, but don't push it yet. ;)

[tolias63]

Ah, ok! I will wait to see the changes.;)

[tolias63]

Ok, I saw the changes and I have some questions.

• When you say “We don't want specific sizes or colors or fonts” you mean I must make, for example, five basically colors, five fonts and five font sizes?
• When you say “We only use code for (pre)(/pre) tags” you mean it must be something like this (pre)(code)...(/code)(/pre)?
• In youtube, what others url form must I include? Work and with this “www.youtube.com/watch?v=KrVC5dm5fFc” and with this “KrVC5dm5fFc”.
• And final, in the spoiler, I must remove complete the onclick from the line, or I can have onclick=function() ? The function will be in the js file.

[Gpower2]

• We will have specific style classes for size, color and fontname, so that we can have the ability to change themes.
• The < pre > tag breaks the formatting and will destroy our look and feel if we use it. So just forget about it.
• Leave the youtube links for me, we already have the code from our old site
• Read about unobtrusive javascript here: http://en.wikipedia.org/wiki/Unobtrusive_JavaScript and specifically check the paragraph "Separation of behavior from markup" and see the example.

[tolias63]

Ok. I read about unobtrusive javascript and I will fix the spoiler. I will delete the lines for < pre >. That I don't quite understand is what I must do for size, color and font. For example, I must make something like this:

php file '/(?<!\)[color(?::\w+)?=red[\/color(?::\w+)?]/si' => "(span class="red">\1(/span)", '/(?<!\)[color(?::\w+)?=green[\/color(?::\w+)?]/si' => "(span class="green">\1(/span), . '/(?<!\)[font(?::\w+)?=Arial[\/font(?::\w+)?]/si' => "(span class="arial">\1(/span)", '/(?<!\)[font(?::\w+)?=Georgia[\/font(?::\w+)?]/si' => "(span class="georgia">\1(/span)", . '/(?<!\)[size(?::\w+)?=15[\/size(?::\w+)?]/si' => "(span class="size_15">\1(/span)", '/(?<!\)[size(?::\w+)?=20[\/size(?::\w+)?]/si' => "(span class="size_20">\1(/span)",

css file span.red {color:red;} span.green {color:green;} . span.arial {font-family:Arial;} span.georgia {font-family:Georgia;} . span.size_15 {font-size:15px;} span.size_22 {font-size:20px;}

[Gpower2]

You don't need to delete any lines, better leave them there for future reference. The color and font names are ok, about the size, you should use tiny, small, normal, medium, large, huge. And then we can decide with css which size to use. ;)

I also made a few more cosmetic changes to the code, so please first take the latest version.

[tolias63]

Ok, man! I will make the changes. ;)

[tolias63]

Are they ok now?

[Gpower2]

issues to check: -replace funny utf8 characters -replace <> characters with its htmml entities

i write them here in order not to forget them, please ignore them for the time being...

[Gpower2]

Made a few changes and created the prototype for a checking function, in order for you to write it. ;) I accidentally linked the commit with the database issue...

[tolias63]

Ok, I saw the example and I understood what I must do. I will write it.;)

[tolias63]

I'm not sure if they are correct...

[tolias63]

I was thinking, what would you say if we include and spell checking?

[Gpower2]

It is far too soon to think about such features, and apart from that, spell checking is a feature that modern browsers support by default. So no, you shouldn't waste your time with spell checking.

I am still waiting on that regex bug though...

[tolias63]

Oh, I forgot about that bug... :oops:

I will fix it...;)