Utworzenie edytowalnych paragrafów JavaScript/PHP

Question

Cześć,

Piszę stronę na której utworzyłem prosty system logowania za pomocą sesji w PHP. Po naciśnięciu przycisku "Zaloguj się" wyskakuje okno (pop-up), w którym wpisujemy nazwę użytkownika oraz hasło i te dane są wysyłane na serwer gdzie PHP interpretuje je za pomocą instrukcji warunkowej i w tym momencie kiedy warunek jest spełniony rozpoczyna się sesja logowania. A kiedy klikniemy na przycisk "Wyloguj", skrypt PHP kończy sesję logowania. Moje pytanie jest następujące czy w JavaScript jest możliwość skonstruowania jakiejś funkcji która będzie wiedziała, że użytkownik jest zalogowany i w tym momencie przestawi wszystkie paragrafy na edytowalne. Wiem, że JavaScript nie będzie wiedziała, że użytkownik się zalogował bo to sprawdza PHP. Wstawiam moje skrypty PHP oraz JavaScript.

var modal = document.getElementById('id01');

window.onclick = function(event) {
    if (event.target == modal) {
        modal.style.display = "none";
    }
 
}



function loggedin() {
	
	var zalogowano = false;
	
	var username = document.getElementById("uname").value;
	var password = document.getElementById("psw").value;
	
	if (username == "Administrator" && password == "zaq1@WSX"){
		zalogowano = true;
		void(document.all.ne.contentEditable='true');
		
	} else {
		zalogowano = false;
	}
	
	alert("logowanie: "+zalogowano);
	
}

<?php
	session_start();
	if(isSet($_POST['signin']))
	{
		if($_POST['uname']=='Administrator' && $_POST['psw']=='zaq1@WSX')
		{
			$_SESSION['loggedin']=$_POST['uname'];
			header("location: index.html");
		} else
		{
			header("location: index.html");
		}
	}
	if(isSet($_SESSION['loggedin']))
	{
		header("location: index.html");
	}
?>

<?php
session_start();
if(!isSet($_SESSION['loggedin']))
{
	header("location: index.html");
}
session_destroy();
echo "Wylogowanie przebiegło pomyślnie. <a href='index.html'> Zaloguj ponownie</a>";
?>

Answer 1

1. Wykonujesz kluczową logikę bezpieczeństwa po stronie js. Nie powinno się tak robić. 
2. Po header Location użyj die. 
3. Jeśli js ma wiedzieć że user jest zalogowany to powinieneś ajaxem wysłać request i sprawdzić czy w sesji jest coś ustawione. 
4. == vs ===
5. Poczytaj o standardzie psr2
6. Zainteresuj się obiektowoscią. Na ogarnianie logowania itp itd przyjdzie czas. Nauka OOP to dobry wstęp do korzystania z frameworków: te posiadają wiele gotowych mechanizmów autoryzacji pozbawionych błędów, które sam na początku popełniasz. 

Comments

Cześć,

Dzięki za wskazówki zaraz to przerobię. 

Co do punktu nr.1 wiem, że nie powinno się tak robić tą funkcję stworzyłem na testy.