Needs review
Project:
Content Taxonomy
Version:
6.x-1.0-rc2
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
11 May 2011 at 11:25 UTC
Updated:
19 Dec 2011 at 12:58 UTC
Jump to comment: Most recent file
To reproduce:
+ Create taxonomy with a few terms in, be sure to put a < or a > in one of the terms
+ Create a CCK content taxonomy field that selects these terms with the html characters in it
+ Go to the node add or edit page
You will now see that the term name is cut off at the < or > character.
I've created a patch that fixes the problem. Any feedback or improvements are very welcome.
Also, this issue might be related: http://drupal.org/node/301124
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | Capture d’écran 2011-11-07 à 15.37.46.png | 23.83 KB | slucas |
| #3 | content_taxonomy_1153858.jpg | 130.92 KB | weseze |
| #1 | htmlspecialchars.patch | 684 bytes | weseze |
| htmlspecialchars.patch | 686 bytes | weseze |
Comments
Comment #1
weseze commentedCorrect patch here (the previous one had the change commented out...)
Comment #2
he0x410 commentedI tried to reproduce that and didn't get cut off term. May you provide example of term/screenshot?
Comment #3
weseze commentedleft in screenshot: the part of my taxonomy listing that I'm using
right in screenshot: the resulting CCK field
Comment #4
he0x410 commentedok
Comment #5
slucas commentedJust try your patch it certainly work, but not for one specific character which is ' in french it could be use like in the phrase "d'architecture"
It renders as '
Do you have any idea why it doesn't work for this character.
It's in a exposed filter of a view on a content taxonomy field ?
Comment #6
weseze commentedTurns out that the problem is not with this module. Please do not apply patch. See: #321375: doesn't handle check_plain ("<any>" or "<none>" brakes)