PartialViewContextImpl should use same VisitContext to reset values as the one used to process the execute and render

[BalusC]

#5396

[BalusC]

cc: @tandraschko

[tandraschko]

+1 but do we really need EXECUTE_LIFECYCLE? AFAICS its not used in our IMPLs but the MF javadoc says something like

     * Hint that indicates that the visit is being performed as part of
     * lifecycle phase execution and as such phase-specific actions
     * (initialization) may be taken.

if i reset something, i dont want to trigger "initialization" of something

[tandraschko]

currently it's only used for partial processing the tree, i dont think we should use it for resetting components

[BalusC]

Fair point, adjusted.

[tandraschko]

nice!

@BalusC
Do you have already plans to merge this and release 4.0.6?
I would like to have a release, so we can further work on DeltaSpike integration tests as they currently failing because of the other PR encoding/encodeRedirectUrl thing which is already fixed and merged