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

[bu...@apache.org]

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: dev@cocoon.apache.org
        ReportedBy: m_rolappe@web.de
                CC: m_rolappe@web.de


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().