[beta 1.5.4] Smart URLs v2.1

[Anguz]

That's funny Cerberus.

I haven't tried this mod on either Enigma or Pfabb.. wonder if it will work?

try it... if it doesn't, I'll try a little change to one of the variables to see if it does the trick... I hope it works the first time though  

[Cerberus]

I would try, but I'm quite busy atm. And I don't see so much sense in this right now, because you're going to release a new version soon.

[Anguz]

I apologize for taking so long to release the new version... I was working on the code to not use the output buffer if it gave trouble, but decided to not make it an option

it'll not use output buffer anymore to change the urls... thus I'm rewritting half the mod, so it'll take me a little more... I'm sorry  

this new version will modify more files than just index.php and the admin ones, but not that many: boardindex.php, messageindex.php, subs.php display.php and maybe load.php

this is because instead of working on the output of those, it'll be working inside them to avoid the buffer part

oh well... I just wanted to explain why I was taking so long... hopefully it'll be released soon...

[pege]

I apologize for taking so long to release the new version... I was working on the code to not use the output buffer if it gave trouble, but decided to not make it an option

it'll not use output buffer anymore to change the urls... thus I'm rewritting half the mod, so it'll take me a little more... I'm sorry  

this new version will modify more files than just index.php and the admin ones, but not that many: boardindex.php, messageindex.php, subs.php display.php and maybe load.php

this is because instead of working on the output of those, it'll be working inside them to avoid the buffer part

oh well... I just wanted to explain why I was taking so long... hopefully it'll be released soon...

Thanks Anguz!  If this works, it'll be about the single most important mod to one of my boards.  I hope they incorporate this into SMF.  I know they already have spiderable threads, but this thing with keywords and all seems to make the optimal urls for search engine ranking.  Once it's stable in a variety of environments, I don't know why it wouldn't be at least a mod in SMF.

[Cerberus]

I just wanted to explain why I was taking so long... hopefully it'll be released soon...

No problem, we can wait !
I'm still wondering about portal support... is that difficult to add? I can try to do it by myself once I got some code as example.

[Anguz]

ok... here it is... version 2

these are the changes:

I removed the relative_url function cause it was too slow and also because after some testing, it wasn't really saving much bandwidth if gzip compression was on... in it's place, I made the urls in links to boards and threads, absolute instead of relative, which should have a similar effect on the spider

I removed the output buffer part of the mod and moved it to the actual scripts that generate the urls we want to change (boardindex, messageindex and display)... this also removed the preg_replace_callback function that some were having problems with

I added another two formats for the urls, so the three available now are: onevar, pathinfo and modrw

onevar uses a variable in the url, but only one, for links to boards or threads, which is good for those that can't use the other two options which seem to not be available to IIS... keywords can't be used in the url with this format
http://domain.tld/bbs/index.php?sUrl=t12345s45

pathinfo is the format used in version 1 of this mod... keywords can be used in the url and the board or thread look like a static file in the url (if you look away from the fact that the script appears earlier in the path)
http://domain.tld/bbs/index.php/keyws/t12345-45.htm

modrw, for those that can use Apache's mod_rewrite, it's like pathinfo, but it'll remove the forum's path up to the script, leaving only the keywords and board/thread (you'll have to write the mod_rewrite to change the url back to the one in the pathinfo format, cause I don't know how to... if someone could contribute that...)
http://domain.tld/keyws/t12345-45.htm

there's a var added to the url in boardindex where $boardseen is also used and in messageindex where $start is used when the value is "new", but they're special cases

I added a new keyword, for the whole forum, so now the url can have one for the whole community, another for the category and another for the board

I added the option to used the keywords in the title of threads, since I've found keywords in the title tag to weigh even more than the ones in the url... of course I may be wrong, cause I'm no search engines genius, but it doesn't hurt to have them
Community > Category > Board > Thread Title

what else... hmm...

the mod can understand any of the new formats and also the ones of previous mods like Spider URLs, so your links in Google will still work right

write the keywords using the proper case and instead of space, use a dash, like this: YaBB-SE... in the url, it'll be changed to this "yabb-se" and in the title will be changed to this "YaBB SE"

I can't remember what else, if there is...

I haven't been able to test this version yet, although it should work fine, cause I was in the middle of some changes in my yabbse install and can't tinker with it until it's finished.. but I thought you guys didn't want to wait until that happened

anyway, use it at your own risk, always backup... nothing bad other than having to go back to your backup files will happen, if it's needed at all... I'm really sorry I couldn't do the testing, so please let me know if there's something I overlooked and I'll fix it right away

the files modified by this mod (so you backup your working copy) are:

english.lng
index.php
template.php
sources/
  boardindex.php
  display.php
  manageboards.php
  managecats.php
  messageindex.php
  modsettings.php
  subs.php

of course, you need to remove the previous version before using this one and the database will already have some of the changes applied by the new version, but many are new... I'm thinking of writting an upgrade script for that, but from what I understand, it'll give an error where the change already existed, but nothing more

