Content Plus Shortlinks

Posted: Sun Sep 27, 2009 12:04 pm

When browsing tags in Content Plus with shortlinks enabled, any tags with more than one word end up like this:
[ Only registered users can see links on this board! Get registered or login! ]

Posted: Sun Sep 27, 2009 2:19 pm

You need to enter a comma between the tags but I see also a wrong url where I have a space included in the tag:
Produced a url for the tag "Web Hosting":
/content-browse-tag-web.html+hosting

Will add this to our bug tracker.

registered · Posted: Sun Sep 27, 2009 8:38 pm

I'm digging into this....

Posted: Mon Sep 28, 2009 9:12 am

Thanks !

Posted: Mon Sep 28, 2009 2:10 pm

cool

I had trouble rewriting an url with + in it awhile back so curious to see the solution Smile

on a related note i found a similar issue with a forum tap.
forum search - sort by post and/or topic - search term of more than one word = [ Only registered users can see links on this board! Get registered or login! ] [ Only registered users can see links on this board! Get registered or login! ]
in the results....

Posted: Mon Sep 28, 2009 5:42 pm

... oh I hate tapping URLs!!! Laughing

Posted: Tue Sep 29, 2009 10:07 pm

took a little reading on perl regular expressions but i think this does the trick (for the most part)

htaccess

Code:

RewriteRule ^content-browsetag-([a-zA-Z0-9_\-\+]*)-order-([0-9]*).html modules.php?name=Content&pa=BrowseTag&tag=$1&order=$2 [L]


RewriteRule ^content-browse-tag-([[a-zA-Z0-9_\-\+]*).html modules.php?name=Content&pa=browse_tag&tag=$1 [L]

and the urlin of GT-Content

Code:

'"(?<!/)modules.php\?name=Content&amp;pa=BrowseTag&amp;tag=([a-zA-Z0-9_\-\+]*)&amp;order=([0-9]*)"',


'"(?<!/)modules.php\?name=Content&amp;pa=browse_tag&amp;tag=([a-zA-Z0-9_\-\+]*)"',

should handle the content plus browse tags shortlinks, except for the "order by" function, which seems to strip the + for a space, which happens in the untapped module [ Only registered users can see links on this board! Get registered or login! ]

and forums......

htaccess

Code:

RewriteRule ^ftopic(t|p)-([0-9]*)-([A-Za-z0-9_\-\+]*).html forums.html?file=viewtopic&$1=$2&highlight=$3 [L]

gt-forum urlin

Code:

'"(?<!/)modules.php\?name=Forums&amp;file=viewtopic&amp;(t|p)=([0-9]*)&amp;highlight=([A-Za-z0-9_\-\+]*)"',

Posted: Wed Sep 30, 2009 8:39 am

Thanks for pointing that out M, I have now fixed it and commited it...

Posted: Wed Sep 30, 2009 11:21 am

cool, i just noticed i used an extra
[
in that first htaccess edit though... Sad

seemed to work anyway but shouldn't be there.. ie
content-browse-tag-([[a-zA-Z0-9_\-\+]*).html
should be
content-browse-tag-([a-zA-Z0-9_\-\+]*).html

maybe you caught it?

Posted: Wed Sep 30, 2009 8:10 pm

jestrella wrote:

Thanks for pointing that out M, I have now fixed it and commited it...

Good job sir!

Posted: Fri Nov 27, 2009 2:27 pm

For the Forums, the rewrites that I have implemented in our repository are:

GT-Forums:

Code:




'"(?<!/)modules.php\?name=Forums&amp;file=viewtopic&amp;t=([0-9]*)&amp;start=([0-9]*)&amp;postdays=([0-9]*)&amp;postorder=(desc|asc)&amp;highlight=([A-Za-z0-9+_-]*)"',


'"(?<!/)modules.php\?name=Forums&amp;file=viewtopic&amp;t=([0-9]*)&amp;postdays=([0-9]*)&amp;postorder=(desc|asc)&amp;highlight=([A-Za-z0-9+_-]*)&amp;start=([0-9]*)"',


'"(?<!/)modules.php\?name=Forums&amp;file=viewtopic&amp;(t|p)=([0-9]*)&amp;highlight=([A-Za-z0-9+_-]*)"',

.htaccess:

Code:




RewriteRule ^ftopic-([0-9]*)-([0-9]*)-days([0-9]*)-order(desc|asc)-([A-Za-z0-9+_-]*).html modules.php?name=Forums&file=viewtopic&t=$1&start=$2&postdays=$3&postorder=$4&highlight=$5 [L]


RewriteRule ^ftopic-([0-9]*)-days([0-9]*)-order(desc|asc)-([A-Za-z0-9+_-]*)-([0-9]*).html modules.php?name=Forums&file=viewtopic&t=$1&postdays=$2&postorder=$3&highlight=$4&start=$5 [L]


RewriteRule ^ftopic(t|p)-([0-9]*)-([A-Za-z0-9+_-]*).html modules.php?name=Forums&file=viewtopic&$1=$2&highlight=$3 [L]

Enjoy!