Bug of MenuManager output in the frontend when using ®

A place to discuss the testing process in beta cycles or against SVN for the CMS Made Simple CORE package.
Locked
User avatar
Augustas
Forum Members
Forum Members
Posts: 241
Joined: Wed Oct 17, 2007 6:09 pm
Location: the world

Bug of MenuManager output in the frontend when using ®

Post by Augustas »

In my opinion MenuManager should not convert text into HTML entities.
line: 219, file: /modules/menumanager/MenuManager.module.php

I think this decision should be left for the content editor, or there should be a possibility to switch of this option.

Here is the trouble I am having with it:

I need to add a "registered trademark" ® sign in the "Menu Text" / $node->menutext .
Moreover, I want this sign to be displayed as html entity: ®

If in ADMIN I enter:
Menu Text: Text ®
The frontend output is "Text ®"
View html source - "Text ®" / it converts "&"

If I enter:
Menu Text: Text ®
The frontend output is "Text ®"
View html source - "Text ®"

Currently I solved this problem by using {$node->menutext|regex_replace:"/®/":"®"}
in the menu template.

If MenuManager would use proper php "htmlentities" function, maybe it would be a better solution than to use "my_htmlentities" from CMS.

I am using the newest CMSMS v1.2, MenuManager v1.5
http://FollowTheRoad.com/ - living on the road...
http://www.kligys.com/ - asmeninis blog'as...
User avatar
alinome.net
Forum Members
Forum Members
Posts: 124
Joined: Thu Jan 25, 2007 2:54 pm
Location: España / Hispanujo / Spain

Re: Bug of MenuManager output in the frontend when using ®

Post by alinome.net »

Augustas wrote: In my opinion MenuManager should not convert text into HTML entities.
line: 219, file: /modules/menumanager/MenuManager.module.php
I think this decision should be left for the content editor, or there should be a possibility to switch of this option.
I agree. I have a similar problem with MenuManager and some CMSMS tags:

http://forum.cmsmadesimple.org/index.php/topic,17264.0.html
Marcos Cruz
Locked

Return to “[locked] Quality Assurance”