Как правильно сделать запрос в БД wordpress?

Question

[m4f1]

Моя проблема с выводом движется к завершению...
И так, по существу:

Имеется запись в БД wordpress

Поздравляем всех моряков-ветеранов МКФ, особенно тех кто служил в Военно-Морском Флоте с праздником! Всем желаем здоровья, бодрости и успехов!

<#pic#3483#0#0#0##>

Совет РОО ветеранов МКФ

С нее нужно вытащить id картинки и сделать запрос к базе данных вордпрес, после чего в другой колонке взять адрес изображения и внести в текст.

Поиск текста и замену делаю через:

function replace_text($text) {
$line_img = 232323;
	$text = preg_replace('<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>', $line_img, $text);
return $text;
}
add_filter('the_content', 'replace_text');

Код рабочий.
Проблема у меня возникает только в запросе к БД вордпреса и поиск нужной информации. Помогите пожалуйста исправить так, чтобы работало!

В данный момент мой корявый код выглядит именно так:

/* Выборка тегов для замены, начиная с картинки */
function replace_text($text) {
	// Поиск информации на странице
	preg_match_all('<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>', $text, $matches);
	print_r($matches[1]);
	
	//$id = $matches[1]
	
	// Запрос в БД
	$query = 'select smallURL from Images where id = ' .(int)$id;
	$result_image = mysqli_query($con, $query);
	$row_image = mysqli_fetch_array($result_image);
	$line_img = $row_image;

mysqli_close($link);
// Замена картинок (ббкод на картинку из БД)
	$text = preg_replace('<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>', $line_img, $text);
// Замена ссылок на человекопонятные (ббкод на обычные ссылки-урл)
	$text = preg_replace('/<#link#(.*)#URL:(.*)#>/', ' <a href="$2">$1</a>', $text);
// Замена лишнего кода на NULL	
	$text = preg_replace('/<#pcat#(.*)#>/', '', $text);
return $text;
return $text_image;
}
add_filter('the_content', 'replace_text');

Answer 1

[Daria Motorina]

без живого вордпресса сложно отдебажить, но IDE сразу дает замечания по ряду моментов:

//$id = $matches[1]
// Запрос в БД
    $query = 'select smallURL from Images where id = ' .(int)$id;

Переменной $id тут неоткуда взяться, расскоментируйте //$id = $matches[1] .
Второе:
Переменной $con не существует в рамках области видимости этой функции, надо передать ее как аргумент функции replace_text (я так понимаю $con - это коннекшн к базе, но я не помню где и как он в вордпрессе):

function replace_text($text, $con) {
//остальной код
$result_image = mysqli_query($con, $query);

Третье:
Два return'а подряд не имеют смысла - они в одной ветке кода. Если надо вернуть и $text, и $text_image - верните массив:

//неправильно
    return $text;
    return $text_image;
//правильно:
    return ['text' => $text, 'text_image' => $text_image];

Чтобы вылавливать синтаксические ошибки - включайте отображение ошибок в php, чтобы вылавливать логические ошибки - используйте функции var_dump($var) и die().

Comments

[m4f1]

Переделал так:

function replace_text($text) {
	global $wpdb;
	
	preg_match_all('<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>', $text, $matches);
	if (empty($matches[1])) return;
	//print_r($matches[1]);
	$id = $matches[1];
	$line_img = $result = $wpdb->get_results('select smallURL from Images where id = ' .(int) $id);
	var_dump($var);

// Замена картинок (ббкод на картинку из БД)
	$text = preg_replace('<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>', '<img scr="'.$line_img.'">', $text);
// Замена ссылок на человекопонятные (ббкод на обычные ссылки-урл)
	$text = preg_replace('/<#link#(.*)#URL:(.*)#>/', ' <a href="$2">$1</a>', $text);
// Замена лишнего кода на NULL	
	$text = preg_replace('/<#pcat#(.*)#>/', '', $text);
return ['text' => $text, 'text_image' => $text_image];
mysqli_close($link);
}
add_filter('the_content', 'replace_text');

Выдает "NULL"

[Daria Motorina]

m4f1 mysqli_close($link); после return не нужен)
в контексте add_filter я неправильно сказала, что нужно возвращать, получается надо вернуть текст после всех переработок, т.е. надо написать вконце
return $text; .
var_dump($var) - имелось в виду не скопипастить, а подставлять в var_dump переменную, чтобы убедиться, что она возвращает то что надо. $line_image возвращает урл картинки?

[Daria Motorina]

TNPTSYWWCC, конечно лучше, но когда "здесь и сейчас" и как я полагаю - без IDE, то не грех)

[m4f1]

Daria Motorina, возвращает нулл, хотя должен давать
images/data/gallery/0_big_1501594830_5999.jpg
остальное исправил

[Daria Motorina]

m4f1, тогда варианты такие:
- это не тот id (или $id невалиден)
- надо переписывать запрос (вопрос - как)
Через phpmyadmin есть возможность убедиться, что этот урл там существует? Если нет, то существует adminer - это скрипт как локальная замена пхпмайадмину, главное его потом удалить после использования.

[m4f1]

Daria Motorina, выше написал что он должен был выдать, id есть и поле smallURL так же заполнено

[m4f1]

Daria Motorina, поставил $matches[0]
в таком случае мне выдает

