Fix for smilie display problem

As many of you know, the parsing of vB smilies is rather odd and can be a huge headache when it comes to multiple codes not parsing correctly

ie:

uke: and

as you can see, the smilies are parsed by display order so

:p

gets parsed before

:puke:

therefore the longer smilie code doesn't display correctly. An easy fix is to parse the smilies in order of their code length. The following two quick hacks will do just that!

BEFORE ATTEMPTING THIS - MAKE BACKUP COPIES OF:

/includes/adminfunctions.php
/includes/functions_bbcodeparse.php

Whenever you are changing files you should always have a proper backup!



open /includes/adminfunctions.php and goto line 2192

you will see this line

	$items = $DB_site->query("SELECT * FROM " . TABLE_PREFIX . "$table ORDER BY imagecategoryid, displayorder");

replace that line with this code

	if ($table == 'smilie') 
                {
                     $items = $DB_site->query("SELECT * FROM " . TABLE_PREFIX . "smilie ORDER BY LENGTH(smilietext) DESC");  
                }
                else
                {
                    $items = $DB_site->query("SELECT * FROM " . TABLE_PREFIX . "$table ORDER BY imagecategoryid, displayorder");  
                }

open includes/functions_bbcodeparse.php and goto line 385

you will see this line

SELECT smilietext, smiliepath, smilieid FROM " . TABLE_PREFIX . "smilie

replace it with this

SELECT smilietext, smiliepath, smilieid FROM " . TABLE_PREFIX . "smilie  ORDER BY LENGTH(smilietext) DESC

After this is complete, go into your Admin CP, view your smilies and just click "Save Display Order" This will update the cached smilie's datastore. This hack won't have any effect until this is done

Enjoy proper smilies ! Let me know how it works for ya!

[zahn]

nice, this one worked fine for me...

[Boofo]

Can you please post this in a text file for those who may have to reinstall this after an upgrade?

[blueshift]

Very good work. This has been an on-going problem throughout vBulletin history. Nicely done.

[Smash World]

works great on my vB 3.0.7 board - thanks!

[weaver]

I am getting

" class="inlineimg" />

after some of my smilies. Is that related to this hack or something else? I've only seen it start happening after I installed this hack.

ETA: Figured it out. It will give that error if you have duplicate smilie codes.

[Gutspiller]

Does this problem still exist in 3.0.7?

[hotwheels]

this is a great hack...............thanks, it is working perfectly