Re: [phpesp-dev] Patch for Auto-publishing

[James E. F. <jf...@ac...>]

Well, I see the problem, but I fixed it in where/test.inc
instead. Now that the handler sanely treats the
QUERY_STRING, I removed the QS mangling code in test.inc,
that made it possible to embed the handler within the
mangagemnt interface. Please checkout the latest revision
of that file and see if your problems go away.

-James

On Tue, 16 Apr 2002, Kon Angelopoulos wrote:

> Hi Guys,
>
> Just downloaded the latest version to have a look at Matthews patch and was
> looking at the code for handler.php
>
>
>      75         $action = $HTTP_SERVER_VARS['PHP_SELF'];
>      76         if (!empty($HTTP_SERVER_VARS['QUERY_STRING']))
>      77                 $action .= "?" . $HTTP_SERVER_VARS['QUERY_STRING'];
>
> should line 77 read $action = "?" .$HTTP_SERVER_VARS['QUERY_STRING']; instead?
>
> otherwise couldn't we  have the following scenario when, for example we test
> surveys?
>
> ..../.../manage.php?where=test&sid=xx - then clicking the submit button would
> generate:
> ../../manage.php?where=test?where=test&sid=xx
>
>
> Regards
>
>
> Kon