[inne][PHP] Brak jednego z $_GET[''] w skrypcie... Opcje

[sloenthran]

index.php

[PHP] pobierz, plaintext 
<?php
 
	error_reporting(E_ALL ^ E_NOTICE & ~E_DEPRECATED);
 
	ob_start();
 
		session_start();
 
		require_once('./function/ClassLoader.php');
 
		$Core = new Core();
 
		$Pages = $Core->ClearText($_GET['pages']);
		$Pages = strip_tags($Pages);
 
		$Controller = $Core->ClearText($_GET['controller']);
		$Controller = strip_tags($Controller);
 
		if(!$Pages || $Pages == '') { $Pages = 'home'; }
		if(!$Controller || $Controller = '') { $Controller = 'other'; }
 
		if(file_exists('./pages/'.$Controller.'/'.$Pages.'.php'))
		{
 
			require_once("./pages/default/header.php");
 
			require_once('./pages/'.$Controller.'/'.$Pages.'.php');
 
			require_once("./pages/default/footer.php");
 
		}
 
		else
		{
 
			header("Location: errors-404.html");
 
		}
 
	ob_end_flush();
 
?>
[PHP] pobierz, plaintext 

.htacess

Kod

RewriteEngine on

RewriteRule ^([^-]*)-([^-]*).html$ index.php?controller=$1&page=$2 [L]

Funkcja ClearText

[PHP] pobierz, plaintext 
		function ClearText($Text)
		{
 
			if(get_magic_quotes_gpc())
			{
 
				$Text = stripslashes($Text);
 
			}
 
			$Text = trim($Text);
			//$Text = mysql_real_escape_string($Text);
			$Text = htmlspecialchars($Text);
			$Text = htmlentities($Text);
 
			return $Text;
 
		}
[PHP] pobierz, plaintext 

Problem jest następujący a mianowicie czego bym nie zrobił to skrypt wcale nie widzi zmiennej controller. Jakieś pomysły?

Ten post edytował sloenhtran 10.07.2014, 07:42:12

[Sephirus]

Po pierwsze primo:

[PHP] pobierz, plaintext 
if(!$Controller || $Controller = '') { $Controller = 'other'; } // w IF masz "=" a nie "=="
[PHP] pobierz, plaintext 

Po drugie primo:

[PHP] pobierz, plaintext 
$Text = htmlspecialchars($Text);
$Text = htmlentities($Text);
[PHP] pobierz, plaintext 

Serio? Przemyśl tą funkcję ClearText - poczytaj o tym czy coś.