[Phplib-trackers] [ phplib-Feature Requests-452437 ] $sess->new_query() required

[<no...@so...>]

Feature Requests item #452437, was opened at 2001-08-18 02:06
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=403614&aid=452437&group_id=31885

Category: None
Group: None
Status: Open
Priority: 5
Submitted By: Richard Archer (richardarcher)
Assigned to: Nobody/Anonymous (nobody)
Summary: $sess->new_query() required

Initial Comment:
Regarding add-query(), the docs say:  Intended usage is
like this:

<a href="<<?
$sess->pself_url().$sess->padd_query(array("again"=>"yes"))
?>"> Reload</a> and log in?

The way it works is that if there is an existing value
in QUERY_STRING, add_query() expects self_url() to have
already ouput it. So it uses & as the sepchar and adds
the extra vars onto the end.

If add_query() is called after purl() and with a value
in QUERY_STRING, add_query() assumes QUERY_STRING has
been output, so uses & as sepchar. In fact purl() won't
have output the QUERY_STRING therefore ? would be the
correct spechar, so add_query() does the wrong thing.

There needs to be a series of methods that allow query
strings to be added to arbitrary URLs. I suggest
methods called new_query() and pnew_query(). Usage
would be:

<a href="<?
$sess->purl("another_page.php").$sess->pnew_query(array("param"=>"val"))
?>">another page</a>

These would function just like add_query() except they
would assume QUERY_STRING had not been output.

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=403614&aid=452437&group_id=31885