[PHP]Logowanie Opcje

[Mastroeni7]

Chcę, aby mój skrypt przekierowywał admina na inną stronę, oraz użytkownika na inną. Problemem jest to, że co nie wpiszę przechodzi mi tylko do 'user.php' Może powiecie co jest nie tak ? Na początku jeszcze wyskakuje mi błąd Notice: Undefined index: login in /home/***//logowanie.php on line 10

[PHP] pobierz, plaintext 
<?php
session_start();
 
include ('db.php');
function clean($value) {
	if(get_magic_quotes_gpc()) $value = stripslashes($value);
	return trim(mysql_real_escape_string($value));
}
 
if($_POST['login'] && $_POST['username'] && $_POST['password']) {
		$username = clean($_POST['username']);
		$password = md5($_POST['password']);
 
		$isAdmin = mysql_query("SELECT count(*) FROM users WHERE user_name = '$username' AND user_type = '1'");
		if(mysql_num_rows($isAdmin) == 1) {
				header("Location: admin.php");
				exit;
		}
 
		$result = mysql_query("SELECT count(*) FROM users WHERE user_name = '$username' AND user_password = '$password' and user_type='2'");
		if(mysql_num_rows($result) == 1) {
				$_SESSION['username'] = $username;
		header("Location: user.php");
				exit;
		} else {
				header("Location: opps.php");
		}
}
 
 
?>
formularz w htmlu
[PHP] pobierz, plaintext 

user_type = 1, mam w mysql dla admina, user_type = 2, dla użytkownika

[CuteOne]

1. Błąd dotyczy braku (lub złej nazwy) <input name="login"> - właściwie to po co ci ta zmienna skoro jej nie używasz?(IMG:style_emoticons/default/questionmark.gif) ?
2. Sprawdzaj czy zmienne $_POST są puste a nie czy istnieją.
3. eee... wystarczy podać login i już zostajesz przekierowany na stronę admina?

[PHP] pobierz, plaintext 
	if(!empty($_POST['username']) && !empty($_POST['password'])) {
 
		$username = clean($_POST['username']);
		$password = md5($_POST['password']);
 
		$query = mysql_query("SELECT password, user_type FROM users WHERE user_name = '$username'");
 
		if(mysql_num_rows($query)) {
 
			$row = mysql_fetch_array($query);
 
			if($row['password'] == $password) {
 
 
				$_SESSION['username'] = $username;
 
				switch($row['user_type']) {
 
					case 1:
					 $_SESSION['user_type'] = 1;
					 header("Location: admin.php");
					 break;
 
					case 2:
					 $_SESSION['user_type'] = 2;
					 header("Location: opps.php");
					 break;
 
					default:
					 header("Location: index.php");
					 break;
				}
			}
		}		
	}
 
	header("Location: login.php");
 
[PHP] pobierz, plaintext 

Skrypt ten nie uwzględnia wyświetlania błędów podczas złego wpisania loginu/hasła itp.

Ten post edytował CuteOne 20.03.2011, 02:52:19