Witam:)
Mam nadzieje, że mi pomożecie ) Od razu zaznaczam Że szukałem dużo w google jak pisać logowanie co najlepiej używać.

Chciałbym Was prosić o sprawdzenie, czy takie logowanie może być, hmm to nie ma być skrypt do jakiegoś dużego serwisu, tylko do strony gdzie dostęp do panelu będzie miał tylko i wyłącznie admin. I chcę też zaznaczyć, że dopiero zaczynam przygodę z tym językiem.

Ogólnie skrypt jest na sesji oraz ciachu, żeby pamiętał logowanie, aktualnie jest na godzinę. Skrypt sprawdzony i ogólnie działa, jedyny błąd jaki występuje to wciąż napis "Musisz podać login oraz hasło." Z tego co mi się wydaje dzieje się tak przez klamrę z 10 linijki, lecz gdy chciałem ją gdzie indziej umieścić zwracał mi błąd;/

Proszę jeszcze o wyrozumiałość i nie negowanie od razu tego co tu spleciłem:P

[PHP] pobierz, plaintext 
<?php
session_start();
ob_start();
include('config.php');
 
if (empty( $_POST['login'] ) && empty( $_POST['haslo'] ) )
{
	echo 'Musisz podać login oraz hasło.';
}	
	else 		
		{
			$login = mysql_real_escape_string($_POST['login']);
			$haslo = mysql_real_escape_string(md5($_POST['login']));
 
			$query = "SELECT * FROM administracja WHERE login = '$login' and haslo = '$haslo'";
			$idquery = mysql_query($query);
			$user = mysql_fetch_array($idquery);
 
			if($login == ''.$user['login'].'' && $haslo == ''.$user['haslo'].'')			
			{
				if(isset($_POST['pamietaj']))				
				{
					$_SESSION['login'] = $user['login'];
					setcookie("admin","admin", time()+3600);
					header("location: index.php");
				}
 
			$_SESSION['login'] = $user['login'];
			}
			else			
				{
					echo 'Podałeś błędny login lub hasło';
				}
 
		}
 
	  if($_SESSION['login'] || isset($_COOKIE['admin'])) 	
	{	
		echo 'Jestes zalgowany jako '.$_SESSION['login'].', <a href="?p=wyloguj">Wyloguj</a>';
	}
		else		
		{
			echo '<form method="POST">
				  <input type="hidden" name="send"/>
				  Login:<input type="text" name="login"><br/>
				  Haslo:<input type="password" name="haslo"><br/>
				  <INPUT TYPE="checkbox" name="pamietaj"><b>Zapamiętaj</b><br />
				  <input type="submit" name="submit" value="Loguj"></form>';
		}	
 
	if($_GET['p']=='wyloguj') 	
	{ 
 
		session_destroy(); 
		setcookie("admin" , "admin", time()-3600);  
		header("Location: index.php");  
	}
ob_end_flush();
?>	
[PHP] pobierz, plaintext 

co do komunikatu "Musisz podac......" skoro przy logowaniu tworzysz zmienną sesyjną wykorzystać to i zamiast POST login i hasło sprawdź czy dana zmienna sesyjna istnieje.

[PHP] pobierz, plaintext 
if($login == ''.$user['login'].'' && $haslo == ''.$user['haslo'].'')
[PHP] pobierz, plaintext 

po co taki bigos?

[PHP] pobierz, plaintext 
if($login == $user['login'] && $haslo == $user['haslo'])
[PHP] pobierz, plaintext 

wystarczy

co do "Musisz podać login oraz hasło."
to daj go jako

[PHP] pobierz, plaintext 
if(isset($_POST)) {
   .... // jeśli formularz wysłany
}
[PHP] pobierz, plaintext 

btw. linia 6 i 13,
nie powinno być

[PHP] pobierz, plaintext 
$haslo = mysql_real_escape_string(md5($_POST['haslo']));
[PHP] pobierz, plaintext 

?

Hmmm;p Kurcze nie rozumiem.

Chodzi o coś takiego?

[PHP] pobierz, plaintext 
if (isset($_SESSION('login')))
{
 
//tutaj ten kod początkowy?
 
[PHP] pobierz, plaintext 

niezupełnie*

[PHP] pobierz, plaintext 
if ( empty( $_POST ) && isset( $_POST ) ) {
 echo 'Musisz podać login oraz hasło.';
}
[PHP] pobierz, plaintext 

Super;) Już jest Ok;) Wystarczyło dodać to

[PHP] pobierz, plaintext 
if(isset($_POST['send']))
{
// dalszy kod
[PHP] pobierz, plaintext 

PS. Dzięki za korekte;P Bo nie zwróciłem uwagi że w zmiennej hasła był login;p


A teraz, czy możecie mi powiedzieć, czy jak na początek to w miarć dobry kod?

dobry, ale co do autologowania
1. nowa tabela w bazie danych
2a. przy logowaniue normalnym z zapamietaniem logowania generowac kod i dodac do bazy danych (i zapisac w cookie)
2b. przy autologowaniu porownanie kodu z cookie do tego w bazie
3. wygenerowanie nowego kodu (po autologowaniu, nie po kazdym przelozeniu strony)
4. jesli to panel admina to jeszcze do bazy zapisywac id i sprawdzac potem cyz jest to samo (daje nam to pewnosc ze nawet jak ktos wykradniem nam kod z cookie to i tak sie nie zaloguje)

Hmmmm:P No kurcze wszystko fajnie pisze:) Ale niestety nie mam zielonego pojęcia jak to zrobić Więc wiesz:D

Średnio z tym:)

Kurcze słuchajcie, mam jeszcze jeden problem z tego co widzę;//

Cały kod mam zaincludowany przed znacznikami <html>. I z tego co widzę, robią mi się krzaki:// Kod pisany jest w notepad++ Kodowanie UTF-8 (bez BOM).
Oto co mi się pokazuje przy wyświetlaniu:/ Same krzaczyska;/ I tekst wylewający się poza strone.
Przykład niepoprawnego wyświetlania


A gdy już nacisnę przycisk loguj, nieważne czy zaloguje, czy wyskoczy błąd hasła, bądź komunikat wprowadź hasło i login, to wszystko wraca tak jak powinno być;// Tutaj potwierdzenie w screenie;// Czy ma ktoś jakiś pomysł
Przykład poprawnego wyświetlania