Hello again,

I have 4 groups that access my webapp and they are listed within the cases in the code below. I keep recieving a parse error I can not for the life of me get around. The line that gives me the parse error is the bottom else { after all of the cases and right before the header info. Heres the code

PHP Code:

<?php
require 'globals/dbconnect.php';

session_start();
     $process_login = "SELECT * FROM users WHERE user_name='$user_name' AND password='$password'";
$login_result = mysql_query($process_login) or die (mysql_error());  
if(mysql_num_rows($login_result) > 0){
$result = mysql_fetch_array($login_result);
$groups = $result['groups'];
switch($groups) { 
case "cbragent":
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cbragent.php");
break;
case "cbrclient":
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cbrclient.php");
break;
case "cpmagent":
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cpmagent.php");
break;
case "cpmclient":
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cpmclient.php");
break;
};  
else { // <--------THIS IS WHERE I HAVE THE PROBLEM :mad: 
header("location: login.php?error=1");
exit;
};
?>

Suggestions?

Remove the ; after the } on the line just above it.

did that.. no dice

however i discovered that you cant use else statements in this kind of environment. And I forgot to add in a default for the case area. SO after dropping the else statement and assigning a default, I still cant get it to work

PHP Code:

<?php
require 'globals/dbconnect.php';

session_start();
     $process_login = "SELECT * FROM users WHERE user_name='$user_name' AND password='$password'";
$login_result = mysql_query($process_login) or die (mysql_error());  
if(mysql_num_rows($login_result) > 0){
$result = mysql_fetch_array($login_result);
$groups = $result['groups'];
switch($groups) { 
case "cbragent":
default: "location: login.php?error=1";
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cbragent.php");
break;
case "cbrclient":
default: "location: login.php?error=1";
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cbrclient.php");
break;
case "cpmagent":
default: "location: login.php?error=1";
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cpmagent.php");
break;
case "cpmclient":
default: "location: login.php?error=1";
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cpmclient.php");
break;
} 
?>

disregard, got it figured out. But my next issue is to get it so I know that I am only looking at my info only. This just puts me on a page thats not going to contain my data.

PHP Code:

<?php
require 'globals/dbconnect.php';

session_start();
     $process_login = "SELECT * FROM users WHERE user_name='$user_name' AND password='$password'";
$login_result = mysql_query($process_login) or die (mysql_error());  
if(mysql_num_rows($login_result) > 0){
$result = mysql_fetch_array($login_result);
$groups = $result['groups'];
switch($groups) { 
case "cbragent":
default: ("login.php?error=1");
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cbragent.php");
break;
case "cbrclient":
default: ("login.php?error=1");
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cbrclient.php");
break;
case "cpmagent":
default: ("login.php?error=1");
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cpmagent.php");
break;
case "cpmclient":
default: ("login.php?error=1");
$_SESSION['groups'] = $groups;
$_SESSION['logged_in'] = TRUE;
$_SESSION['user_name'] = $user_name;
header("location: cpmclient.php");
break;}
}; 
?>

You can only have one default for a switch statement and it does not take a value. The default clause is meant as a catch all for all other values not covered by other cases. See simple example below.

Your code has multiple defaults and pass values to it; that is your problem.

PHP Code:

<?php

$n = 2;
$value = "";

switch( $n )
{

    case 1:  $value = "one";
                 break;
    case 2:  $value = "two";
                 break;
    default:  $value = "A really big number.";
                 break;


}
?>

ok. so how do i assign each case to get to their own page that is in their respective headers?

Try removing the default clauses from your switch statement and replace them with one line at the end of your switch statement:

PHP Code:

default:  header("location: login.php?error=1");