Как правильно открыть кастомный php файл через AJAX в Wordpress?

Question

[Artego]

Всем привет!
Существует задача открыть краткий просмотр товара через fancybox.
Насколько удалось понять, существует 2 основных способа решения:

1) Создаем файл quick_view.php и в его начала подключаем wp-load.php. Далее через fancybox открываем файл, передавая требуемые параметры (вроде id товара). Но многие пользователи и статьи в интернете утверждают, что это максимально вредный способ, не приводя аргументов.

2) Используем functions.php. Внутри файла создаем конструкцию

add_action('wp_ajax_card_quick_view', 'card_quick_view');
add_action('wp_ajax_nopriv_card_quick_view', 'card_quick_view');

function card_quick_view() {
//код шаблона
}

и далее вызываем её через ajax в js файле. В данном способе мне не нравится момент, что придется захломлять файл functions.php объемным кодом шаблона.

Оба способа рабочие, но лично мне больше нравится первый из-за возможности сохранить некую иерархию и отдельного расположения файлов.

Вопрос: почему вариант номер 1 это плохая практика?

Comments

[Дмитрий]

Оба способа одинаково плохи. И оба способа имеют право на применение в некоторых случаях. Если вы знаете, зачем именно вам нужен именно такой способ, то проблемы нет.

В 90% случаев 1й способ применять нельзя. Аргументировать в тысячный раз почему нельзя мне лень. Читайте документацию, там все есть.

В вашем случае (если показывать инфу нужно с страницы этого же товара), я бы вообще отказался от аякса. Все такие запросы не кэшируются, зачем лишняя нагрузка?
Можно эту инфу предварительно выводить в скрытый див на этой же странице, показывать в попапе после клика уже готовое. Или, что лучше, выводить эти данные как значения js переменной, и по клику рендерить попап на основе этих данных.

Чтоб ответ был более полным - вместо запроса к admin-ajax.php можно использовать rest api, получать данные по товару оттуда.

Ну и "придется захломлять файл functions.php объемным кодом шаблона"
а что мешает вынести шаблон в отдельный файл и заинклудить его в functions.php?

[Artego]

Дмитрий, спасибо за ответ!

В том-то и дело, что все, кто называют это плохой практикой, не аргументируют =)
А хотелось бы разобраться) Может сохранилась ссылка, где пишут, почему это плохо?

Без AJAX не хотелось бы, так как в быстром просмотре выводится слайдер с картинками и много другой инфы. Если код каждой карточки встраивать в скрытый div, будет большой вес у страницы.

Про include не подумал, попробую)

Answer 1

[Надим]

Задачу можно решить не привлекая php, на чистом JacaScript. Аяксом качаем страницу с карточкой товара, парсим инфу и выводим в нужном нам виде в модельном окне. Задача решена.

Comments

[Artego]

Можно и так, но чем этот способ лучше вышеназванных?

[Надим]

Artego, зачем создавать лишний php обработчик, если можно обойтись и без него?

[Artego]

Ну это как вариант решения)
Конкретно мне быстрее вставить нужные переменные в готовый html код, чем парсить страницу.
Нагрузка на сервер в моем и вашем варианте как понимаю, не сильно будет различаться. Или в чем-то ошибаюсь?