one more thing, I'll ask if you could contribute the translation to the .lng file texts, so it's available in more than english (I'll write the spanish one myself later, didn't even do that... I'm terrible lol)

here's the link to the zip file
Smart URLs v2.1

thank you for waiting, sorry for taking so long

edit: added a dbmod script to add the new fields to the database, without touching the existing ones from v1, so they're intact and you don't get errors installing... this is good in case you need to go back to that version while I fix a found bug in v2

[Shadow's Pawn]

I'll try this out on Pfabb and/or Enigma later on.. probably tomorrow, and let you know if it works.

Regardless, I'll definitely be installing this new version on the plain YabbSE sites I run.  Great job with this mod.  I mean holy jeez, this is awesome.

[pege]

Thanks Anguz!

Can't wait to try this later today!

[pege]

woohoo! works!! THANKS!!!!

for some reason, in Index.php I did have to add several " ' "'s to make it work:

had to change

switch($modSettings['sUrl_format']){
      case 'onevar':
         $bbscript = $scripturl;
         $sUrl_arr2 = array(
            '?sUrl=t\\1;start=new;boardseen=1',
            '?sUrl=t\\1;boardseen=1',
            '?sUrl=t\\1s\\2;boardseen=1',
            '?sUrl=t\\1;start=new',
            '?sUrl=t\\1,
            '?sUrl=t\\1s\\2,
            '?sUrl=t\\1;start=new',
            '?sUrl=t\\1,
            '?sUrl=t\\1s\\2,
            '?sUrl=t\\1;prev_next=\\2',
            '?sUrl=t\\1,
            '?sUrl=b\\1,
            '?sUrl=b\\1s\\2,
            '?sUrl=b\\1,
   

to this:

       switch($modSettings['sUrl_format']){
               case 'onevar':
                       $bbscript = $scripturl;
                       $sUrl_arr2 = array(
                               '?sUrl=t\\1;start=new;boardseen=1',
                               '?sUrl=t\\1;boardseen=1',
                               '?sUrl=t\\1s\\2;boardseen=1',
                               '?sUrl=t\\1;start=new',
                               '?sUrl=t\\1',
                               '?sUrl=t\\1s\\2',
                               '?sUrl=t\\1;start=new',
                               '?sUrl=t\\1',
                               '?sUrl=t\\1s\\2',
                               '?sUrl=t\\1;prev_next=\\2',
                               '?sUrl=t\\1',
                               '?sUrl=b\\1',
                               '?sUrl=b\\1s\\2',
                               '?sUrl=b\\1',
                               );

and, in ModSettings.php,

// now let's do all the textfield/select values
       // set default values
       if(!isset($HTTP_POST_VARS['sUrl_format']))
               $HTTP_POST_VARS['sUrl_format']) = 'onevar';

should be

// now let's do all the textfield/select values
       // set default values
       if(!isset($HTTP_POST_VARS['sUrl_format']))
               $HTTP_POST_VARS['sUrl_format'] = 'onevar';

other than that, GREAT!!
Yayy!!!! Thanks Anguz! wooo!

[Anguz]

oops... sorry about that one  

I fixed the code in index.php but I don't understand what's wrong with the modsettings.php one

updated the mod in the zip file, thx for letting me know

can I have your url to see it working?

[pege]

yep, in the modsettings, the only thing was

'sUrl_format'])

should be

'sUrl_format']"

Also, line 191 of Display.php,

             $keywords = ''

should be

             $keywords = '';

here's the *board* (experimental).


yeah!  THANKS again!!

[Anguz]

fixed... zip updated

thank you for your help pege  

[pege]

sorry, I made an incorrect quote in the correction for Index.php above.  It should be

       switch($modSettings['sUrl_format']){
               case 'onevar':
                       $bbscript = $scripturl;
                       $sUrl_arr2 = array(
                               '?sUrl=t\\1;start=new;boardseen=1',
                               '?sUrl=t\\1;boardseen=1',
                               '?sUrl=t\\1s\\2;boardseen=1',
                               '?sUrl=t\\1;start=new',
                               '?sUrl=t\\1',
                               '?sUrl=t\\1s\\2',
                               '?sUrl=t\\1;start=new',
                               '?sUrl=t\\1',
                               '?sUrl=t\\1s\\2',
                               '?sUrl=t\\1;prev_next=\\2',
                               '?sUrl=t\\1',
                               '?sUrl=b\\1',
                               '?sUrl=b\\1s\\2',
                               '?sUrl=b\\1',
                               );

[Anguz]

I compared the two codes and they are the same... anyway, it looks okay to me...

I'm really sorry I couldnt' test it myself, I'd've found these typos before releasing the mod

[Anguz]

here's the *board* (experimental).

cool! it looks nice and it's as fast as I was expecting it

Smart URLs: +0.005s smart_url(29)
Smart URLs: +0.001s smart_url(5)

btw, your yabbse counter is shown twice, is it something in your template?

also, did you test the keywords?