tapestry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Davor Hrg (JIRA)" <...@tapestry.apache.org>
Subject [jira] Commented: (TAPESTRY-2249) Add a "hidden fields" parameter to BeanEditor and BeanEditForm
Date Sun, 09 Mar 2008 18:33:46 GMT

    [ https://issues.apache.org/jira/browse/TAPESTRY-2249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12576806#action_12576806
] 

Davor Hrg commented on TAPESTRY-2249:
-------------------------------------

This could also be solved by using @DataType("hidden")

https://issues.apache.org/jira/browse/TAPESTRY-2012

and making "hidden" bean block contribution



> Add a "hidden fields" parameter to BeanEditor and BeanEditForm
> --------------------------------------------------------------
>
>                 Key: TAPESTRY-2249
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2249
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.0.11
>            Reporter: Geoff Callender
>            Priority: Minor
>
> A "hidden fields" parameter would allow code like this:
> <t:beaneditform t:id="form" object="person" submitLabel="Save" hidden="version"/>
> instead of this:
> <t:beaneditform t:id="form" object="person" submitLabel="Save">
> 	<!-- Include version in the form to prevent it being updated/refreshed in onActivate(),
which would defeat optimistic locking. -->
> 	<t:parameter name="version">
> 			<t:hidden t:id="version" value="person.version"/>
> 	</t:parameter>
> </t:beaneditform>
> Please note this is not the same as using remove, eg:
> <t:beaneditform t:id="form" object="person" submitLabel="Save" remove="version"/>
> which in this example would defeat optimistic locking because version's value would no
longer make the round-trip to the client.
> In case it helps, here's what onActivate() might looks like in this example:
> 	private Long _personId;
> 	@Persist("flash")
> 	private Person _person;
> 	void onActivate(Long id) throws Exception {
> 		_personId = id;
> 		if (_person == null) {
> 			_person = getPersonService().findPerson(_personId);
> 			if (_person == null) {
> 				throw new Exception("Person " + _personId + " does not exist.");
> 			}
> 		}
> 	}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org


Mime
View raw message