My conditional needs a look over... 2

[ITGL72]

I have a page with just the following code in it:

<?php
if ($_COOKIE['company'] == 'HC')
{
header('Location: subscribeto_hc.php');
}
elseif ($_COOKIE['company'] == 'SVC')
{
header('Location: subscribeto_svc.php');
}
else
{
header('Location: subscribeto_sti.php');
}
?>



When I bring up this page I get errors like so:

Notice: Undefined index: company in C:\Accounts\stiesaco\

http://wwwRoot\online\subscribeto.php

on line 2

Notice: Undefined index: company in C:\Accounts\stiesaco\

http://wwwRoot\online\subscribeto.php

on line 6

Warning: Cannot modify header information - headers already sent by (output started at C:\Accounts\stiesaco\

http://wwwRoot\online\subscribeto.php:2)

in C:\Accounts\stiesaco\

http://wwwRoot\online\subscribeto.php

on line 12





Very green at PHP, can someone direct me to the proper code syntax to make this work?

 

[sleipnir214]

The "headers already sent" error will likely go away when you fix the first two.

The problem is that $_COOKIE['company'] is empty. If you attempt to use an uninitialized variable in a PHP conditional, you're going to get that error.

Take care of the case where $_COOKIE is empty first, then take care of the rest of the cases:

<?php
if (!isset($_COOKIE['company'])
{
	header('Location: subscribeto_sti.php');
}
else
{
	switch ($_COOKIE['company'])
	{
		case 'HC':
			header('Location: subscribeto_hc.php');
			break;
		case 'SVC':
			header('Location: subscribeto_svc.php');
			break;
		default:
			header('Location: subscribeto_sti.php');
	}
}
?>

Want the best answers? Ask the best questions! TANSTAAFL!

 

[ITGL72]

I'm now getting an error:

Parse error: parse error, unexpected '{' in C:\Accounts\stiesaco\

http://wwwRoot\online\subscribeto.php

on line 3

But to me it looks like the '{' is right but dont go by what I see.

 

[sleipnir214]

The code was untested.

Replace:

if (!isset($_COOKIE['company'])

with:

if (!isset($_COOKIE['company']))


Want the best answers? Ask the best questions! TANSTAAFL!

 

[ITGL72]

that did it, thanks!

 

[overyde]

Don't worry sleipnir214...
I'll organise a star for you!

Reality is built on a foundation of dreams.

 

[sleipnir214]

Thanks, overyde. I'm sure ITGL72 would have done it had he actually thought about it.


Want the best answers? Ask the best questions! TANSTAAFL!

 

[ITGL72]

Didnt know I could do that. I overlooked it.