[TAP5-773] Select component should have parameter to allow option labels to be rendered w/o HTML entity escaping

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.3.6, 5.2.5, 5.1.0.5
• Fix Version/s: 5.4
• Component/s: tapestry-core
• Assignee: Howard Lewis Ship
• Reporter: Andy Blower
• Created: 10/Jul/09 15:08
• Updated: 02/Aug/14 04:21
• Resolved: 01/Aug/14 18:26

Description

All ampersands that are returned from getLabel() in an OptionModel get encoded to &. This makes it impossible to use any html entities (—   etc) within option labels in selects. This is not that uncommon and should definitely be supported by Tapestry out of the box.

It can be fixed by changing line 60 of org.apache.tapestry5.internal.util.SelectModelRender from

writer.write(optionModel.getLabel());

to

writer.writeRaw(optionModel.getLabel());

This would mean that ampersands will need to be encoded for option labels, but this is at least possible for a developer to do. Currently it's not possible to use entities without using a custom select component, you can't even extend the T5 Select class and override the Render nested class.