Hi.
Although your example is very simple in code as such, it suffers from some
drawbacks, mainly because you don't have activation context for Save page.
1. Your create and edit actions on Start page use ActionLinks which make 2
HTTP requests (action + render Save page), whereas PageLinks with "myBeanId"
context value would execute just one HTTP request which will just render
Save page, and that is all you actually need
2. Its always good practice to pass only necessary activation context values
(in this case myBeanId), and let page itself be responsible to
reconstructing all that is necessary, meaning, it is not good to instantiate
new MyBean or fetch it from database outside of Save page as in your case,
but to do it inside it, in onPrepare event method. This way if you have some
additions to save page which maybe lead to another page and get back to it,
all you need to do when getting back to Save page is to make certain this
entity ID is present there, and page will know how to reconstruct itself.
3. If you keep only "myBeanId" inside Save as Session persistent value, your
Session has much less overhead in cluster recplication then when holding
whole instances. Though I guess this is just small gain, often dismissable.
Of course, all the written above depends on situation and your needs.
Regards,
Vjeran
----- Original Message -----
From: "Michael Courcy" <[EMAIL PROTECTED]>
To: "Tapestry users" <users@tapestry.apache.org>
Sent: Saturday, November 17, 2007 12:16 AM
Subject: How to make a basic crud : Feedback wanted
http://wiki.apache.org/tapestry/Tapestry5How_to_make_a_basic_crud
--
Michael Courcy
http://courcy.blogspot.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
