On 10/11/05, Peter Speltz <[EMAIL PROTECTED]> wrote:
> On 10/11/05, Aaron Trevena <[EMAIL PROTECTED]> wrote:
> > Looking at the internal redirection code and threads I think it's
> > pretty clear that the workflow needs to be revised before we can have
> > a done flag or internal redirection.
>
> I disagree. I've been doing simple redirection for a year with my
> Redirect plugin.
Yes - simple internal redirection, as opposed to nice redirection that
works properly with the workflow. I was reading through the Apache
Modules (eagle) book last night thinking about how to implement it and
they have designed it very clearly and very powerfully.
Thats the way I think we should do it - working with the workflow
rather than working around it.
There is no reason not to use/release the plugin in the meantime, I
imagine it will pretty much work as expected even after reworking the
workflow and adding an internal_redirect method.
> $r->redirect("/$table/$action/$args");
>
> See the redirect plugin at www.cafes.net/peter
> Also could we consider $r->redirect rather than $r->internal_redirect.
I was looking at it last night and felt too much of a hack to go into
the Maypole core, as well as likely to complicate rather than simplify
the workflow. Sorry :(
> That way you could say:
> $r->redirect('http://www.google.com', new_window=>1 );
> just as
> $r->redirect('/beer/view/1');
ick - 'redirect' makes me think of HTTP redirect rather than
redirecting within the framework - thats why we have redirect request
and soon - redirect_internal which will combine ideas from your plugin
and the apache internal redirection.
cheers,
A.
ps cc:d to maypole-devel in case anybody has any thoughts
-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Maypole-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/maypole-devel
