PHP- Redirect to another page

Question

My question is very simple. but I am struggling with it too much. The problem is once I hit the submit button that page should redirect to some other page. Kindly just tell me what I have done wrong? Because I have tried everything.

Code:

Update-docket.php

<?php
//error_reporting(0);
$link = mysqli_connect("localhost", "home_db", "root", "");

// Check connection 
if($link === false){
    die("ERROR: Could not connect. " . mysqli_connect_error());
}

// Escape user inputs for security
$sql = 'SELECT * FROM prod_details';
$result = mysqli_query($link,$sql);

while ($row = mysqli_fetch_array($result))
{ 
     $Quantity1 = $_POST['p_'.$row['id']. ''];          
     $id = $row['id'];
     $store = $row['rstore'];
     // echo $store;

     foreach ($_POST as $key => $value) 
     {}

     if(!empty($Quantity1)) {
         $query="update prod_details SET rdocket='$Quantity1' WHERE id='$id'";
         if(mysqli_query($link, $query)) {
            header('Location: /docket-details.php');
            exit();
         } else {
            echo "ERROR: Could not able to execute $sql. " . mysqli_error($link);
         }
      }
  }

docket-details.php

<form class="form-horizontal" action="update-docket.php" method="post" name="myform" autocomplete="off">
  <button  type='submit'  name='submit' class='btn c-theme-btn c-btn-square c-btn-uppercase c-btn-bold'>Submit</button>

Please help!!!

Seems your are using the wrong action on your form...is it right? — Albeis
– Albeis, Commented Jul 5, 2018 at 10:04
you have "update-details.php" but you reference "update-docket.php" in your form action — SuperDJ
– SuperDJ, Commented Jul 5, 2018 at 10:05
Maybe u sent the header already. Check: stackoverflow.com/a/21229246/10026429 for more info. — Wick96
– Wick96, Commented Jul 5, 2018 at 10:07

treyBake · Accepted Answer · 2018-07-05 10:42:31Z

3

You're Location line is incorrect, redirect using:

header('Location: /url');

https://secure.php.net/manual/en/function.header.php

update

I reckon your if statement is not being met. Try enabling error logging again and var_dumping inside each conditional section to see where the real problem lies. Now you've updated your code to use header it should in theory work. But my guess is $_POST not returning what you want/expect.

edited Jul 5, 2018 at 10:42

answered Jul 5, 2018 at 10:05

treyBake

6,5698 gold badges28 silver badges64 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Sharayu Over a year ago

Tried. Updated the code. No Changes. still the same.

Reflective · Accepted Answer · 2018-07-05 10:15:42Z

0

Edited!

Try this:

header('Location: /docket-details.php');
// i'm sorry, forgot to add exit
exit();

of course you can fill the full url if you refer.

Important: No output should be made before this line as it will generate a header. Not a single echo before sending the response header!

edited Jul 5, 2018 at 10:15

answered Jul 5, 2018 at 10:07

Reflective

3,9371 gold badge17 silver badges25 bronze badges

2 Comments

Reflective Over a year ago

No output should be made before this line as it will generate a header. Not a single echo before sending the response header!

Sharayu Over a year ago

I have updated my code. still facing same issue. It is only redirecting to update-details.php

treyBake · Accepted Answer · 2018-07-05 10:21:00Z

0

It should be something like that:

if(!empty($Quantity1)) {
    $query="update prod_details SET rdocket='$Quantity1' WHERE id='$id'";
    $res = mysqli_query($link, $query);

    if($res) {
        header('Location: /docket-details.php', true, '200');
    } else {
        echo "ERROR: Could not able to execute $sql. " . mysqli_error($link);
    }
}

edited Jul 5, 2018 at 10:21

treyBake

6,5698 gold badges28 silver badges64 bronze badges

answered Jul 5, 2018 at 10:10

RB_

1,1921 gold badge17 silver badges38 bronze badges

3 Comments

Sharayu Over a year ago

This page isn’t working is currently unable to handle this request. HTTP ERROR 500

RB_ Over a year ago

Well, it is hard to say what is wrong from your description. Try to break down the problem to a small chunks, and solve each of them. I suggest you to set the condition to true inside the if - if(!empty($Quantity1)) = if (true), and then leave only redirect line inside the if body, see if it works. If yes, then try executing query, if it doesn't work it can be another problem.

SuperDJ Over a year ago

@RB_ the error is quite easy. It's the ' before the / of the url

SuperDJ · Accepted Answer · 2018-07-05 11:22:19Z

you have "update-details.php" but you reference "update-docket.php" in your form action. So the form action redirects to a different file.

As stated by @thisGuyHasTwoThumbs your Location line is incorrect.

header('Location: /url');
exit();

You also want to exit() as it will prevent any execution of code after that line.

https://secure.php.net/manual/en/function.header.php https://secure.php.net/manual/en/function.exit.php

Jus for the details:

"update prod_details SET rdocket='$Quantity1' WHERE id='$id'"

Wil look much cleaner and is easier to read like:

"UPDATE `prod_details` SET `rdocket` = '$Quantity1' WHERE `id` ='$id'"

Keeps MySQL words in capital. Wrap column and table names in backticks will prevent "MySQL serverd word error"

Making it:

<form class="form-horizontal" action="update-docket.php" method="post" name="myform" autocomplete="off">

<?php
if (!empty( $Quantity1) )
{
    $query = "UPDATE `prod_details` SET `rdocket` = '$Quantity1' WHERE `id` ='$id'";
    if( mysqli_query( $link, $query ) )
    {
        header( 'Location: /docket-details.php' );
        exit();
    } else {
        echo "ERROR: Could not able to execute $sql. " . mysqli_error($link);
    }
}

The variable $Quantity1 is open for SQL-injection. To resolve this you can escape the variable if it is a string but you should use prepared statement.

Zlatan · Accepted Answer · 2019-07-17 08:15:14Z

0

It's also best to not output anything before the header() .. eg; don't echo "ERROR: Could not able to execute $sql. " . mysqli_error($link);

answered Jul 17, 2019 at 8:15

Zlatan

138 bronze badges

Comments

Clas Östman · Accepted Answer · 2019-08-30 06:41:04Z

0

It seems your file is named "Update-docket.php", with a capital U. But in the action of the form you call for action="update-docket.php".

answered Aug 30, 2019 at 6:41

Clas Östman

154 bronze badges

Comments

khyati naik · Accepted Answer · 2018-07-05 11:24:39Z

-1

header(location:pagename with extension)

If the above code is not running properly and it give error that "header does not change" the put the below line on the top of the page and bottom of the page respectively

ob_start() - on the top of the page ob_end_flush() - bottom of the page

answered Jul 5, 2018 at 11:24

khyati naik

11 bronze badge

Collectives™ on Stack Overflow

PHP- Redirect to another page

7 Answers 7

update

1 Comment

2 Comments

3 Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

7 Answers 7

update

1 Comment

2 Comments

3 Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related