[
https://issues.apache.org/jira/browse/TAP5-2099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15158528#comment-15158528
]
Lance edited comment on TAP5-2099 at 2/23/16 8:34 AM:
------------------------------------------------------
Until you find a place to store the new value I think that throwing exception
is the right behaviour. Otherwise a mixin would happily update the value but
the old value would still return when {{get()}} is invoked
You are correct that the binding needn't store the value inside itself. In the
case of the {{prop:}} binding it passes through to the property setter which
ultimately results in a PerThreadValue being updated (all page / component
properties are transformed into PerThreadValues via a plastic)
was (Author: uklance):
Until you find a place to store the new value I think that throwing exception
is the right behaviour. Otherwise a mixin would happily update the value but
the old value would still return when `get()` is invoked
You are correct that the binding needn't store the value inside itself. In the
case of the `prop:` binding it passes through to the property setter which
ultimately results in a PerThreadValue being updated (all page / component
properties are transformed into PerThreadValues via a plastic)
> Update parameters bound to literal: or symbol: from a mixin
> -----------------------------------------------------------
>
> Key: TAP5-2099
> URL: https://issues.apache.org/jira/browse/TAP5-2099
> Project: Tapestry 5
> Issue Type: Improvement
> Affects Versions: 5.3.6, 5.4
> Reporter: Lance
> Labels: patch
> Attachments: TAP52099-1.patch
>
>
> Currently, I cannot update a component component parameter that is
> initialized with a literal: or symbol: binding from a mixin. If I try to
> @BindParameter the param and change it's value, I get a "Binding %s is
> read-only" exception (originating from AbstractBinding).
> I propose that literal and symbol bindings are sourced from a PerThreadValue
> that can be updated by a mixin.
> My original need for this was to create a "SinglePage" mixin for the Grid
> component which would set the rowsPerPage parameter to Integer.MAX_VALUE. I
> can't currently do this because by default, the parameter is a symbol:
> binding.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
