Never mind. I answered my own question. The RI is wrong. I found
the following lines in ViewHandlerImpl:
UIViewRoot result = new UIViewRoot();
result.setViewId(viewId);
Should we report this to the EG? I doubt it will be fixed this late
in the game but maybe it should be added to the list for the first 1.2
release.
sean
On 8/30/05, Sean Schofield <[EMAIL PROTECTED]> wrote:
> There is also a call to create view in WebFlowNavigationStrategy. I
> don't know anything about that so I'm not going to attempt a patch but
> it should probably be fixed to set the view Id as well.
>
> Craig, is there a bug in the RI then? I don't understand how your
> stuff would work against the RI if the RI was compliant in this
> regard.
>
> sean
>
> On 8/30/05, Gary VanMatre <[EMAIL PROTECTED]> wrote:
> > >>
> > > > This is a bug in Shale. I will post a bug to Bugzilla shortly.
> > > >
> > >>
> > >> I assume they are talking about the RestoreViewPhase verses a decorator
> > >> of
> > >> a handler calling createView on its original handler which would make it
> > >> the
> > >> caller? I guess that I'm wondering if the ShaleViewHandler could
> > >> explicitly
> > >> set the viewId on the returned UIViewRoot?
> > >
> > >
> > >Sean was correct ... the Shale ViewHandler (or any other ViewHandler)
> > >should
> > >*not* set the viewId into the created UIViewRoot instance ... that should
> > >be
> > >done by the caller. I didn't look at our Clay or Tiles view handlers to
> > >make
> > >sure they don't suffer from the same problem, but I fixed ShaleViewHandler
> > >and the dialog navigation handler per Sean's patches.
> >
> > Nothing gets past those myfaces guys. I did a quick check and I think Clay
> > and Tiles view handlers are ok.
> >
> > Gary
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
