>>>>> "Shawn" == Shawn McKenzie <nos...@mckenzies.net> writes:

Shawn> Shawn McKenzie wrote:
>> John Stoffel wrote:
>>> Hi,
>>> I'm looking for some help with PHP5.2.x hacking on a web site to
>>> cleanup how the user submitted search form is handled.  Basically I
>>> want to make it simple for the end user to a) refine their query, b)
>>> not have to answer popups if they hit the reload button, and c) so I
>>> can learn more PHP.  I found this nice reference:
>>> http://us2.php.net/manual/en/language.variables.external.php
>>> which has two sections by "wayne" and "Jonas Lindel" talking about how
>>> you can take the results of a PHP POST request, stuff the vars into
>>> your session, and redirect back to the same page without the POST vars
>>> set, then complete the query.
>>> The idea is so that when people hit the "back" or "reload" buttons,
>>> they don't keep getting asked whether they want to resubmit their
>>> data.
>>> What I'm trying is not strictly needed, but it would be a nice cleanup
>>> of the interface of the search forms for this site.  
>>> I've been googling, reading php5.x docs and trying to understand what
>>> needs to be done here.  I think I'm just missing how to re-assign
>>> $this properly, so that when the redirect happens, the proper env is set.
>>> Here's the snippet I'm trying to use, which bombs out with:
>>> Fatal error: Cannot re-assign $this in .... on line 45
>>> And the code I'm using is:
>>> <?
>>> // We want to put the search query into a session so we can restore it
>>> // easily when users goto look at full_holdings.php and then return to
>>> // the overall display page.  We'll need to do a session_destroy in
>>> // index.php I think.  See the tutorial(s) at
>>> // http://www.phpf1.com/tutorial/php-sessions.html
>>> // http://us3.php.net/manual/en/book.session.php
>>> session_start();
>>> // Now see Jonas Lindel's & Wayne's code at
>>> // http://us2.php.net/manual/en/language.variables.external.php for
>>> // this *nice* trick.
>>> //is there POST or GET data we should look at
>>> if($_POST or ($_GET and (!isset($_SESSION['skipnextget'])))) {
>>> //is there GET data?
>>> if($_GET) {
>>> $newgetarr = array();
>>> foreach($_GET as $key => $value) {
>>> if(is_array($value)) {
>>> //if it's an array, convert it to comma separated
>>> $newvalue = implode(',',$value);
>>> } else {
>>> $newvalue = $value;
>>> }
>>> $newgetarr[] = $key.'='.$newvalue;
>>> }
>>> $newget = implode('&',$newgetarr);
>>> $_SESSION['skipnextget'] = 1;
>>> }
>>> foreach($_POST as $key => $value) {
>>> $this[$key] = $value;
>>> }
>>> $_SESSION['postvars'] = serialize($this); //put POST in the session
>>> header("Location: http://"; .$_SERVER['HTTP_HOST']
>>> . $_SERVER['PHP_SELF'] . '?\
>>> ' . $newget);
>>> //reload page without POST data and shorter GET data
>>> exit();
>>> } else { //after reloading we'll come right to this spot
>>> session_unregister('skipnextget');
>>> if(isset($_SESSION['postvars'])) {
>>> //load the POST variables from the session
>>> // Error on this line below!
>>> $this = unserialize($_SESSION['postvars']);
>>> session_unregister('postvars'); //delete the session stuff we
>>> //don't need anymore.
>>> return $this;
>>> }
>>> }
>>> // rest of my code goes here...
>>> _______________________________________________
>>> Thanks,
>>> John
>>> j...@stoffel.org
>> I'm not sure if it is technically reserved, but $this is used in a class
>> in order for an object to refer to itself.  AFAIK you can't use it
>> anywhere other than that.  Just change all $this to $vars or something else.

Shawn> Also, as you can see by the use of $this, this is old code.
Shawn> You need to change all of the session_unregister('varname')
Shawn> stuff to unset($_SESSION['varname']). session_unregister() is
Shawn> deprecated and will be removed in PHP 6.

Thanks for the feedback, someone should ask the admins to update some
of the comments on the php.net web pages then, esp since I think
they're trying to kill off 4.x support.

Anyway, I'm wondering if I'm doing this all wrong in some ways.  From
further googling and reading, I think I need to change from my current

        index.php - form for user to fill in.  POSTS to
        results.php  - checks input (barely :-) does query and shows
                     results.  I'm trying to have results.php redirect
                     to itself and clean out the POST vars.

                     It works, esp since I've got pagination and such
                     for my results pages, etc.  

                     The user can either do a new search, or refine
                     the existing results and goes back to the
                     index.php search page in that case. 

But I think I need to re-work things so that instead I do all form
validation and checks on the index.php page, and once I get all the
$_POST stuff, I redirect to the results.php page, using *only* session

Though I'm not sure how my page navigation is going to work, since
it's all based on buttons with hidden values.  I guess I should just
change them to session vars in that case...

But back to the $this = unserialize(...) stuff.  I tried your initial
suggestion and renamed it to 

           $that = unserialize(...)
           return $that;

and it doesn't work, I get nothing back at all.  So I suspect that
somehow I need to do something different.  

Is there a good reference for building search only forms with multiple
fields, multiple results on a page and sub-pages for detailed results,



PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to