Как исправить циклическое перенаправление WordPress?

Question

[Natasha000]

Нужно скрыть сайт для не авторизованных пользователей (то есть сделать пере направление их на страницу входа) , делаю вот так

function notlogin_redirect(){
    auth_redirect();
}
add_action( 'init', 'notlogin_redirect' );

или так

function notlogin_redirect(){
    if ( !(is_user_logged_in())  ){

        wp_safe_redirect( 'https://www.exampl.com/mysite/my-account/' );
        die;
    }
}
add_action( 'init', 'notlogin_redirect' );

выдает: циклическое перенаправление на странице
мой .htaccess в него изменения не вносила (это прописал сам WordPress )

# BEGIN WordPress
# Директивы (строки) между `BEGIN WordPress` и `END WordPress`
# созданы автоматически и подлежат изменению только через фильтры WordPress.
# Сделанные вручную изменения между этими маркерами будут перезаписаны.
<IfModule mod_rewrite.c>
RewriteEngine On
RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]
RewriteBase /research/
RewriteRule ^index\.php$ - [L]
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule . /research/index.php [L]
</IfModule>

# END WordPress

в Настройки>>>Общие
поля Адрес WordPress (URL) и Адрес сайта (URL)
совпадают и прописаны с https

проверка вот этим ssl чекером https://sectigostore.com/ssl-tools/ssl-checker.php
пишет все норм.
Без кода на redirect - сайт работает нормально (все станицы открываются)

Answer 1

[Владимир Брумер]

как - то странно Вы это делаете!.

например так (100% рабочий вариант):



з.ы. Зачем столько тегов?))

Comments

[Владимир Брумер]

VasyaID,
Идиотизм и не уважение, когда вопрос задают и код выкладывают скриншотом. Тут я бы согласился. В данном случае Ваш комментарий как затычка ненужная. Тем более кода не простыня, а менее строчки. Ибо у автора вопроса в блокноте уже есть условие. Правда условие зачем-то в дополнительных скобках. От части именно поэтому и скриншот. Чтобы автор, возможно, заметил разницу между записаным на скриншоте и у него, перепечатывая. Не думаю, что напечатать функцию header и подставить туда свою ссылку будет проблемой для кого-то...

З.ы. спасибо за коммент, рассмешили.

[Natasha000]

Владимир Брумер, код работает только с wp-login (wp-admin), если прописать ссылку
вот так

if (!defined('ABSPATH')) exit;
if (!is_user_logged_in()) {
header("Location:" . "https://www.exampl.com/mysite/my-account/");
}

то происходит циклическое перенаправление на странице...
Просто мне нужно направить пользователя на страницу мой акккаунт (она на сайте), а не за ее пределы (wp-login это стандарт вордпресаа без шапки и т.д. ). К томуже хотелось добавить через плагин (по этому использовала хук), а не редактировать тему или ее чаилд.
Также мой код работает только wp-login (wp-admin) с другим хуком

function notlogin_redirect(){
    wp_redirect('https://www.exampl.com/mysite/wp-login.php',);
    exit;
}
add_action('template_redirect','notlogin_redirect');

Владимир Брумер, VasyaID, помогите пожалуйста разобраться (возможно проблема в хуке ?)

[Владимир Брумер]

Natasha000,

возможно проблема в хуке

проблема не в акшине(событии), а в том, что Вы не проверяете текущую страницу. У Вас в проверке только проверяется авторизован пользователь или нет. Кроме проверки авторизации необходимо исключить саму страницу авторизации из редиректа.

Просто мне нужно направить пользователя на страницу мой акккаунт (она на сайте)

Вот с таких деталей и надо начинать вопрос.

Пример

add_action( 'template_redirect', 'my_super_redirect' );
if ( ! function_exists( 'my_super_redirect' ) ) {
	function my_super_redirect() {
		$page =get_post();
		if ( empty($page) || ( 'Карта сайта' != $page->post_title && !is_user_logged_in() ) ){
			// wp_redirect( esc_url(wp_login_url()) );
			wp_redirect( 'https://www.exampl.com/карта-сайта' );
			exit();
		}
}}

можно так (тогда, например, 404 страница будет видна):

if ( ! empty($page) ){
			$title	= $page->post_title;
			if ( $title !='Карта сайта' && !is_user_logged_in()) {
				// wp_redirect( esc_url(wp_login_url()) );
				wp_redirect( 'https://www.exampl.com/карта-сайта' );
				exit();
			}
		}

[Владимир Брумер]

VasyaID, типичный представитель молодого поколения, который думает, что все ему должны. только бы с копипастить, не думая, не напрягая мозги... не знающий того, что при наборе текста, либо написании информация воспринимается и запоминается лучше. А лишние телодвижения могут иметь самый что ни на есть смысл, но как Вы заметили .... Вам этого не понять.

З.ы. А кто Вы делать мне замечания!? (вопрос риторический. я знаю на него ответ)

[Владимир Брумер]

Natasha000,
я написал пример проверки через post_title.
Не забывайте, что Вы можете проверить через идентификатор, либо ещё через что-то... смотрите, что возвращает функция get_post() и выбирайте, что Вам по душе...

[Natasha000]

Владимир Брумер, Код работает прекрасно , спасибо большое !
Я немного изменила - вынесла условие проверки , что бы лишний раз не вызывать функцию get_post() - так я думаю будет быстрее работать для авторизованных пользователей и использовала post_name для определения (так будет более универсально)
вот полный код

add_action( 'template_redirect', 'my_account_redirect' );
if ( ! function_exists( 'my_account_redirect' ) ) {
    function my_account_redirect() {
        if (!is_user_logged_in()) {
            $page =get_post();
            if ( empty($page) || ( 'my-account' != $page->post_name ) ){
                // wp_redirect( esc_url(wp_login_url()) );
                wp_redirect( 'https://www.exampl.com/mysite/my-account/' );
                exit();
            }
        }

    }}

еще раз спасибо за помощь

Answer 2

[Артем Золин]

Бесконечный редирект вы получаете при попадании на страницу my-account, нужно добавить ее в условие, чтобы попадая на нее редирект не происходил

function notlogin_redirect() {
	if ( ! is_user_logged_in() && ! is_page( 'my-account' ) ) {
		wp_safe_redirect( 'https://www.exampl.com/mysite/my-account/' );
		die;
	}
}
add_action( 'init', 'notlogin_redirect' );