myfaces-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adam Winer" <awi...@gmail.com>
Subject Re: [Trinidad] tr:table PPR failing
Date Thu, 13 Sep 2007 16:45:30 GMT
On 9/13/07, Adam Winer <awiner@gmail.com> wrote:
> On 9/13/07, Andrew Robinson <andrew.rw.robinson@gmail.com> wrote:
> > Timothy,
> >
> > I stripped the page and the AJAX down to the ID attributes only
> > (FYI I used this set of commands with VIM to do this:
> > %s/></>\r</g | %s/^.\+\(id="[^"]\+"\).*/\1/ | %s/^[^i].\+\n// | sort
> > )
>
> Can I get you on permanent retainer for VIM-editing? :)
>
> > Here are the results:
> >
> > Page:
> > id="acctPage:acctTable"
> > id="acctPage:acctTable:j_id_jsp_1566647018_17"
> > id="acctPage:acctTable:j_id_jsp_1566647018_19"
> > id="acctPage:acctTable:j_id_jsp_1566647018_21"
> > id="acctPage:acctTable__xc_ha"
> > id="acctPage:acctTable__xc_sa"
> > id="acctPage:acctTabledd0"
> > id="j_id0"
> >
> > AJAX response:
> > id="_acctForm_Postscript"
> > id="acctPage:acctTable"
> > id="acctPage:acctTable:0:j_id_jsp_1566647018_33"
> > id="acctPage:acctTable:j_id_jsp_1566647018_17"
> > id="acctPage:acctTable:j_id_jsp_1566647018_19"
> > id="acctPage:acctTable:j_id_jsp_1566647018_21"
> > id="acctPage:acctTable__xc_ha"
> > id="acctPage:acctTabledd0"
> > id="j_id0"
> >
> > As you can see, the following items in the AJAX were not in the page:
> > id="_acctForm_Postscript"
> > id="acctPage:acctTable:0:j_id_jsp_1566647018_33"
> >
> > Looking at your page source, there is no form element. The form is
> > required AFAIK. This may be just because you didn't provide the full
> > source though.
>
> I suspect so... the submission wouldn't have gotten far at all
> without a form element.  The "postscript" in the PPR reply
> is in fact part of the form.
>
> It looks as though the issue is exactly related to trying to get that
> expanded row to show up, since the one missing ID is ":0:"
> (that is, the first row of the table).  Lemme try this locally.

Ech, ignore that commentary.  Diffing the full set of IDs isn't
really relevant here:  PPR can, of course, introduce new IDs.
It just can't introduce new *top* IDs - each fragment has to
point to an existing ID.  We've got two fragments in here:

<div id="acctPage:acctTable">
<span id="_acctForm_Postscript">

And these do seem to be in the page already.  But even
if they weren't, it seems that the current JS code would simply
drop them and move on.

Tim, could you by any chance use FireBug's JS debugger to
step through TrPage.prototype._handlePprResponse()?  It's
not clear where this JS error is coming from.

(FWIW, the basic table row-disclosure demo seems to be working
fine for me, at least on FF (haven't tested IE)).

-- Adam

Mime
View raw message