Почему не срабатывает условие для страниц?

Question

[Shimpanze]

Добрый день!

Данное условие, без проблем срабатывает для записей, категорий, меток и т. д. но не срабатывает для страниц - почему???

if ( !empty( $_POST ) || ( !empty( $_GET['action'] ) && $_GET['action'] == 'edit' ) || ( defined( 'XMLRPC_REQUEST' ) && XMLRPC_REQUEST ) ) {
    add_filter( ... );
}

Буду очень благодарен за помощь.

Comments

[Игорь Воротнёв]

А что это вы такое интересное пытаетесь сделать? Что это за куча проверок таких?

[Shimpanze]

Игорь Воротнёв, Фильтр, разрешающий заглавные буквы в URL

[Игорь Воротнёв]

Shimpanze, хм.. а зачем столько танцев с бубном?

Во-первых, это делается ручками достаточно легко, см тут https://wordpress.stackexchange.com/questions/5029... Код там старый, но переписать свежую версию не проблема:
1. Берете код функции ядра sanitize_title_with_dashes()
2. Копируете себе в плагин / functions.php темы под своим названием
3. Убираете из кода все что связано с strtolower / mb_strtolower
4. Отключаете из хука дефолтную функцию и подключаете свою (как в ответе на StackOverflow)
5. Профит!

Во-вторых, это не очень разумно делать. Для поисковиков регистр пофиг, для самого URL - не пофиг, ибо по спецификации он case sensitive (все, что кроме доменного имени). И для PHP / MySQL в обработке тоже все будет case sensitive. Что это означает? То что простым пользователям нафиг не нужно запоминать что у вас страницы в верхнем регистре, они будут писать в привычном нижнем. А это будет приводить к 404, так как страниц с такими названиями существовать не будет (ибо одна и та же страница со слагом в верхнем и нижнем регистре - это две разные страницы). Это, конечно, можно обойти - хукнуться куда-нибудь в самое начало обработки запроса и перезаписывать значения в $_SERVER, делая первую букву заглавной. Но это гемор еще тот будет, запутаетесь в отладке всех кейсов.

[Shimpanze]

Игорь Воротнёв, Блин, дружище, этот код не работает со страницами! В этом все и дело... Я его уже несколько лет!!! пытаюсь заставить работать со страницами

[Игорь Воротнёв]

Shimpanze, да что ты говоришь! :)

[Shimpanze]

Игорь Воротнёв, Ок, попробуй эти (я уже несколько лет мучаюсь):

site.ru/xPath
site.ru/Примеры_jQuery

[Игорь Воротнёв]

Shimpanze, слаг xPath будет работать, как и положено:



А вот Примеры_jQuery - не будет, но совершенно по другой причине. Где именно собака порылась - пока не разобрался. Но вот что интересно:



Rewrite rule страницы срабатывает, SQL запрос по такому post_name запись находит. Функция get_page_by_path(), собственно, возвращает 1 найденную строку из БД. Но Main Query почему-то ищет запись типа 'post', а не 'page'. Вот это как минимум странно..



Ковыряю, пытаюсь понять почему.

[Игорь Воротнёв]

Shimpanze, Игорь Воротнёв, so far so good:



parse_request таки корректно разбирает url и получает страницу из базы. Смотрим дальше...

[Shimpanze]

Игорь Воротнёв, Друг, я в этом не разбираюсь пока. С меня бутылка если поймешь почему.

[Игорь Воротнёв]

Shimpanze, кажется проблему нашел. Вот 2 правила rewrite rules, под которые данный урл попадает:

(.?.+?)(?:/([0-9]+))?/?$	pagename=$matches[1]&page=$matches[2]
([^/]+)(?:/([0-9]+))?/?$	name=$matches[1]&page=$matches[2]

Первое - это страницы. Второе - посты.

Так вот конкретно для этой страницы почему-то срабатывает правило для постов (и устанавливается переменная name вместо pagename).

Дальнейшая отладка привела меня в функцию get_page_by_path(), которая таки получает из базы нужную страницу, но после получения результатов идет проверка на совпадения path и $page->post_name (то есть слага), и вот тут случается false, вместо ожидаемого true:



А если конкретнее, то вот само сравнение:



Как мы видим, строки отличаются регистром, и сравнение == возвращает false:

‌‌'%D0%9F%D1%80%D0%B8%D0%BC%D0%B5%D1%80%D1%8B_jQuery' == '‌%d0%9f%d1%80%d0%b8%d0%bc%d0%b5%d1%80%d1%8b_jQuery'
‌false

