Поправьте PHP Код пожалуйста

[vave]

Вот код на закрытие доступа от незарегистрированных пользователей

if( !$user->user_exists && substr($page, 0, 5) == "user_" ) 
{ 
  header("Location: login.php?return_url=".$url->url_current()); 
  exit(); 
}

но он предназначен только на файлы с началом USER_
например user_home

как можно сделать что бы этот код работал на абсолютно все файлы, не только с началом "user_"

ПОЖАЛУЙСТА, ЭТО ОЧЕНЬ ОЧЕНЬ СРОЧНО НУЖНО!!!!!

 

[ShoxMaster]

Вот код на закрытие доступа от незарегистрированных пользователей

if( !$user->user_exists && substr($page, 0, 5) == "user_" ) 
{ 
  header("Location: login.php?return_url=".$url->url_current()); 
  exit(); 
}

но он предназначен только на файлы с началом USER_
например user_home
как можно сделать что бы этот код работал на абсолютно все файлы, не только с началом "user_"
ПОЖАЛУЙСТА, ЭТО ОЧЕНЬ ОЧЕНЬ СРОЧНО НУЖНО!!!!!

А если убрать второе условие? Тоесть:

if( !$user->user_exists) 
{ 
  header("Location: login.php?return_url=".$url->url_current()); 
  exit(); 
}

 

[vave]

почти но тогда он определяет зарегистрированного пользователя и перенаправляет его, а не зарегистрированного оставляет на месте, а нужно что бы он незарегистрированного пользователя перенаправлял на страницу регистрации, но я сейчас уже придумал как сделать, буду пробовать)

 

[pion]

почти но тогда он определяет зарегистрированного пользователя и перенаправляет его, а не зарегистрированного оставляет на месте, а нужно что бы он незарегистрированного пользователя перенаправлял на страницу регистрации, но я сейчас уже придумал как сделать, буду пробовать)

так добавь в условие редикект.
если зареганый if, else для незарегенного

 

[pion]

скажи хоть как сам сделал, так общего развития так сказать

 

[darkway666]

Объясни в своём коде:

if( !$user->user_exists && substr($page, 0, 5) == "user_" )  
{  
  header("Location: login.php?return_url=".$url->url_current());  
  exit();  
}

вот этот небольшой кусочек, что он делает и что именно лежит в $page:

substr($page, 0, 5)

мне кажется эта строка подозрительной

 

[gres_18]

substr($page, 0, 5)

мне кажется эта строка подозрительной

Почему бы это...

он предназначен только на файлы с началом USER_
например user_home
как можно сделать что бы этот код работал на абсолютно все файлы, не только с началом "user_"

В переменной $page, по всей видимости, хранится имя файла. А код проверяет начинается ли имя файла с 'user_', и всего делов-то

 

[darkway666]

а простым перечислением делу не поможешь?
или там папок вагон?

 

[artxaker]

согласен с gres_18.

substr($page, 0, 5)

Как видно $page имеет ссылки и это может послужить к php inject.
постарайся фильтрировать эти данные,или возможно использовать произвольные запросы со стороны ламера.