[DISCUSS] DELTASPIKE-512

[John D. Ament]

All,

Per the bug, this method has an error:

    @Override
    public void handleNavigation(FacesContext context, String
fromAction, String outcome)
    {
        if (!this.activated || isUnhandledExceptionQueued(context))
        {
            this.wrapped.handleNavigation(context, fromAction, outcome);
        }

        //don't refactor it - currently we need the lazy wrapping due
to special jsf2 constellations
        getWrappedNavigationHandler().handleNavigation(context,
fromAction, outcome);
    }

I'm thinking that the correct impl should be:

    @Override
    public void handleNavigation(FacesContext context, String
fromAction, String outcome)
    {
        if (this.activated || isUnhandledExceptionQueued(context))
        {
            getWrappedNavigationHandler().handleNavigation(context,
fromAction, outcome);
        }

        //don't refactor it - currently we need the lazy wrapping due
to special jsf2 constellations
        // is this message still valid?
    }

The only catch I see is the comment below.  WDYT?