[MSSQL][PHP]Niedziałający warunek sprawdzania uprawnień użytkownika Opcje

[Adym]

Cześć,

Mam problem z warunkiem spawdzającym uprawnienia użytkownika do wykonania funkcji usunięcia zamówienia, który ma realizować poniższy kod:

[PHP] pobierz, plaintext 
public function delete(){
		if($this->order_id) {	
		if($_SESSION['user_type'] == 1 || $_SESSION['userid'] == $order['user_id']) {			
				$stmt_A = $this->conn->prepare("
					DELETE FROM ".$this->orderTable." 				
					WHERE order_id = ?");
 
				$order_id = htmlspecialchars(strip_tags($this->order_id));
 
				$stmt_A->bind_param("i", $order_id);
 
					if ($stmt_A->execute()) {
						$stmt_B = $this->conn->prepare("
						DELETE FROM ".$this->orderProductsTable." 				
						WHERE order_id = ?");
 
						$stmt_B->bind_param("i", $order_id);
 
						if ($stmt_B->execute()) {
						$stmt_C = $this->conn->prepare("
						DELETE FROM ".$this->orderRelationsTable." 				
						WHERE relation_order_id = ?");
 
						$stmt_C->bind_param("i", $order_id);
											}
						if ($stmt_C->execute()) {
							return true;
						}
					}
				}
			}
			else {return false;}
		}
[PHP] pobierz, plaintext 

Na początku sprawdzam czy funkcja została "nakarmiona" numerem id zamówienia (order_id) a później czy zalogowany użytkownik jest adminem ($_SESSION['user_type'] == 1) albo czy jest autorem kasowanego zamówienia ($_SESSION['userid'] == $order['user_id']). Jeżeli któryś z tych warunków jest prawdziwy funkcja powinna przejść dalej. W moim przypadku mimo tego warunku użytkownik albo kasuje wszystko albo admin nie może kasować. Nie wiem co jest przyczyną takiego działania. Czy ktoś może mi wytłumaczyć co robię źle?

Pozdrawiam

Ten post edytował Adym 8.02.2021, 10:25:58

[nospor]

Cos do poprawy? Hm... wszystko?
Pomijajac fakt, ze mega mieszasz logike i odpowiedzialnosc klas, to:

nie
$order_id = htmlspecialchars(strip_tags($this->order_id));

a:
$order_id = (int) $this->order_id);
Id to liczba i jako liczba masz to przedstawiac a zadne inne bezsensowne filtrowania nie beda potrzebne.

No i czemu do jednego zapytania to zabezpieczasz a do innego juz nie?
Czemu w jednym bindujesz a w innym juz walisz jako string?

nie ==
a ===

Jak juz zwracasz TRUE gdy sie powiedzie, to badz konsekwentny i zwracaj FALSE gdy sie nie powiedzie