Как подключить стили в WordPress с GET запросом?

Question

[Роман Говоров]

Написал такой код, для сжатия CSS:

$file_css = [
        'default',
        'content',
        'modules',
        'woocommerce',
        'mobile',
        '../../../plugins/contact-form-7/includes/css/styles',
        '../plugins/owlcarousel/css/owl.carousel.min'
    ];
    $file_css = implode(',', $file_css);
    wp_enqueue_style( 'GeneralStyle', '/wp-content/themes/rstheme/compress.php?type=css&files='.$file_css.'');

Пытаюсь добавить по правилам WP вместо желаемого результата выводит только это:

<link rel='stylesheet' id='GeneralStyle-css'  href='https://site.ru/wp-content/themes/rstheme/compress.php'  media='all' />

Он как будто удаляет весь GET запрос и не дает добавить так как это мне нужно..

Если подключать не по правилам WP то все работает..

Answer 1

[Роман Говоров]

Ошибка была в коде, где я удалял версии скриптов...

add_filter( 'script_loader_src', '_remove_script_version' );
add_filter( 'style_loader_src', '_remove_script_version' );
function _remove_script_version( $src ){
    $parts = explode( '?', $src );
    return $parts[0];
}

Исправил на следующий код:

add_filter( 'script_loader_src', '_remove_script_version' );
add_filter( 'style_loader_src', '_remove_script_version' );
function _remove_script_version( $src ){
    $parts = explode( 'ver', $src );
    return $parts[0];
}

Answer 2

[Дмитрий]

1. конкатенация стилей не нужна, если у вас http/2
2. по стандартам WP будет что-то такое
wp_enqueue_script( 'script-name', get_template_directory_uri() . '/compress.php?type=css&files=' . $file_css, array() );
3. если параметры с запроса удаляются, смотрите, то за фильтры у вас висят, какие плагину установлены. вполне возможно, что это делает тема/какие-то плагины оптимизации

Comments

[Роман Говоров]

Тема мной написана, плагинов нет.
wp_enqueue_script - подключает скрипты, я же вывожу wp_enqueue_style как стили.

[Дмитрий]

Роман Вопроскин, да, не то скопипастил, нужно wp_enqueue_style
но смысл от этого не изменится

[Роман Говоров]

К сожалению ничего не помогает, хз почему после «?» WP все стирает..

[Дмитрий]

Роман Вопроскин, попробуйте обратится к разработчику этого шаблона.
уточните у него, что он там такого понаделал, что удаляется.

ну или отключите все плагины, посмотрите без них

[Роман Говоров]

Дмитрий,

Тема мной написана, плагинов нет.

[Дмитрий]

Роман Вопроскин, ну а я как без доступа к коду гадать должен?)
значит написано в теме что-то, что удаляет.

стандартно из коробки удалять не должно

[Дмитрий]

как это выяснить, я написал в пункте 3 - смотрите, какие фильтры/хуки висят, на каком этапе и что удаляет.

но, я бы не рекомендовал добавлять такое в шаблон.
судя даже по тому коду, что видно, написано не особо:
1. почему один скрипт добавляется через '../../../plugins', а другой как '../plugins'?
2. как уже писал, склеивать стили - плохо, для скорости загрузки сайта вредно. эту же проблему можно решить боле эффективно другими способами
3. если позже нужно будет добавить новые плагины (или текущие плагины после обновления поменяют название файлов/пути), нужно будет править этот код
4. может поломаться порядок загрузки стилей
5. чтоб ваш код работал, возможно, у вас убраны некоторые стандартные хуки, вроде wp_head и тп

но вполне может быть, что я не прав, и у вас там отличный код, а сюда просто для упрощения такое добавлено

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

Роман Вопроскин, потому что WordPress обрабатывает эту строку по своему и в процессе делает ей explode по символу "?". Посмотрите код метода enqueue() класса WP_Dependencies (и другие связанные методы).

А вообще, положа руку на сердце, выдавать стили динамикой и вообще поднимать для этого целый движок PHP и WordPress - крайне неэффективная мера и сомнительная практика. Вы "не туда оптимизируете", коллега :)

[Роман Говоров]

1. это разные папки на FTP (/wp-content/plugins) и (/wp-content/themes/rstheme/plugins)
2. где не читал все советуют все стили сжимать и совать в один файл.. Если есть более эффективный способ, посоветуйте, пожалуйста.
3. согласен, но по другому ни как, оптимизация...
4. порядок указываю я, ручками, так что не поломается
5. по ползаю по ищу хуки с wp_head , но явно такого чтобы удалял GET запросы я не делал

P.S.
6. закинул этот код на стандартные темы WP и все так же, стирает все после «?»

[Роман Говоров]

Игорь Воротнёв, Посоветуйте как оптимизировать стили без плагинов

[Дмитрий]

Игорь Воротнёв, не, обрабатывать то оно обрабатывает, но как у автора ломать не должно.
Иначе как бы тогда добавлялись ссылки на всякие гугл шрифты или прочие кастомные внешние ассеты с параметрами?
Ну и вот только что потестил на 2х произвольных темах - везде сохраняет и, если есть версия ассета в 4м параметре wp_enqueue_style, то и она корректно добавляется
https://www.dropbox.com/s/30tlspgc14y8s3x/Screensh...

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

Роман Вопроскин, Во-первых, стили - не самая большая проблема и никогда ими не были. Основной стиль темы в голову документа, как обычно. Второстепенные - в шапке или подвале с async или defer (тут смотрите по ситуации, что они делают). В идеале из основного можно извлечь critical css и запихнуть в html (в head), но это не так часто нужно в реальности - не дает прям сильного прироста по производительности. А вот что даст реальный мега-заметный прирост, так это использование протокола HTTP/2, настройки веб-сервера максимально быстро отдавать статику и, конечно же, адекватные заголовки кеширования для этих файлов.

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

Дмитрий,

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

Кстати да, я об этом даже не подумал. Должно же работать. Хз в таком случае, явно в коде тогда проблема. Ну. в любом случае, подобный метод сам по себе не очень ок, так что в другую сторону копать надо.

[Роман Говоров]

Дмитрий, это то да, а ты попробуй не .css а .php и тогда увидишь результат)

[Дмитрий]

Роман Вопроскин, все так же
https://www.dropbox.com/s/khibbigg9oexujp/Screensh...

[Роман Говоров]

Дмитрий, понял.. у меня бомбит, я пошел копать, спасибо за обратную связь!)