FLUID-3844: RendererUtilities.js: Renderer does not expand UIMessage properly.

Metadata

Source
FLUID-3844
Type
Bug
Priority
Critical
Status
Closed
Resolution
Fixed
Assignee
Colin Clark
Reporter
y z
Created
2010-11-17T16:40:52.355-0500
Updated
2010-12-13T14:53:50.328-0500
Versions
  1. 1.2
Fixed Versions
  1. 1.3
Component
  1. Renderer

Description

The renderer has historically supported the construction where the fields for a UIMessage may themselves be a UIBound - for example in the form
{messagekey: {
valuebinding: "reference.to.key.in.model"
}}
which allows the resolution of an I18N message to be geared through a model. However, there is a "fault" in the implementation of the protocomponent expander, that it is not possible to take advantage of this facility by means of writing a UIMessage protocomponent with EL reference that expands to this form - for example the following construction
{messagekey: "${reference.to.key.in.model}"}
fails to expand properly (although "args") are expanded. This patch corrects the protocomponent expander so that the messagekey member of UIMessage is expanded as if it were a UIBound.

Comments

  • y z commented 2010-11-17T16:50:49.364-0500

    A patch with a test case

  • y z commented 2010-11-17T16:51:42.558-0500

    Hi Antranig, could you please review the patch. Thanks.

  • Justin Obara commented 2010-11-17T17:07:38.871-0500

    "Bug Parade Infusion 1.3"

  • Antranig Basman commented 2010-11-17T17:35:43.258-0500

    Patch committed at revision 10265 with a few minor tweaks to naming in test case

  • Justin Obara commented 2010-12-13T14:53:50.321-0500

    Antranig reviewed and committed Yura's patch