DO NOT REPLY [Bug 26854] New: - [PATCH] redirect problems

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=26854>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=26854

[PATCH] redirect problems

           Summary: [PATCH] redirect problems
           Product: Cocoon 2
           Version: 2.1.3
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: core
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]
                CC: [EMAIL PROTECTED]


symptom(s):

assume following pipeline:

...
        <map:pipeline>
                <map:match pattern="flow-google-redir">
                        <map:call function="flow"/>
                </map:match>
                
                <map:match pattern="google-redir">
                        <map:redirect-to uri="cocoon:/google"/>
                </map:match>

                <map:match pattern="google">
                        <map:redirect-to uri="http://www.google.de"/>
                </map:match>
        </map:pipeline>
...

and the flow script:

function flow()
{
        cocoon.sendPage("google-redir");
}


requesting 'flow-google-redir' or 'google-redir' will yield a blank page.

cause:
TreeProcessor.ForwardEnvironmentWrapper doesn't correctly propagate the redirect
to the environment.

resolution:
the attached patched makes ForwardEnvironmentWrapper override redirect() and
propagate the redirect to the wrapped environment. it also implements the
changes from bug #25437 and might resolve bug #22132 and bug #26571.

it also removes the release() of the SourceResolver in configure(), since the
resolver is released in dispose().