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

Tak jak małe dziecko uczy sie mówić naśladując zasłyszane słowa od rodziców tak ja uczę się php wykorzystując fragmenty kodu bardziej doświadczonych kolegów.

Tylko ze male dziecko, z racji faktu ze nie ma wybory, to musi tak robic. Ty zdaje sie umiesz czytac i jakies podstawy wpierw bys mogl ogarnac sam, nie sadzisz?

Skoro to $order gdzies tam przeciez masz, to nie mozesz go przekazac do funkcji delete() jako parametr? Ewentualnie jako wlasciwosc klasy na podobnej zasadzie co $this->order_id