Ampersand in taxonomy term rendered as &

BENNYSOFT - June 9, 2008 - 11:03
Project:Taxonomy Browser
Version:5.x-1.3
Component:Code
Category:bug report
Priority:critical
Assigned:NancyDru
Status:closed
Description

If a taxonomy term name contains an ampersand (&) it is rendered as & in the category list.

On the rest of the site the terms are displayed with &.

#1

NancyDru - June 9, 2008 - 13:53
Assigned to:Anonymous» NancyDru
Status:active» postponed (maintainer needs more info)

Hmm, I'm not seeing this. Is there a URL where I can see it?

#2

NancyDru - June 9, 2008 - 14:04

I do see that the June 6 commit made a small change in this area. Are you using the versions after that?

#3

BENNYSOFT - June 9, 2008 - 17:24

I use 6.x-1.x-dev (2008-Jun-07)6.x-1.x-dev (2008-Jun-09).

#4

NancyDru - June 9, 2008 - 17:38

I'd like to see this on your site, if possible. As you can see from the attached image, I have several special HTML characters that display fine.

AttachmentSize
tb_amp_display.jpg 18.13 KB

#5

BENNYSOFT - June 9, 2008 - 18:18

Nancy, have you got my e-mail?

It works properly with the Selection type 'Check boxes' but not with 'Selection box'!

#6

NancyDru - June 9, 2008 - 18:29

Yes, I was just responding to it, and discovered the same thing. I don't know why the Forms API distinguishes, but the problem is in the core Forms module.

I'll take a look at the core code and see if it's worth submitting an issue and patch to fix it.

#7

NancyDru - June 10, 2008 - 16:54
Status:postponed (maintainer needs more info)» fixed

Fix committed to both branches.

#8

Junyor - June 19, 2008 - 19:54
Version:6.x-1.x-dev» 5.x-1.3
Priority:normal» critical
Status:fixed» active

After updating to Taxonomy Browser 5.x-1.3, the taxonomy browser page stopped displaying. I found that it was caused by using PHP 4 with this patch: htmlspecialchars_decode wasn't added until 5.1.0.

#9

NancyDru - June 20, 2008 - 04:09
Status:active» fixed

Yes, I discovered that in a another module and didn't realize I was using that here too.

Fix committed to both branches.

BTW, D7 will require at least PHP 5.2.5 and MySql 5+. Time to get your host started with upgrading to supported software levels.

#10

Junyor - June 21, 2008 - 19:34

Thank you.

#11

Anonymous (not verified) - July 5, 2008 - 19:43
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.