Wordpress: как избавится от PHP Notice?

Question

[zeaovede]

В functions.php добавил функцию

/*Для вывода дата регистрации юзера в мой аккаунт */
function getUserRegDate($id = 0){
global $wpdb;
$date = $wpdb->get_var($wpdb->prepare('SELECT user_registered FROM '.$wpdb->users.' WHERE ID="'.$id.'"'));
$date = explode(' ', $date);
$date = explode('-', $date[0]);
return $date[2].'/'.$date[1].'/'.$date[0];
}

В нужном месте осуществил вывод

<?php echo getUserRegDate(get_current_user_id()); ?>

Код рабочий, но получил PHP Notice - "Функция wpdb::prepare вызвана неправильно. Аргумент запроса метода wpdb::prepare() должен содержать описатель преобразований. Дополнительную информацию можно найти на странице «Отладка в WordPress». (Это сообщение было добавлено в версии 3.9.0.) in /home/e/evgsam/test.calc/public_html/wp-includes/functions.php on line 6031".

Как избавится от PHP Notice?

Answer 1

[Виктор]

prepare - это "подготовить", а "подготавливают" некий шаблон, в который потом пробрасывают значения. "Подготавливать" без значений некорректно.

- $wpdb->get_var($wpdb->prepare('SELECT user_registered FROM '.$wpdb->users.' WHERE ID="'.$id.'"'));
+ $wpdb->get_var('SELECT user_registered FROM '.$wpdb->users.' WHERE ID="'.$id.'"');

Однако, выглядит это не безопасно (как минимум, $id не типизирован). По-этому лучше пробросить его в prepare

$wpdb->get_var(
    $wpdb->prepare("SELECT user_registered FROM $wpdb->users WHERE ID=%d", $id)
);

Comments

[zeaovede]

Попробовал применить данное решение

/*Для вывода дата регистрации юзера в мой аккаунт */
function getUserRegDate($id = 0){
global $wpdb;
$wpdb->get_var(
    $wpdb->prepare("SELECT user_registered FROM $wpdb->users WHERE ID=%d", $id)
);
$date = explode(' ', $date);
$date = explode('-', $date[0]);
return $date[2].'/'.$date[1].'/'.$date[0];
}

Получил следующую ошибку:

Notice: Undefined variable: date in /home/e/evgsam/test.calc/public_html/wp-content/themes/eloboost-child/functions.php on line 2533

Notice: Undefined offset: 2 in /home/e/evgsam/test.calc/public_html/wp-content/themes/eloboost-child/functions.php on line 2535

Notice: Undefined offset: 1 in /home/e/evgsam/test.calc/public_html/wp-content/themes/eloboost-child/functions.php on line 2535

С такой же ошибкой работает и первый предложенный вариант:

$wpdb->get_var('SELECT user_registered FROM '.$wpdb->users.' WHERE ID="'.$id.'"');

[Виктор]

zeaovede, похоже много недочетов в проекте, можете отключить ошибки notice в целом, или исправлять по одной.

[maksam07]

zeaovede, вы же хоть немного смотрите, что вам предлагают. Как вы еще полностью весь свой код не заменили предложенной строкой)

[zeaovede]

Виктор, notice я специально таки и включил, для идентификации ошибок. С вашим решением код перестал работать. Я так понимаю, дело не в проекте, а в том что теперь отсутствует необходимая связь между тем что записано в функцию и тем, что выводится в нужном месте:

<?php echo getUserRegDate(get_current_user_id()); ?>

Это логично, как минимум

[zeaovede]

maksam07, увидел осечку. Спасибо за наводку. Завтра протестирую

[zeaovede]

maksam07, в общем записать нужно было вот так

/*Для вывода дата регистрации юзера в мой аккаунт */
function getUserRegDate($id = 0){
global $wpdb;
$date = $wpdb->get_var(
    $wpdb->prepare("SELECT user_registered FROM $wpdb->users WHERE ID=%d", $id)
);
$date = explode(' ', $date);
$date = explode('-', $date[0]);
return $date[2].'/'.$date[1].'/'.$date[0];
}

[Виктор]

zeaovede, а теперь посмотрите на вопрос, который вы создали

Wordpress: как избавится от PHP Notice?

[zeaovede]

Виктор, я неправильно записал код с первого раза. Немного не доглядел был. Сейчас все четко. Спасибо

Answer 2

[yarovikov]

Я бы рекомендовал избегать лишний раз кастомных запросов в базу

$udata = get_userdata( $user->ID );
$registered = $udata->user_registered;

Comments

[zeaovede]

Если это выводить на фронтенд сайта, будет работать? Если да, то как правильно записать.

[yarovikov]

function getUserRegDate() {
$udata = get_userdata(get_current_user_id());
return $udata->user_registered; //  возможно формат даты под ваши требования надо поменять
}

echo getUserRegDate();

[zeaovede]

yarovikov, проверил, работает. Хотелось бы в человеческом формате вывести
Пример:
"31 марта, 22:55
месяц назад"

[yarovikov]

zeaovede, это уже для самостоятельного изучения. гуглится за 10 секунд

[zeaovede]

yarovikov, мне удалось вывести в формате 23/03/2024. Что касается моего примера выше, автор сильно слукавил когда написал, что это гуглится за 10 секунд. В действительности это и близко не так, потому что готовых решений, как минимум нет. Поклацав туда сюда, мне удалось достичь такого результата "May 2024". Дальше самое сложное, локализация. Есть пример в интернете с выводом локализованной текущей даты, т.е. сегодня, здесь, сейчас. А у меня другая задача. Соединить два решения в одно не получилось, для того чтобы вывести дату регистрации на русском. Поэтому как-то так. В целом решение можно считать ответом

[yarovikov]

zeaovede, нагуглил за 10с) https://stackoverflow.com/questions/1416697/conver...

[zeaovede]

yarovikov, про месяца ни слова. Но даже взяв вот это и сделав перевод 12 месяцев в ручном режиме, то как потом это соединить с тем что есть у меня сейчас. Для меня это темный лес

[yarovikov]

zeaovede, все там есть, я по этому примеру делал форматирование даты типа "1 год 3 месяца 6 дней назад", но для этого копировать-вставить недостаточно, придется разобраться. А если нужно готовое решение - то для этого существует фриланс.

[zeaovede]

yarovikov, сложное решение в общем. Готового на вордпрессе под задачу ничего нет, как понял. Нужно миксовать одно с другим чтобы получить вкусное блюдо

[zeaovede]

yarovikov, спасибо за ссылку, в целом все не так уж и сложно оказалось. На решение вопроса ушло минут 5-7, именно от начала и до того момента, когда я смог применить решение на практике при условии отсутствия элементарных знаний в php( путем наглядного примера получилось ). Т. е. для человека который понимает в php хотя бы базовые понятия, это равносильно тому, что семечку из скорлупы достать.

[yarovikov]

zeaovede, да, главное не спешить и с кофеечком вдумчиво разобраться) это мой подход всегда