Question

144

Javascript Redirect using setTimeout

rated 0 times [ 148] [ 4] / answers: 1 / hits: 44296 / 12 Years ago, tue, august 14, 2012, 12:00:00

I have a button that I want to run a piece of script and then redirect, however nothing is happening

I have another page that uses similar code and that works. but this one just isn't working

Code to work:

<a href=javascript:void(0); onclick='$.get(features.php,{ cmd: confirm_order, id: <?php echo $o_id; ?>, name: <?php echo $_SESSION['user_name']; ?> email: <?php echo $_SESSION['user_email']; ?>};setTimeout(window.location.href=order.php?order_id=<?php echo $o_id;?>, 100);' class=button>

Code that already works on a different page:

<a href=javascript:void(0); onclick='$.get(features.php,{ cmd: remove, id: <?php echo $o_id; ?>, prod: <?php echo $row_prod['prod_id']; ?> });setTimeout(window.location.href=window.location.href, 100);'>

Now I know it's all to do with my setTimeout, I'm just not sure what I'm doing wrong.

EDIT

Link now:

<a href=javascript:void(0); class=button confirm>Confirm</a>

Code below the link:

<script type=text/javascript>

$('a .confirm').on('click',function(e){

    $.get(features.php,{

        cmd: confirm_order,

        id: <?php echo $o_id; ?>,

        name: <?php echo $_SESSION['user_name']; ?>,

        email: <?php echo $_SESSION['user_email']; ?>

    });

    setTimeout(

        function(){

            window.location = order.php?order_id=<?php echo $o_id;?> 

        },

    100);

});

</script>

which still isn't working, and is rendering the in the code

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

zachary

Add To Favorites

Follow

Total Points: 175

Total Questions: 89

Total Answers: 108

Location: Fiji

Member since Wed, Jul 14, 2021

3 Years ago

zachary questions

1 Bootstrap 5 - Modal Open on Page Load

Tue, Oct 12, 21, 00:00, 3 Years ago

1 React + TypeScript: How to fix return type 'Element[]' is not a valid JSX element.?

Wed, Jun 9, 21, 00:00, 3 Years ago

1 Encountered Error: Material-UI: capitalize(string) expects a string argument. when using snackbar in React Material-UI

Sat, Jul 18, 20, 00:00, 4 Years ago

1 how to format date in jQuery DataTables

Thu, Jun 18, 20, 00:00, 4 Years ago

1 Node.js unable to import ES6 module

Sat, Jan 25, 20, 00:00, 4 Years ago

View All

answered 12 Years ago iselauniquef · Accepted Answer

I agree with Felix Kling comment that you shouldn't add that much code to an html attribute. I see that you're using jQuery so why don't you just add this to your javascript code:

$('a .button').on('click',function(e){

    $.get(features.php,{ 

          cmd: confirm_order,

          id: <?php echo $o_id; ?>,

          name: <?php echo $_SESSION['user_name']; ?>, // you were missing this comma 

          email: <?php echo $_SESSION['user_email']; ?>

        }).done(function(){

             window.setTimeout( function(){

                 window.location = order.php?order_id=<?php echo $o_id;?>;

             }, 100 );

        }); // you where missing this parenthesis          



  e.preventDefault(); 

});