5

I am attempting to pass variables through the URL using the header function as a way to redirect the page. But when the page is redirected it passes the actual variable names rather than the values associated with the variables. I am new to PHP and do not fully understand the syntax so any further explanation as to the proper way to do this would be much appreciated.

header('location: index.php?id=".$_POST[ac_id]."&err=".$login."');
Joe
  • 51
  • 1
  • 1
  • 2

6 Answers6

17

You want:

header("Location: index.php?id=".$_POST['ac_id']."&err=".$login);

You were combining ' and " in this string, which is why it couldn't interpolate the variables properly. In my example above, you are strictly opening the string with " and concatenating the variables with the string.

Mike Lewis
  • 63,433
  • 20
  • 141
  • 111
  • Both lost quotes in $_POST ;-) – zerkms Apr 26 '11 at 02:33
  • this will raise a notice because of the unquoted string for the array key. – Stephen Apr 26 '11 at 02:33
  • there is no reason to concatenate the strings & variables, as PHP will expand variables inside double quoted strings – Stephen Apr 26 '11 at 03:04
  • It is personal preference, and if you want to be nit picking about micro-performance see here(concatenation is slightly faster): http://stackoverflow.com/questions/13620/speed-difference-in-using-inline-strings-vs-concatenation-in-php5/13665#13665 – Mike Lewis Apr 26 '11 at 03:07
  • It's generally accepted by most developers I've worked with that if you want to do string concatenation, use single quotes, if you want to use inline variable expansion, use double quotes. – Stephen Apr 26 '11 at 12:46
3

You have quotes within quotes. Try this instead:

header('location: index.php?id=' . urlencode($_POST['ac_id']) . '&err=' . urlencode($login));

The urlencode() function takes care of any reserved characters in the url.

What I would do instead is use http_build_query(), if you think you will have more than one or two variables in the URL.

header('Location: index.php?' . http_build_query(array(
    'id' => $_POST['ac_id'],
    'err' => $login
)));

Also, you technically can't use relative paths in the location header. While it does work with most browsers, it is not valid according to the RFCs. You should include the full URL.

Brad
  • 159,648
  • 54
  • 349
  • 530
1

Try SESSION storage. header is use to redirect the page. and if you really want to pass values through header only then u have genrate url. header('location:destination.php? value1=1&value2=3'); but it is not a good practice for vars. just store values in SESSION global var. B4 the header() redirection call. @the recieve page u have to test, if the session val isset() n !empty() then ... or else ...

Hope this wil help.

Junaid
  • 35
  • 3
-1

In my case, many times header() not working properly. Instead of header function, I use window.location.href. You can try like this,

echo "<script> 
            window.location.href='index.php?id=".$_POST[ac_id]."&err=".$login';
      </script>";
nafischonchol
  • 104
  • 1
  • 9
-3
header('location: index.php?id='.$_POST['ac_id'].'&err='.$login);
LanceHub
  • 590
  • 1
  • 4
  • 5
-3

Try this:

header("location: index.php?id=$_POST[ac_id]&err=$login");

PHP variables are expanded inside double-quoted strings.

Stephen
  • 18,597
  • 4
  • 32
  • 33