[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]

to jest zbedne
$order_id = $this->order_id;

Dzialaja bezposrednio na $this->order_id


return false leci tylko w okreslonym wypadku. Po drodze jest cala masa IFow, ktore nie zwracaja nic. Zamiast ten else co dodales, daj poprostu
return false
na samym koncu funkcji

Unikaj zagniezdzen.

Zamiast kobylastdego IF
if($this->order_id) {
.......
}


daj tak

if(!$this->order_id) {
return false;
}
......