Осталось копнуть и найти тот момент, где регистр становится верхним. В сам get_page_by_path() строка передается в нижнем регистре. А далее вот в этом месте строка превращается в верхний регистр.

‌‌$page_path = rawurlencode(urldecode('%d0%9f%d1%80%d0%b8%d0%bc%d0%b5%d1%80%d1%8b_jQuery'));
‌%D0%9F%D1%80%D0%B8%D0%BC%D0%B5%D1%80%D1%8B_jQuery

Далее эта строка проходит через фильтр 'sanitize_title', что, теоретически, дает нам возможность ее перехватить и обратить в нижний регистр:

/**
 * Make sure rawurlencoded part of the path is always lowercase.
 * 
 * @param string $title
 * @param string $raw_title
 * @param string $context
 *
 * @return string
 */
function lowercase_rawurlencode_path( $title, $raw_title, $context ) {
    if ( $context == 'query' && preg_match( "@^[a-zA-Z0-9%+-_]*$@", $title ) ) { // контекст должен быть query, а строка результат rawurlencode

        $title_decoded = rawurldecode( $title ); // декодируем строку обратно
        $title_parts = preg_split( "/(_|-)/", $title_decoded, -1, PREG_SPLIT_DELIM_CAPTURE ); // разбиваем на части по '_' и '-', сохраняя эти самые разделители (чтобы потом собрать обратно)

        foreach( $title_parts as $index => $part ) { // перебираем полученные части
            $part_encoded = rawurlencode( $part ); // кодируем часть
            if ( $part !== $part_encoded ) { // если она закодировалась
                $title_parts[ $index ] = strtolower( $part_encoded ); // приводим в нижний регистр
            }
        }

        return implode( '', $title_parts ); // собираем в строку и возвращаем
    }

    return $title; // в остальных случаях возвращаем без изменений
}
add_filter( 'sanitize_title', 'lowercase_rawurlencode_path', 10, 3 );

Возможно есть более элегантное решение с точки зрения PHP чем декодирование и перебор, но данное решение работает:

[Игорь Воротнёв]

Shimpanze, см. код выше, проверяй. Если все ок – отпишись, перенесу в ответ. На будущее пригодится :)

[Shimpanze]

Игорь Воротнёв, 1) При вставке этого сниппета, вообще не возможно создать страницу с заглавными URL;

2) Теперь не работает вывод всех записей из категории, допустим «http://site.ru/Рубрикатор/Кулинария»;

3) Не срабатывает автоназначение URL (с Заглавными Буквами) вот здесь:



... эта проверка:

if ( !empty( $_POST ) || ( !empty( $_GET['action'] ) && $_GET['action'] == 'edit' ) || ( defined( 'XMLRPC_REQUEST' ) && XMLRPC_REQUEST ) ) {
  add_filter( 'sanitize_title', 'f_processing_title', 9 );
}

... и нужна была, для этого.

[Игорь Воротнёв]

Shimpanze,

1) При вставке этого сниппета, вообще не возможно создать страницу с заглавными URL;

Это не отдельный сниппет, а дополнение к тому, который по ссылке на StackOverflow

2) Теперь не работает вывод всех записей из категории, допустим «http://site.ru/Рубрикатор/Кулинария»;

3) Не срабатывает автоназначение URL (с Заглавными Буквами) вот здесь:

Я вообще-то тестил. Если у вас не работает, то проблема точно в другом месте.

[Shimpanze]

Игорь Воротнёв, В общем, работает вот в такой комбинации:

/*
Разрешаем заглавные URL.

ВНИМАНИЕ! Этот кусок кода брать из актуальной версии WordPress (файл wp-includes\formatting.php).

Код из версии WordPress 4.9.1 */

