You are viewing a plain text version of this content. The canonical link for it is here.

Posted to users@tapestry.apache.org by George Christman <gc...@cardaddy.com> on 2012/01/04 20:27:35 UTC

Server Side Validation with ajax form loop

Hello, I'm using the ajaxformloop component with a value encoder similar to
jumpstarts AjaxFormLoop Tailored version. The component works flawlessly
when there is no server side errors to return, however when server side
errors exist and the form is returned, my main pr object looses all the
ajaxformloop values that haven't yet been committed to the db. 

Is there way for the onValidate method to return my pr object without losing
the lineItem values and without persisting any of the data?

Please see code below. 

            <t:AjaxFormLoop t:id="lineItem" element="ul"
source="pr.lineItems" value="lineItem" addRow="block:addRow" show="show"
encoder="encoderLineItem">
                                <t:Label
for="quanity">QTY</t:Label><t:TextField disabled="roleManager.disabled"
title="Quantity" t:id="quanity" class="number qty"
value="lineItem.quanity"/>
</t:AjaxFormLoop>


   @OnEvent(EventConstants.ACTIVATE)
    void setupRender() {
        if (this.pr == null) {
            this.pr = new PurchaseRequest();
        } 
}

   void onValidateFromPR() throws Exception  {
//errors to return 
form.recordError(message + fieldName);
}

    @CommitAfter
    Object onSuccess() {
        session.saveOrUpdate(pr);
}

   @SuppressWarnings("unchecked")
    public ValueEncoder getEncoderLineItem() {
        return new ValueEncoder<LineItem>() {

            public String toClient(LineItem value) {
                Long id = value.getId();
                return id == null ? NEW_OBJ : id.toString();
            }

            public LineItem toValue(String idAsString) {
                LineItem lineItem = null;
                Long id = null;

                if (!idAsString.equals(NEW_OBJ)) {
                    id = new Long(idAsString);
                }

                // If new obj, return an empty obj.
                if (id == null) {
                    lineItem = new LineItem();

                    if (!request.isXHR()) {
                        pr.getLineItems().add(lineItem);
                    }
                } // Else, return existing obj
                else {
                    lineItem = (LineItem) session.get(LineItem.class, id);
                }

                // AjaxFormLoop will overwrite several fields of the obj
returned.
                // AjaxFormLoop can't handle null obj, so if null we return
a new empty obj.
                lineItem = lineItem == null ? new LineItem() : lineItem;
                lineItem.setPurchaseRequest(pr);

                return lineItem;
            }
        };
    }


Thanks. 



--
View this message in context: http://tapestry.1045711.n5.nabble.com/Server-Side-Validation-with-ajax-form-loop-tp5120576p5120576.html
Sent from the Tapestry - User mailing list archive at Nabble.com.

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

Re: Server Side Validation with ajax form loop work around

Posted by Geoff Callender <ge...@gmail.com>.

That's good news. Thanks for letting me know.

Geoff

On 18/04/2012, at 2:31 AM, David Canteros wrote:

> I have done a quick test and it works perfectly!
> Thanks Geoff!
> 
> 
> ------------------------------------------------------------------
> David Germán Canteros
> 
> 
> 2012/4/12 Geoff Callender <ge...@gmail.com>
> 
>> Hi all,
>> 
>> I've taken on board all the suggestions and observations here about
>> AjaxFormLoop problems and fixes, and done a complete rewrite of the
>> JumpStart examples. I've put a beta on the demo site. Is every corner case
>> handled now? I sure hope so. See if you can find a hole in it.
>> 
>>       http://jumpstart.doublenegative.com.au/jumpstart/
>> 
>> Cheers,
>> 
>> Geoff
>> 
>> 
>> On 23/03/2012, at 1:57 AM, George Christman wrote:
>> 
>>> Hi David, after some testing I wanted to provide you with some updates
>> based
>>> on some scenarios I ran into.
>>> 
>>> You'll notice in the !request.isHXR condition, I'm checking for a null
>>> lineItem id. This prevents duplicate objects from being added to the
>>> collection.
>>> 
>>>   @SuppressWarnings("unchecked")
>>>   public ValueEncoder getEncoderLineItem() {
>>>       return new ValueEncoder<LineItem>() {
>>>           public String toClient(LineItem value) {
>>>               Long id = value.getId();
>>>               return id == null ? NEW_OBJ : id.toString();
>>>           }
>>> 
>>>           public LineItem toValue(String idAsString) {
>>>               lineItem = null;
>>> 
>>>               if (!idAsString.equals(NEW_OBJ)) {
>>>                   Long id = new Long(idAsString);
>>>                   lineItem = (LineItem) session.get(LineItem.class, id);
>>>               }
>>> 
>>>               // AjaxFormLoop can't handle null obj, so if null we
>> return
>>> a new empty obj.
>>>               lineItem = lineItem == null ? new LineItem() : lineItem;
>>> 
>>>               if (!request.isXHR() && lineItem.getId() == null) {
>>>                   lineItem.setPurchaseRequest(pr);
>>>                   pr.getLineItems().add(lineItem);
>>>               }
>>>               return lineItem;
>>>           }
>>>       };
>>>   }
>>> 
>>> I also found a cleaner way to copy the persisted data to the new session.
>>> 
>>> 
>>>   void onActivate() {
>>>       System.out.println("onActivate " + this.pr);
>>> 
>>>       if (this.pr == null) {
>>>           this.pr = prPersist != null ? prPersist : new
>> PurchaseRequest();
>>>           prPersist = null;
>>>       }
>>> 
>>>   }
>>> 
>>>   Class<?> onActivate(Long prId) {
>>> 
>>>       PurchaseRequest purchaseRequest = prPersist != null ? prPersist :
>>> (PurchaseRequest) session.get(PurchaseRequest.class, prId);
>>>   }
>>> 
>>> If you have any better solutions, please feel free to share.
>>> 
>>> 
>>> --
>>> View this message in context:
>> http://tapestry.1045711.n5.nabble.com/Server-Side-Validation-with-ajax-form-loop-tp5120576p5586509.html
>>> Sent from the Tapestry - User mailing list archive at Nabble.com.
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>>> For additional commands, e-mail: users-help@tapestry.apache.org
>>> 
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
>> For additional commands, e-mail: users-help@tapestry.apache.org
>> 
>> 


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

