Charset problems for links, 1.6

For questions and problems with the CMS core. This board is NOT for any 3rd party modules, addons, PHP scripts or anything NOT distributed with the CMS made simple package itself.
Post Reply
Terabyte
Forum Members
Forum Members
Posts: 79
Joined: Fri Aug 18, 2006 8:57 am

Charset problems for links, 1.6

Post by Terabyte »

Well, upgrading went very smooth exept for this problem.

I upgraded several sites, some from 1.1, some from 1.41, some from 1.54 ... all to 1.6

In all sites I get this problem where having the chars "å,ä,ö" (Swedish) in the title of the links, gives a lot of HTML characters like "ä" "ö" ...

In regular text there are no issues, only in links.

What can have gone wrong or is it a bug?
Last edited by Terabyte on Tue Jul 07, 2009 12:34 am, edited 1 time in total.
Terabyte
Forum Members
Forum Members
Posts: 79
Joined: Fri Aug 18, 2006 8:57 am

Re: Charset problems for links, 1.6

Post by Terabyte »

Anyone maybe could give a hint where to start looking for this error?
stevew
Forum Members
Forum Members
Posts: 74
Joined: Wed May 21, 2008 7:02 am

Re: Charset problems for links, 1.6

Post by stevew »

Not sure this is the same problem, but if you are using {cms_selflink}-type links, try this thread: http://forum.cmsmadesimple.org/index.php/topic,34935.0.html.
Terabyte
Forum Members
Forum Members
Posts: 79
Joined: Fri Aug 18, 2006 8:57 am

Re: Charset problems for links, 1.6

Post by Terabyte »

Thanks. I didn't find it there, but the thread led me to another:
http://forum.cmsmadesimple.org/index.ph ... #msg165925
blast2007 wrote: I think the main issue of  function.cms_selflink.php between 1.5.4 and 1.6 iis that in 1.6 release there's these lines
345

Code: Select all

$label = cms_htmlentities($label);
361

Code: Select all

$title = cms_htmlentities($title);
439

Code: Select all

$linktext = cms_htmlentities($linktext);
not present in 1.5.4 version.
All text in selflink is html escaped.
So commenting out, as suggested by blast2007, that lines 345,361,439 gives me a good result again.

So it is a bug ... Where should it be reported?
Post Reply

Return to “CMSMS Core”