Как правильно экранизировать php код в js?

Question

[Emmet1]

*перезалил код целеком
код выглядит так:

$('.list_cat a').click(function(){
			cat_id =$(this).attr('class');
			$("div.pic").remove()
			data = {
				action: 'my_action',
				cat_id: cat_id,
			};

			var ajaxurl = "/wp-admin/admin-ajax.php";

			$.ajax({
				type: 'POST',
				url: ajaxurl,
				data: data,
				beforeSend:function() {
					$('#loader').show();
				},
				success: function(response){
					response = JSON.parse(response);
					console.log(response);
					response.forEach(function(item, i, arr){
						function myContent(title, link, id){
							 var container = document.createElement('div');
							 container.className = "pic pic-3d";
							   container.innerHTML = '<?php echo get_the_post_thumbnail(' + id + '); ?> <span class="pic-caption open-right"> \
							    <span class="pic-title">'+title+'</span> \
							    <p><a href="/' + link +'">Смотреть</a></p> \
							  </span>'
							  return $('.portfolio_item').append(container);
						}
						myContent(item.post_title, item.post_name, item.ID);
						$('#loader').hide();

					});
				}
			});
		});

Вся проблема в php echo get_the_post_thumbnail, оно не хочет его выводить. Как мне показалось, все из-за того что я не правильно вписал + id + или не экранировал "?". Но если его экранировать, то пхп код перестает работать. выводится закомментированным

Comments

[Lynn «Кофеман»]

phpfaq.ru/newbie/na_tanke

Answer 1

[Дмитрий Калюк]

Перестать пихать JS в фйлы с PHP

Answer 2

[KzLIFE]

Позьвольте уточнить две вещи, ID это переменная или строка ? Если переменная, то в php её нужно вывести таким образом

Функция(переменная), если это строка то функция ("строка").

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

И кстати вы не сможете в коде js постоянно использовать (вызывать) код php, т.к php грузится только один раз при загрузке скрипта. По мне проще это ajaxom решить

Comments

[Emmet1]

Это все аяксом и делается. Скинул кусок кода. А надо было целый.

суть скрипта, получение на сервере массива с обьектами, передача массива в формате json. А потом с помощью forEach перебираю его. Скину вам лучше ка скрин полного кода clip2net.com/s/3Dw8Ksp

PS я начинаючий в php, в js все еще хуже

[Emmet1]

Также залил полный js код в начало темы

[Stalker_RED]

Emmet1: Ваш js код работает в браузере. Он конечно сможет в тело страницы вывести строку <?php ... ?> но функция get_the_post_thumbnail() не сработает, т.к. она будет просто фрагментом текста в браузере, и интерпретатор php на сервере даже не будет подозревать о ее существовании.

[KzLIFE]

Emmet1: этот код JS в отдельном файле.js или на странице с php ? Если отдельно то js не знает о php, если вы явно не укажете в настройках своего сервера, это обычно в .htaccess указывают

[KzLIFE]

И зачем вы перебирая массив создаёте функцию? Надо сначала создать функцию выше ответа Ajax.

function test(){}

Ajax(each(test()))

а вы создаёте функцию в ответе и там же её вызываете. Это ни есть хорошо

[KzLIFE]

И вам не нужно делать return после append, т.к return возвращает данные , а вам и возвращать там нечего. Это плохо, вся структура js у вас плохо поставлена