I created a new "Content types" and called it "Member's Websites" (with the apostrophe).

It displays fine in the "Content Types" list, but under the "Create Content" area (side menu and main menu list) it is displaying as "Member's Websites"

Same thing happens with a quote (") which adds: "

Interestingly the back tick (`) is fine...

Please advise on a patch.

CommentFileSizeAuthor
#4 menu_check_plain.patch1.06 KBtheborg
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neofactor’s picture

neofactor CreditAttribution: neofactor commented

Looks like my post got displays wrong... I will re-dsiplay the output by breaking it up...

An apostrophe ' displays this: & #039; (without the space)
A quote " displays this: & quot;

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Version: 6.0-rc2 » 6.x-dev

I can confirm this in the latest CVS version. I'm not sure if it qualifies as a bug, though, since when adding a new content type the page clearly states:
This name must begin with a capital letter and contain only letters, numbers, and spaces.

On the other hand, it's not so obvious to me why this restriction should exist on the "human-readable" version of the content type name... perhaps someone else knows a good reason for it, though?

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented

Also, I did a quick check and it looks like this problem does not exist in Drupal 5, for what it's worth...

theborg’s picture

theborg CreditAttribution: theborg commented
Status: Active » Needs review
FileSize
menu_check_plain.patch1.06 KB

The content type title has already passed 'check_plain' security function thought the menu system and is called again before building the link.

This patch checks if check_plain has been called for an item.

theborg’s picture

theborg CreditAttribution: theborg commented
Title: Content Type issue » Content Type title displays incorrectly
Component: other » menu system
chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Reviewed & tested by the community

Looks quite good.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Good, committed, thanks.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented

So maybe the warning on the add new content type page should be changed?
This name must begin with a capital letter and contain only letters, numbers, and spaces.

It seems like this isn't true. Is there any restriction on the human-readable name of the content type at all?

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.