>
> This looks like it fails if validation is set to alert mode; _autoSubmit
> always calls _validateInput, which calls _validateInline.
Thanks, fixed.
Also, if immediate is true, the _autoSubmit param validateForm is true?
> That seems weird...
BodyRenderer actually uses the snippet below, so whole-form validation
shouldn't happen if immediate is true.
builder.append(immediate ? "0" : "1");
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.
Do you mean abstract both copies into a common method, or are you
questioning the actual code. If that's the case, I spent a few hours one
evening trying to understand why I wasn't getting the 'real' event object in
IE.
-- 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
>
--
Chordiant Software Inc.
www.chordiant.com