print_r($matches[0]); - Array ( [0] => #pic#3483#0#0#0## )
var_dump($line_image); - NULL

[Daria Motorina]

m4f1, $matches[0] это первое вхождение регулярки, это не id. Если $matches[1] не дает id в чистом виде, до просто вытяните его костылями до состояния, чтобы id было просто числом без массивов, пробелов и прочего и подставляйте в запрос, потому что на этом этапе id тупо не работает в строке запроса

[m4f1]

Daria Motorina, при var_dump($id);
пишет:
array(1) { [0]=> string(4) "3483" }

[Daria Motorina]

m4f1, $id = $id[0], тогда будет int

[m4f1]

Daria Motorina, можно подробнее?

[Daria Motorina]

m4f1, переменная id (которая из matches[1]) имеет тип массива, у которой элемент по нулемому индексу - это то значение id, которое вы никак не можете получить. Если перезаписать $id = $id[0], то id можно будет подставить в запрос. Подучите php

[m4f1]

Daria Motorina, Сделал так

/* Выборка тегов для замены, начиная с картинки */
function replace_text($text) {
	global $wpdb;
	
	preg_match_all('/<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>/', $text, $matches);
	if (empty($matches[1])) return;
	//print_r($matches[1]);
	$id = $id[0] = $matches[1];
	$line_img = $result;
	$result = $wpdb->get_results('SELECT * FROM `Images` WHERE `id`='.(int)$id[0]);
	//var_dump($line_image);
	var_dump($id[0]);
	
// Замена картинок (ббкод на картинку из БД)
	$text = preg_replace('/<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>/', '<img scr="'.$line_img.'">', $text);
// Замена ссылок на человекопонятные (ббкод на обычные ссылки-урл)
	$text = preg_replace('/<#link#(.*)#URL:(.*)#>/', ' <a href="$2">$1</a>', $text);
// Замена лишнего кода на NULL	
	$text = preg_replace('/<#pcat#(.*)#>/', '', $text);
return $text;
}
add_filter('the_content', 'replace_text');

Выдает:
string(4) "3483"
И вместо картинки:
<p><img scr=""></p>

[m4f1]

Daria Motorina, Если вывести дамп $result, то будет так:

array(1) { [0]=> object(stdClass)#9296 (6) { ["id"]=> string(4) "3483" ["Name"]=> string(19) "С днем ВМФ!" ["smallURL"]=> string(45) "images/data/gallery/0_big_1501594830_5999.jpg" ["Description"]=> string(0) "" ["bigURL"]=> string(45) "images/data/gallery/0_big_1501594830_5999.jpg" ["Keywords"]=> NULL } }

[Daria Motorina]

m4f1, все эти вопросы решаемы пошаговой отладкой и гуглом в течении одних суток, посидите над этим сами

[m4f1]

TNPTSYWWCC, Есть, но как ее вытащить оттуда? С этим у меня проблемы, сижу с 10 утра, но так и не могу понять как вывести

[Daria Motorina]

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

[m4f1]

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

[m4f1]

TNPTSYWWCC,

/* Выборка тегов для замены, начиная с картинки */
function replace_text($text) {
	global $wpdb;
	
	preg_match_all('/<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>/', $text, $matches);
	if (empty($matches[1])) return;
	//print_r($matches[1]);
	$id = $matches[1];
	$line_img = $result;
	$result = $wpdb->get_results('SELECT * FROM `Images` WHERE `id`='.(int)$id[0], OBJECT_K);
	//var_dump($line_image);
	//var_dump($id[0]);
	var_dump($result);
	
// Замена картинок (ббкод на картинку из БД)
	$text = preg_replace('/<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>/', '<img scr="'.$line_img.'">', $text);
// Замена ссылок на человекопонятные (ббкод на обычные ссылки-урл)
	$text = preg_replace('/<#link#(.*)#URL:(.*)#>/', ' <a href="$2">$1</a>', $text);
// Замена лишнего кода на NULL	
	$text = preg_replace('/<#pcat#(.*)#>/', '', $text);
return $text;
}
add_filter('the_content', 'replace_text');

[m4f1]

TNPTSYWWCC, Нашет то что нужно, мне надо было не get_results, а get_var, т.к. мне надо получить определенное значение в ячейке. Но снова возник "Затык", вместо текста выводит просто пустоту.
Код сейчас выглядит так:

/* Выборка тегов для замены, начиная с картинки */
function replace_text($text) {
  global $wpdb;
  
  preg_match_all('/<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>/', $text, $matches);
  if (empty($matches[1])) return;
  //print_r($matches[1]); Array ( [0] => 3483 )
  $id = $matches[1];
  $line_img = $result = $wpdb->get_var('SELECT * FROM Images WHERE id='.(int)$id[0], 2, 0);
  //var_dump($line_image);
  //var_dump($id[0]);
  var_dump($line_img);
  
// Замена картинок (ббкод на картинку из БД)
  $text = preg_replace('/<#+[a-z]+#+(\d*)#\d*#\d*#\d*##>/', '<img scr="'.$line_img.'">', $text);
// Замена ссылок на человекопонятные (ббкод на обычные ссылки-урл)
  $text = preg_replace('/<#link#(.*)#URL:(.*)#>/', ' <a href="$2">$1</a>', $text);
// Замена лишнего кода на NULL	
  $text = preg_replace('/<#pcat#(.*)#>/', '', $text);
return $text;
}
add_filter('the_content', 'replace_text');

[m4f1]

TNPTSYWWCC, Если закомментировать //if (empty($matches[1])) return;, то выводится текст статьи, но картинки вообще перестали работать