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!

[red_baron2000]

man oh man!! you've heard me!!!

[Oblivion Knight]

Nicely done, I know that many will love you for this..

[sabret00the]

another fix may be to try and maintain display order

PHP Code:

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


but i aint tested it so who knows :happysad:

[CtrlAltDel]

good idea and great work putting it together

[Cap'n Steve]

THANK YOU! I've been putting off trying to reorganize my smilies so they all worked ever since I switched to vbulletin.

[Boots]

Originally Posted by sabret00the

another fix may be to try and maintain display order

PHP Code:

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


but i aint tested it so who knows :happysad:

Not a bad idea. I may implement this and test it later. Sorry for the delay on the text file. you know how the holidays are

[frage]

Boots

you made my day!

:up:
frage

[mtha]

I did exactly the same thing, and suggested vb.com, but seems they didnt want to sort out the problem

[high]* mtha clicks install[/high]

[JohnBee]

Doing this modification now produces this when updating cache counters.

Database error in vBulletin 3.0.3:

Invalid SQL: 
    		    	SELECT smilietext, smiliepath, smilieid FROM smilie  ORDER BY LENGTH(smilietext) DESC  ORDER BY LENGTH(smilietext) DESC
				
mysql error: You have an error in your SQL syntax.  Check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORDER BY LENGTH(smilietext) DESC' at line 1

mysql error number: 1064

Date: Monday 09th of May 2005 09:39:44 AM
Script: http://www.mysite.net/web/forum/admincp/misc.php
Referer: http://www.mysite.net/web/forum/admincp/misc.php
Username: JohnBee
IP Address: *.*.*.*

Any ideas?

[JohnBee]

I found it!
It was my own typo, carry on ... :classic:

Originally Posted by JohnBee

Doing this modification now produces this when updating cache counters.

Database error in vBulletin 3.0.3:

Invalid SQL: 
    		    	SELECT smilietext, smiliepath, smilieid FROM smilie  ORDER BY LENGTH(smilietext) DESC  ORDER BY LENGTH(smilietext) DESC
				
mysql error: You have an error in your SQL syntax.  Check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORDER BY LENGTH(smilietext) DESC' at line 1

mysql error number: 1064

Date: Monday 09th of May 2005 09:39:44 AM
Script: http://www.mysite.net/web/forum/admincp/misc.php
Referer: http://www.mysite.net/web/forum/admincp/misc.php
Username: JohnBee
IP Address: *.*.*.*

Any ideas?