Re: Server Side Validation with ajax form loop work around

Posted by David Canteros <da...@gmail.com>.

I have done a quick test and it works perfectly!
Thanks Geoff!


------------------------------------------------------------------
David Germán Canteros


2012/4/12 Geoff Callender <ge...@gmail.com>

> Hi all,
>
> I've taken on board all the suggestions and observations here about
> AjaxFormLoop problems and fixes, and done a complete rewrite of the
> JumpStart examples. I've put a beta on the demo site. Is every corner case
> handled now? I sure hope so. See if you can find a hole in it.
>
>        http://jumpstart.doublenegative.com.au/jumpstart/
>
> Cheers,
>
> Geoff
>
>
> On 23/03/2012, at 1:57 AM, George Christman wrote:
>
> > Hi David, after some testing I wanted to provide you with some updates
> based
> > on some scenarios I ran into.
> >
> > You'll notice in the !request.isHXR condition, I'm checking for a null
> > lineItem id. This prevents duplicate objects from being added to the
> > collection.
> >
> >    @SuppressWarnings("unchecked")
> >    public ValueEncoder getEncoderLineItem() {
> >        return new ValueEncoder<LineItem>() {
> >            public String toClient(LineItem value) {
> >                Long id = value.getId();
> >                return id == null ? NEW_OBJ : id.toString();
> >            }
> >
> >            public LineItem toValue(String idAsString) {
> >                lineItem = null;
> >
> >                if (!idAsString.equals(NEW_OBJ)) {
> >                    Long id = new Long(idAsString);
> >                    lineItem = (LineItem) session.get(LineItem.class, id);
> >                }
> >
> >                // AjaxFormLoop can't handle null obj, so if null we
> return
> > a new empty obj.
> >                lineItem = lineItem == null ? new LineItem() : lineItem;
> >
> >                if (!request.isXHR() && lineItem.getId() == null) {
> >                    lineItem.setPurchaseRequest(pr);
> >                    pr.getLineItems().add(lineItem);
> >                }
> >                return lineItem;
> >            }
> >        };
> >    }
> >
> > I also found a cleaner way to copy the persisted data to the new session.
> >
> >
> >    void onActivate() {
> >        System.out.println("onActivate " + this.pr);
> >
> >        if (this.pr == null) {
> >            this.pr = prPersist != null ? prPersist : new
> PurchaseRequest();
> >            prPersist = null;
> >        }
> >
> >    }
> >
> >    Class<?> onActivate(Long prId) {
> >
> >        PurchaseRequest purchaseRequest = prPersist != null ? prPersist :
> > (PurchaseRequest) session.get(PurchaseRequest.class, prId);
> >    }
> >
> > If you have any better solutions, please feel free to share.
> >
> >
> > --
> > View this message in context:
> http://tapestry.1045711.n5.nabble.com/Server-Side-Validation-with-ajax-form-loop-tp5120576p5586509.html
> > Sent from the Tapestry - User mailing list archive at Nabble.com.
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> > For additional commands, e-mail: users-help@tapestry.apache.org
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
> For additional commands, e-mail: users-help@tapestry.apache.org
>
>