[Resolved] please please help with my if statement

webdesignhelper

Hi,

I am going nuts trying to work this out, i have five variables coming into a script, and depending on the combination of the variables i want a different forward to work, but i can not get it to work correctly

my code:

<?php

if($product == '1' && $currency == '1' && $paytime == '1' && $method == '1' && $setup == '1'){

$forward="Location:https://www.paypal.com/subscriptions/business=contact%40webdesignhelper%2eco%2euk&item_name=Starter%20Hosting%20Package%20%2d%20with%20an%20extra%20%a310%2e00%20add%20to%20the%20first%20month%20for%20registration%20of%20a%20new%20domain%20name&item_number=1&no_shipping=1&no_note=1&currency_code=GBP&a1=12%2e99&p1=1&t1=M&a3=2%2e99&p3=1&t3=M&src=1&sra=1";
header($forward);

}

if(($product == '1') && ($currency == '1') && ($paytime == '1') && ($method == '1') && ($setup == '2')){

$forward="Location:https://www.paypal.com/subscriptions/business=contact%40webdesignhelper%2eco%2euk&item_name=Starter%20Hosting%20Package%20%2d%20using%20an%20existing%20domain%20name&item_number=1&no_shipping=1&no_note=1&currency_code=GBP&a3=2%2e99&p3=1&t3=M&src=1&sra=1";
header($forward);

}

above is two methods i have tried but neither work ??

so all 5 variables have to match the combination to go into the if statement and use the forward inside it.

What am i doing wrong ?

bradgrafelman

"$forward=" will do absolutely nothing except set a variable.

Read up on using a proper [man]header()[/man] function.

webdesignhelper

the headers bits i will figure out later, its the if statment which i need the help with 🙂

bradgrafelman

This line:

if($product == '1' && $currency == '1' && $paytime == '1' && $method == '1' && $setup == '1'){

looks fine. Maybe the variables are not what you think they are. Try doing:

 echo "product: $product<br>currency: $currency<br>paytime: $paytime<br>method: $method<br>setup: $setup";

so you can see the actual values. Paste the values in here, and we'll go from there.

webdesignhelper

i'm sending in my own variables via the link so i know what is going in:

http://www.hostinghelper.co.uk/order_page5.php?domain=hostinghelper.co.uk&setup=1&product=1&currency=1&paytime=1&method=1

i have also already checked the variables and they are coming in correct.

to be certain i also ran the line which gave:

product: 1
currency: 1
paytime: 1
method: 1
setup: 1

webdesignhelper

my if statements are working correctly as you said, how do i set the header function you said i should do ?

thank you for your help, it is very much appreciated 🙂

bradgrafelman

Let's try a "debug" script, as I would call it.

if($product == '1') {
echo "product is '1'";
} elseif($product == 1) {
echo "product is 1";
}

echo '<br>';

if($currency == '1') {
echo "currency is '1'";
} elseif($currency == 1) {
echo "currency is 1";
}

echo '<br>';

if($paytime == '1') {
echo "paytime is '1'";
} elseif($paytime == 1) {
echo "paytime is 1";
}

echo '<br>';

if($method == '1') {
echo "method is '1'";
} elseif($method == 1) {
echo "method is 1";
}

echo '<br>';

if($method == '1') {
echo "method is '1'";
} elseif($method == 1) {
echo "method is 1";
}

echo '<br>';

if($setup == '1') {
echo "setup is '1'";
} elseif($setup == 1) {
echo "setup is 1";
}

echo '<br>';

Paste the output again if you could.

bradgrafelman

Instead of "$forward=" simply use:

header("Location: example.com");

NOTE: The above post was submitted before I had a chance to read your previous post. Please disregard it.

webdesignhelper

i have updated my script to:

<?php

if($product == '1' && $currency == '1' && $paytime == '1' && $method == '1' && $setup == '1'){

// echo "product: $product currency: $currency paytime: $paytime method: $method setup: $setup";

header("Location:https://www.paypal.com/subscriptions/business=contact%40webdesignhelper%2eco%2euk&item_name=Starter%20Hosting%20Package%20%2d%20with%20an%20extra%20%a310%2e00%20add%20to%20the%20first%20month%20for%20registration%20of%20a%20new%20domain%20name&item_number=1&no_shipping=1&no_note=1&currency_code=GBP&a1=12%2e99&p1=1&t1=M&a3=2%2e99&p3=1&t3=M&src=1&sra=1");

exit;
}

if($product == '1' && $currency == '1' && $paytime == '1' && $method == '1' && $setup == '2'){

// echo "product: $product currency: $currency paytime: $paytime method: $method setup: $setup";

header("Location:https://www.paypal.com/subscriptions/business=contact%40webdesignhelper%2eco%2euk&item_name=Starter%20Hosting%20Package%20%2d%20using%20an%20existing%20domain%20name&item_number=1&no_shipping=1&no_note=1&currency_code=GBP&a3=2%2e99&p3=1&t3=M&src=1&sra=1");

exit;
}

if i take out the comments for the echos the if statements work just fine, but if i run the script above with the header lines in it always redirects the browser to the second statement and not the first ?

bradgrafelman

I don't quite understand what you're saying, but try to use 'else' here.

if(...) {

....

} elseif (...) {

...

}

webdesignhelper

i think that has cracked it, thank you so much 😃 😃 😃

webdesignhelper

i will have a longer play in the morning, and marked it as solved if it has 🙂

webdesignhelper

thank you for your help, it has been soooo much appreciated, the problem is now resolved 😃 😃