This looks like it fails if validation is set to alert mode; _autoSubmit
always calls _validateInput, which calls _validateInline.
Also, if immediate is true, the _autoSubmit param validateForm is true?
That seems weird...
I think I've seen this code:
+ if (_agent.isIE)
+ {
+ // in many forms there is a hidden field named "event"
+ // Sometimes IE gets confused and sends us that instead of
+ // the true event, so...
+ if (event["type"] == "hidden")
+ event = window.event;
+ }
+
... elsewhere. Never a good thing to cut-and-paste in JS.
-- Adam
On 9/25/07, Danny Robinson <[EMAIL PROTECTED]> wrote:
> I'd like some quick eyes on the patch I've attached to TRINIDAD-37 which I'd
> like to commit if there are no issues.
>
>
>
>
> On 9/25/07, Adam Winer <[EMAIL PROTECTED]> wrote:
> > I think Danny still has some things?
> >
> > -- Adam
> >
> >
> > On 9/25/07, Matthias Wessendorf <[EMAIL PROTECTED]> wrote:
> > > anyone else that has some changes to commit ?
> > >
> > > Otherwise I start the branch tomorrow morning (German time)
> > >
> > > -Matthias
> > >
> > > On 9/25/07, Adam Winer <[EMAIL PROTECTED]> wrote:
> > > > OK, I've fixed 735, and checked in the patches
> > > > for 674 and 676.
> > > >
> > > > -- Adam
> > > >
> > > >
> > > > On 9/24/07, Adam Winer <[EMAIL PROTECTED]> wrote:
> > > > > We've also got patches available for TRINIDAD-718 and
> > > > > TRINIDAD-665 (actually looks like TRINIDAD-718 has
> > > > > already been checked in, but the JIRA wasn't closed).
> > > > >
> > > > > TRINIDAD-735 would be nice, but isn't critical.
> > > > >
> > > > > I'll check in the fixes for TRINIDAD-674 and TRINIDAD-676
> > > > > ASAP - straightforward fixes from Tomas Havelka, a shame
> > > > > to let them go unused.
> > > > >
> > > > > Anyone else have something they really want fixed for 1.0.3?
> > > > >
> > > > > Going forward, I don't think we need to have a strict
> > > > > "release the plugins, then release core" rule - I think we
> > > > > can mostly get away with just releasing core, when it's
> > > > > ready. Which is good. :)
> > > > >
> > > > > -- Adam
> > > > >
> > > > >
> > > > > On 9/24/07, Simon Lessard < [EMAIL PROTECTED]> wrote:
> > > > > > Ok, commit is done on my side.
> > > > > >
> > > > > >
> > > > > > On 9/24/07, Matthias Wessendorf < [EMAIL PROTECTED]> wrote:
> > > > > > > Sounds good to me,
> > > > > > > this email was just a heads-up ;-)
> > > > > > >
> > > > > > > On 9/24/07, Simon Lessard < [EMAIL PROTECTED]> wrote:
> > > > > > > > I need to commit the patch for statusIndicator. I'll do that
> later
> > > > > > today, we
> > > > > > > > should not release until it's done.
> > > > > > > >
> > > > > > > >
> > > > > > > > On 9/24/07, Matthias Wessendorf < [EMAIL PROTECTED]> wrote:
> > > > > > > > > one more.
> > > > > > > > >
> > > > > > > > > from September 30th => October 15th (incl.) I am away from
> any
> > > > > > computer !
> > > > > > > > :-)
> > > > > > > > > (vacation)
> > > > > > > > >
> > > > > > > > > So, the goal of this email was to prepare as much as
> possible to get
> > > > > > > > > the release out
> > > > > > > > > by end of this week ;-)
> > > > > > > > >
> > > > > > > > > Otherwise someone else needs to do the release procedure.
> > > > > > > > >
> > > > > > > > > -Matthias
> > > > > > > > >
> > > > > > > > > On 9/24/07, Matthias Wessendorf < [EMAIL PROTECTED]> wrote:
> > > > > > > > > > On 9/24/07, Danny Robinson < [EMAIL PROTECTED]>
> wrote:
> > > > > > > > > > > Did I miss the conversation leading up to this, or is
> this it?
> > > > > > > > > >
> > > > > > > > > > there was no thread regarding this.
> > > > > > > > > >
> > > > > > > > > > Usually after the release of the plugins, we start with
> the release
> > > > > > > > > > procedure of the core.
> > > > > > > > > > Since the vote on plugins is already ongoing, I was asking
> if we
> > > > > > > > > > should wait more days or not.
> > > > > > > > > > Looks like you've something for the 1.0.3, so we wait.
> That's fine.
> > > > > > > > > >
> > > > > > > > > > -M
> > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > To Do
> > > > > > > > > > > * resolve the current xOffset/yOffset conversation and
> fix the
> > > > > > plugins
> > > > > > > > > > > * Update AutoSubmitUtils to output '
> TrPage._autoSubmit()' as the
> > > > > > > > onchange
> > > > > > > > > > > handler
> > > > > > > > > > >
> > > > > > > > > > > Danny
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > On 9/24/07, Matthias Wessendorf < [EMAIL PROTECTED]>
> wrote:
> > > > > > > > > > > > Hi,
> > > > > > > > > > > >
> > > > > > > > > > > > I'd like to branch today for the upcoming 1.0.3-core
> release of
> > > > > > > > > > > > Trinidad. Does one need to commit something before ?
> > > > > > > > > > > >
> > > > > > > > > > > > thx,
> > > > > > > > > > > > Matthias
> > > > > > > > > > > >
> > > > > > > > > > > > --
> > > > > > > > > > > > Matthias Wessendorf
> > > > > > > > > > > >
> > > > > > > > > > > > further stuff:
> > > > > > > > > > > > blog:
> http://matthiaswessendorf.wordpress.com/
> > > > > > > > > > > > mail: matzew-at-apache-dot-org
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > --
> > > > > > > > > > > Chordiant Software Inc.
> > > > > > > > > > > www.chordiant.com
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > > Matthias Wessendorf
> > > > > > > > > >
> > > > > > > > > > further stuff:
> > > > > > > > > > blog:
> http://matthiaswessendorf.wordpress.com/
> > > > > > > > > > mail: matzew-at-apache-dot-org
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > Matthias Wessendorf
> > > > > > > > >
> > > > > > > > > further stuff:
> > > > > > > > > blog:
> http://matthiaswessendorf.wordpress.com/
> > > > > > > > > mail: matzew-at-apache-dot-org
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Matthias Wessendorf
> > > > > > >
> > > > > > > further stuff:
> > > > > > > blog: http://matthiaswessendorf.wordpress.com/
> > > > > > > mail: matzew-at-apache-dot-org
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > >
> > > --
> > > Matthias Wessendorf
> > >
> > > further stuff:
> > > blog: http://matthiaswessendorf.wordpress.com/
> > > mail: matzew-at-apache-dot-org
> > >
> >
>
>
>
> --
>
> Chordiant Software Inc.
> www.chordiant.com