function f_processing_title($title) {

  remove_filter('sanitize_title', 'sanitize_title_with_dashes');

  add_filter('sanitize_title', function($title) {


  /*
  В оригинальном коде, произвести следующие изменения:

  1. Закомментировать эту часть кода:

     if (function_exists('mb_strtolower')) {
       $title = mb_strtolower($title, 'UTF-8');
     }

  2. ...и закомментировать эту строку:

     $title = strtolower($title);

  3. ...и в этой строке разрешить заглавные буквы:

     $title = preg_replace('/[^%a-z0-9 _-]/', '', $title);

       * должно получиться вот так:

       $title = preg_replace('/[^%a-zA-Z0-9 _-/]/', '', $title);

  4. ...а пробелы, автоматически заменять на нижнее подчёркивание, а не на тире:

     $title = preg_replace('/\s+/', '_', $title); */


    $title = strip_tags($title);
    // Preserve escaped octets.
    $title = preg_replace('|%([a-fA-F0-9][a-fA-F0-9])|', '---$1---', $title);
    // Remove percent signs that are not part of an octet.
    $title = str_replace('%', '', $title);
    // Restore octets.
    $title = preg_replace('|---([a-fA-F0-9][a-fA-F0-9])---|', '%$1', $title);

    if (seems_utf8($title)) {
      //if (function_exists('mb_strtolower')) {
        //$title = mb_strtolower($title, 'UTF-8');
      //}
      $title = utf8_uri_encode($title, 200);
    }

    //$title = strtolower($title);

    if ( 'save' == $context ) {
      // Convert nbsp, ndash and mdash to hyphens
      $title = str_replace( array( '%c2%a0', '%e2%80%93', '%e2%80%94' ), '-', $title );
      // Convert nbsp, ndash and mdash HTML entities to hyphens
      $title = str_replace( array( ' ', ' ', '–', '–', '—', '—' ), '-', $title );
      // Convert forward slash to hyphen
      $title = str_replace( '/', '-', $title );

      // Strip these characters entirely
      $title = str_replace( array(
        // iexcl and iquest
        '%c2%a1', '%c2%bf',
        // angle quotes
        '%c2%ab', '%c2%bb', '%e2%80%b9', '%e2%80%ba',
        // curly quotes
        '%e2%80%98', '%e2%80%99', '%e2%80%9c', '%e2%80%9d',
        '%e2%80%9a', '%e2%80%9b', '%e2%80%9e', '%e2%80%9f',
        // copy, reg, deg, hellip and trade
        '%c2%a9', '%c2%ae', '%c2%b0', '%e2%80%a6', '%e2%84%a2',
        // acute accents
        '%c2%b4', '%cb%8a', '%cc%81', '%cd%81',
        // grave accent, macron, caron
        '%cc%80', '%cc%84', '%cc%8c',
      ), '', $title );

      // Convert times to x
      $title = str_replace( '%c3%97', 'x', $title );
    }

    $title = preg_replace('/&.+?;/', '', $title); // kill entities
    $title = str_replace('.', '-', $title);

    $title = preg_replace('/[^%a-zA-Z0-9 _-]/', '', $title);
    $title = preg_replace('/\s+/', '_', $title);
    $title = preg_replace('|-+|', '-', $title);
    $title = trim($title, '-');

    return $title;

  });

  return $title;

}

add_filter('sanitize_title', 'f_processing_title', 9);


/*
Исправление ошибки в ядре WordPress при обработке URL страниц.
Автор: Игорь Воротнёв [@HeadOnFire] */

/**
 * Make sure rawurlencoded part of the path is always lowercase.
 *
 * @param string $title
 * @param string $raw_title
 * @param string $context
 *
 * @return string
 */

function lowercase_rawurlencode_path($title, $raw_title, $context) {
  if ($context == 'query' && preg_match("@^[a-zA-Z0-9%+-_]*$@", $title)) { // контекст должен быть query, а строка результат rawurlencode

    $title_decoded = rawurldecode($title); // декодируем строку обратно
    $title_parts = preg_split("/(_|-)/", $title_decoded, -1, PREG_SPLIT_DELIM_CAPTURE); // разбиваем на части по '_' и '-', сохраняя эти самые разделители (чтобы потом собрать обратно)

    foreach($title_parts as $index => $part) { // перебираем полученные части
      $part_encoded = rawurlencode($part); // кодируем часть
      if ($part !== $part_encoded) { // если она закодировалась
        $title_parts[$index] = strtolower($part_encoded); // приводим в нижний регистр
      }
    }

    return implode('', $title_parts); // собираем в строку и возвращаем
  }

  return $title; // в остальных случаях возвращаем без изменений
}

add_filter('sanitize_title', 'lowercase_rawurlencode_path', 10, 3);

[Shimpanze]

Игорь Воротнёв, Я там первый сниппет подробно прокомментировал.
Оформите как нужно, поместите в ответ, и давайте почистим эту переписку.
Я напишу вам, на указанную у вас в контактах почту, на счет бутылки :)
Большое спасибо!