Tim, could the auto-generated-by-Stripes FieldsPresent field berepurposed (for
lack of a better word) to support this?
-BD aka RJ
----- Original Message ----
From: Tim Fennell <[EMAIL PROTECTED]>
To: Stripes Users List <[EMAIL PROTECTED]>
Cc: Stripes Development List <[email protected]>
Sent: Wednesday, January 31, 2007 3:18:19 PM
Subject: Re: [Stripes-dev] [Stripes-users] stripes:errors displays errors in
sorted order, which is not required
Hey Barry,
Yes, I'd be open to patches....but I think this needs to be discussed in depth
on the dev list before anyone attempts a patch. I've cc'd that list and
suggest we continue the discussion there....
The best solution that I can think of that doesn't involve copious amounts of
developer intervention per-use-case would be to have the s:form tag build up a
cache of JSP->field ordering. All Stripes fields register with their parent
form tag - and assuming that the first pass render of the page isn't
problematic, would allow Stripes to figure out and capture the order of the
fields on the page (assuming Stripes tags are used). It'd seem to be
reasonable to hold this in a static Map of JspPath/Class -> List of Field
names. My only concern about this is that the footprint of caching this type
of information for each and every JSP would be non-trivial.
With this information in hand, the Errors tag could ask the form tag about the
ordering, and use that to sort the errors appropriately.
How does that solution sound to folks? It sounds a bit more complex for
sorting errors than I'd like....and I'm open to other solutions.
-t
On Jan 31, 2007, at 3:38 PM, Barry Davies wrote:
I've had users ask about this same issue, as well. I understand that a
stripes-tag-based approach to this issue wouldn't be sufficient for every use
case of stripes, but that doesn't mean that it isn't worthwhile. I mean, I
could choose to not use the <stripes:form> tag, but getSourcePageResolution()
won't work without it--and that wouldn't have been a good reason to not create
of getSourcePageResolution().
Tim, how open are you to receiving patches from the community for attempts at
providing a develper-specified order to validation errors?
-BD aka RJ
----- Original Message ----
From: Nic Holbrook <[EMAIL PROTECTED]>
To: Stripes Users List <[EMAIL PROTECTED]>
Sent: Wednesday, January 31, 2007 1:46:54 PM
Subject: Re: [Stripes-users] stripes:errors displays errors in sorted order,
which is not required
Just tell the users to quit being so picky... JK.
A custom index property would work with a stripes tag
but as Tim pointed out, you can use freemarker, plain
old jsp, etc. You don't necessarily need to use the
stripes tags so it wouldn't cover every situation.
Nic
--- "Newman, John W" <[EMAIL PROTECTED]> wrote:
> Ah I've always wondered why the errors come out in
> "random" order.
> Turns out it's not random and there's a good reason
> for it, I just never
> made the connection about the way the request params
> are sent in.
>
> How about using the tabindex attribute to order the
> errors? Or maybe a
> custom errorindex attribute? Not a great idea
> really but it would
> quickly solve the problem. I've actually had a few
> users mention "that
> is weird, make them come up in the right order"
>
> <stripes:text name="username" tabindex="1" />
> <stripes:text name="password" tabindex="2" />
>
> thoughts?
>
> - John
>
>
> -----Original Message-----
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]
> On Behalf Of Tim
> Fennell
> Sent: Wednesday, January 31, 2007 9:25 AM
> To: Stripes Users List
> Subject: Re: [Stripes-users] stripes:errors displays
> errors in sorted
> order,which is not required
>
> This is actually *much* trickier than it would seem.
> You are right
> in asserting that Stripes displays error messages in
> alphabetical
> order by field name. The reason it does this is to
> ensure some kind
> of consistency in error reporting.
>
> The "display errors in the order it encountered the
> error" is
> essentially arbitrary from Stripes' perspective.
> When a form is
> submitted there is no ordering to the parameters in
> the request, and
> so the order in which they are validated and bound
> is also arbitrary
> (not random though).
>
> Given that the ordering of fields on the page isn't
> required to
> mirror the ordering of fields on the ActionBean (or
> nested domain
> objects), and often won't match, the only way to
> figure out the
> "right" order would be to somehow get the source of
> the view and try
> and parse out the fields in order. Given that
> Stripes supports JSP
> and FreeMarker, and you aren't even required to use
> the Stripes tags,
> that's a bit difficult!!!
>
> For simple pages you can do the following:
> <s:errors field="username"/>
> <s:errors field="password"/>
>
> to force them out in order. But for larger pages,
> that's not
> practicable. If anyone has a good solution to this,
> I'm all ears.
>
> -t
>
> On Jan 31, 2007, at 9:13 AM, Ignacio Coloma wrote:
>
> > Are you using annotations? If so, in which order
> did you put them
> > in place?
> >
> > On 1/31/07, Tulip Agarwal <[EMAIL PROTECTED]>
> wrote:
> >>
> >> stripes:errors displays errors in sorted order,
> which is not what
> >> I require.
> >> I want that it should display errors in the order
> it encountered
> >> the error.
> >>
> >> For eg, if there are two fields on page
> "username" and "password"
> >> and I
> >> leave both the fields empty. I have validation on
> both the fields.
> >>
> >> stripes:errors would display error for password
> first and then for
> >> username,
> >> password is empty
> >> Username is empty
> >>
> >> where as it should say that
> >> Username is empty
> >> password is empty
> >>
> >> Please help me if anyone has a solution.
> >> --
> >> View this message in context:
> http://www.nabble.com/stripes%
> >>
>
3Aerrors-displays-errors-in-sorted-order%2C-which-is-not-required-
>
> >> tf3148349.html#a8727785
> >> Sent from the stripes-users mailing list archive
> at Nabble.com.
> >>
> >>
> >>
>
---------------------------------------------------------------------
>
> >> ----
> >> Take Surveys. Earn Cash. Influence the Future of
> IT
> >> Join SourceForge.net's Techsay panel and you'll
> get the chance to
> >> share your
> >> opinions on IT & business topics through brief
> surveys - and earn
> >> cash
> >> http://www.techsay.com/default.php?
> >> page=join.php&p=sourceforge&CID=DEVDEV
> >> _______________________________________________
> >> Stripes-users mailing list
> >> [EMAIL PROTECTED]
> >>
>
https://lists.sourceforge.net/lists/listinfo/stripes-users
> >>
> >
> >
>
----------------------------------------------------------------------
>
> > ---
> > Take Surveys. Earn Cash. Influence the Future of
> IT
> > Join SourceForge.net's Techsay panel and you'll
> get the chance to
> > share your
> > opinions on IT & business topics through brief
> surveys - and earn cash
> > http://www.techsay.com/default.php?
> > page=join.php&p=sourceforge&CID=DEVDEV
> > _______________________________________________
> > Stripes-users mailing list
> > [EMAIL PROTECTED]
> >
>
https://lists.sourceforge.net/lists/listinfo/stripes-users
>
>
>
------------------------------------------------------------------------
> -
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get
> the chance to share
> your
> opinions on IT & business topics through brief
> surveys - and earn cash
>
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDE
> V
> _______________________________________________
> Stripes-users mailing list
> [EMAIL PROTECTED]
>
https://lists.sourceforge.net/lists/listinfo/stripes-users
>
>
-------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get
> the chance to share your
> opinions on IT & business topics through brief
> surveys - and earn cash
>
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Stripes-users mailing list
> [EMAIL PROTECTED]
>
https://lists.sourceforge.net/lists/listinfo/stripes-users
>
____________________________________________________________________________________
Never Miss an Email
Stay connected with Yahoo! Mail on your mobile. Get started!
http://mobile.yahoo.com/services?promote=mail
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Stripes-users mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/stripes-users
Have a burning question? Go to Yahoo! Answers and get answers from real people
who
know.-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642_______________________________________________
Stripes-users mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/stripes-users
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development
____________________________________________________________________________________
Never miss an email again!
Yahoo! Toolbar alerts you the instant new Mail arrives.
http://tools.search.yahoo.com/toolbar/features/mail/
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development
