Почему у меня в куке устанавливаются непонятные цифры?

Question

[Genri_Rus]

Устанавливаю куку так:

add_action( 'init', 'my_setcookie_example' );
function my_setcookie_example() {
	setcookie( "wp_id", time()+3600 );
}

В итоге выводятся вот такие цифры:


Откуда они вообще берутся ? И как перезаписать куку ?

add_action('wp_ajax_id_add', 'add_to_id');
add_action('wp_ajax_nopriv_id_add', 'add_to_id');

function add_to_id() {
	setcookie( "wp_id", time()+3600 );
	$arr_id = [];

	if ( !empty( $_POST['variation_id'] ) ) {
		$id = absint( esc_attr( $_POST['variation_id'] ) );
	} else {
		$id = absint( esc_attr( $_POST['product_id'] ) );
	}
	
	if ( $arr_id  == explode( ',', $_COOKIE["wp_id"] ) ) {
		return;
	}

	array_push($arr_id, $id);
	
	$_COOKIE["wp_id"] = implode( ",", $arr_id );
	
	wp_die();
}

Суть в том, что я кликаю по кнопке и считываю id поста, далее мне нужно как-то записать этот id в куку, но ничего не перезаписывается

Comments

[Дмитрий]

а что делает функция time()?

[Genri_Rus]

Дмитрий, вы про time в функции add_to_id ?

[approximate solution]

Дмитрий, видимо человеку лень почитать доку, даже я из старости помню что time ставило версию кэша. Посмотрим не подведёт ли память на старости лет.

[Дмитрий]

Genri_Rus, я вот про эту строчку
setcookie( "wp_id", time()+3600 );

[Genri_Rus]

Дмитрий, я ошибся:

add_action( 'init', 'my_setcookie_example' );
function my_setcookie_example() {
  setcookie( "wp_id", "", time()+3600 );
}

Но тогда мне пишет вот так:

[Дмитрий]

Genri_Rus, окей, пойдем дальше
зачем нужна строчка $_COOKIE["wp_id"] = implode( ",", $arr_id );?
почему нельзя посчитать нужное число и сразу же задать куку с полученным значением?

[Genri_Rus]

Дмитрий, как лучше ?

Мне нужно хранить id в таком виде 40,52.90

Но у меня ничего не перезаписывается, как был deleted, так и осталось

[Дмитрий]

function add_to_id() {
if ( ! isset( $_POST['variation_id'] ) || empty( $_POST['variation_id'] ) ) {
wp_send_json_error();
}
$value = intval( $_POST['variation_id'] );
if ( isset( $_COOKIE['wp_id'] ) {
$value = implode( ',', array_push( explode( ',', $_COOKIE['wp_id'] ), $value ) );
}
setcookie( 'wp_id', $value, time() + 3600 );
wp_send_json_success();
}

[Дмитрий]

и рекомендую обязательно посмотреть на остальные параметры функции
https://www.php.net/manual/ru/function.setcookie.php
лучше разобраться и указать все

[Genri_Rus]

Дмитрий, спасибо, но есть одна проблема

После обновления у меня в куке опять ничего нету:

add_action( 'init', 'my_setcookie_example' );
function my_setcookie_example() {
  setcookie( "wp_id", "", time()+3600 );
}

Т.е. я запускаю куку по init, а потом при клике считываю id и аяксом запускаю функцию add_to_id
Id в куку сохраняется, но после обновления кука снова пустая

[Дмитрий]

Genri_Rus, вот это
add_action( 'init', 'my_setcookie_example' );
function my_setcookie_example() {
setcookie( "wp_id", "", time()+3600 );
}
вообще не нужно.
Плюньте в лицо тому, кто вам сказал, что без этого работать не будет

[Genri_Rus]

Дмитрий, есть очередная проблема

Если я уберу:

add_action( 'init', 'my_setcookie_example' );
function my_setcookie_example() {
setcookie( "wp_id", "", time()+3600 );
}

То после обновления у меня куки вообще не будет

[Дмитрий]

будет.
повторюсь
"рекомендую обязательно посмотреть на остальные параметры функции
https://www.php.net/manual/ru/function.setcookie.php
лучше разобраться и указать все"

нужно еще указать домен и путь

[Genri_Rus]

Дмитрий, правильно ?

setcookie( 'wp_id', $value, time() + 3600, COOKIEPATH, COOKIE_DOMAIN );

Насколько я знаю, то это вордпрессовские константы

[Genri_Rus]

Дмитрий, вообщем у меня почему-то все время перезаписывается кука

Допустим в куке был id 41, зашел на другую страницу и кликнул для добавления нового id, в итоге 41 перезаписывается на 47

А мне нужно как-то сделать, чтобы к 41 добавился 47, так: 41,47

[Дмитрий]

Genri_Rus,
ну значит я где-то в этой строчке опечатался
$value = implode( ',', array_push( explode( ',', $_COOKIE['wp_id'] ), $value ) );
набирал прям тут, без подсветки, мог ошибиться или напутать с скобками
логика должна быть такая:
explode строку из куки в массив
array_push добавляем к массиву новое число
implode склеиваем обратно в строку

если не нужно проверять на дубликаты, можно вообще тупо склеивать к старому значению
$value = $_COOKIE['wp_id'] . ',' . $value;

[Genri_Rus]

Дмитрий, ваш вариант вообще не сохраняет куку

Мой вариант ее сохраняет, но перезаписывает:

add_action('wp_ajax_id_add', 'add_to_id');
add_action('wp_ajax_nopriv_id_add', 'add_to_id');

function add_to_id() {
  $arr_id = [];

  if ( !empty( $_POST['variation_id'] ) ) {
    $id = absint( esc_attr( $_POST['variation_id'] ) );
  } else {
    $id = absint( esc_attr( $_POST['product_id'] ) );
  }
  
  if ( $arr_id  == explode( ',', $_COOKIE["wp_id"] ) ) {
    return;
  }

  array_push($arr_id, $id);
  
  setcookie( "wp_id", implode( ",", $arr_id ), time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
  
  wp_die();
}

[Дмитрий]

Genri_Rus, а, ну да
я там зачем-то запихал array_push внутрь и упустил product_id, оставил только variation_id

тогда так

function add_to_id() {
if ( isset( $_POST['variation_id'] ) && ! empty( $_POST['variation_id'] ) ) {
$value = intval( $_POST['variation_id'] );
} else if ( isset( $_POST['product_id'] ) && ! empty( $_POST['product_id'] ) ) {
$value = intval( $_POST['product_id'] );
} else {
wp_send_json_error();
}

if ( isset( $_COOKIE['wp_id'] ) {
$data = explode( ',', $_COOKIE['wp_id'] );
array_push( $data, $value );
$value = implode( ',', $data );
}
setcookie( 'wp_id', $value, time() + 3600 );
wp_send_json_success();
}

[Дмитрий]

почему код выше плох
$arr_id = []; - лучше писать $arr_id = array();
это, впрочем, дело стиля

$id = absint( esc_attr( $_POST['variation_id'] ) );
esc_attr - лишнее
absint - я бы заменил на intval, на случай, если позже нужны будут отрицательные числа

if ( $arr_id == explode( ',', $_COOKIE["wp_id"] ) ) {
тут никогда не будет true
и в $arr_id старые значения никак не попадают

wp_die() лучше заменить на exit или wp_send_json_success()
wp_die() - выводит отформатированное как html сообщение, оно тут нафик не нужно.
если результаты выполнения скрипта нужно получить из ajax запроса, лучше вернуть это как json через wp_send_json_success

[Genri_Rus]

Дмитрий, спасибо

Я так понял заработало ?

[Genri_Rus]

Дмитрий, а как теперь дубликаты запретить добавлять ?

Я правильно понял, что можно так ?

if ( isset( $_COOKIE['wp_id'] ) ) {
		$data = explode( ',', $_COOKIE['wp_id'] );
		array_push( $data, $value );
		if ($data == explode( ',', $_COOKIE['wp_id'] )) {
			return;
		}
		$value = implode( ',', $data );
	}

[Genri_Rus]

Дмитрий, спасибо, а как тогда посоветуете лучше всего убрать дубликаты ?

[Genri_Rus]

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

function add_to_id() {
	if ( isset( $_POST['variation_id'] ) && ! empty( $_POST['variation_id'] ) ) {
		$value = intval( $_POST['variation_id'] );
	} else if ( isset( $_POST['product_id'] ) && ! empty( $_POST['product_id'] ) ) {
		$value = intval( $_POST['product_id'] );
	} else {
		wp_send_json_error();
	}

	if ( isset( $_COOKIE['wp_id'] ) ) {
		$data = [];
		$data = explode( ',', $_COOKIE['wp_id'] );
		
		wp_send_json_success( count( $data ) );

		if ( in_array( $value, $data ) ) {
			return;
		}

		array_push( $data, $value );
		$value = implode( ',', $data );
	}

	setcookie( "wp_id", implode( ",", $value ), time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
}

[Дмитрий]

Genri_Rus,
wp_send_json_success|wp_send_json_error - должны быть последними функциями
тк они вызывают exit

все, что ниже их не будет выполняться

если кука уже есть, $value = implode( ',', $data ) будет выполняться дважды. Зачем?

чтоб оставить только уникальные значения в массиве, есть функция array_unique()
https://www.php.net/manual/ru/function.array-unique.php

рекомендую написать алгоритм вначале на бумаге тупо словами. например так:
проверяем есть ли $_POST['variation_id'] и оно не пустое
если да -> записываем это значение в переменную $value
и тп

а потом переписать это уже на нужном языке

сейчас впечатление, что вы половину кода добавили просто чтоб оно было, а зачем и что оно делает, не понимаете

[Genri_Rus]

Дмитрий, у меня изначально была немного другая задумка )

Мне нужно при клике добавить id, а при повторном клике - удалить id (делаю избранные товары)

Вот тут:

if ( in_array( $value, $data ) ) {
      return;
}

Как мне из куки удалить id, который второй раз приходит ?

[Genri_Rus]

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

Мне нужно возвращать либо 0, либо 1

Т.е. человек кликнул - 1, при повторном клике - 0, функция count
Сделал так:

function add_to_id() {
	if ( isset( $_POST['variation_id'] ) && ! empty( $_POST['variation_id'] ) ) {
		$value = intval( $_POST['variation_id'] );
	} else if ( isset( $_POST['product_id'] ) && ! empty( $_POST['product_id'] ) ) {
		$value = intval( $_POST['product_id'] );
	} else {
		wp_send_json_error();
	}

	if ( isset( $_COOKIE['wp_id'] ) ) {
		$data = [];
		$data = explode( ',', $_COOKIE['wp_id'] );

		if ( in_array( $value, $data ) ) {
			unset( $data[ array_search( $value, $data ) ] );

			$value = implode( ',', $data );

			setcookie( "wp_id", $value, time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
			wp_send_json_success( count( $data ) );
			return;
		}

		array_push( $data, $value );
		$value = implode( ',', $data );
	}

	setcookie( "wp_id", $value, time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
	wp_send_json_success( count( $data ) );
}

[Genri_Rus]

Дмитрий, вообще я сидел и думал как лучше со счетчиком поступить, пришлось только так:

function add_to_id() {
	if ( isset( $_POST['variation_id'] ) && ! empty( $_POST['variation_id'] ) ) {
		$value = intval( $_POST['variation_id'] );
	} else if ( isset( $_POST['product_id'] ) && ! empty( $_POST['product_id'] ) ) {
		$value = intval( $_POST['product_id'] );
	} else {
		wp_send_json_error();
	}
	
	$counter = 0;

	if ( isset( $_COOKIE['wp_id'] ) ) {
		$data = [];
		$data = explode( ',', $_COOKIE['wp_id'] );

		if ( in_array( $value, $data ) ) {
			unset( $data[ array_search( $value, $data ) ] );

			$value = implode( ',', $data );

			setcookie( "wp_id", $value, time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
			wp_send_json_success( count( $data ) );
			return;
		}

		array_push( $data, $value );
		$value = implode( ',', $data );
	}

	setcookie( "wp_id", $value, time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
	
	if ( !empty( $data ) ) {
		wp_send_json_success( count( $data ) );
	} else {
		wp_send_json_success( $counter + 1 );
	}
}

Проблема вот в чем, при клике мне нужно сохранить куку, далее при последующем клике - удалить куку

Также мне нужно плюсовать счетчик, допустим человек кликнул по кнопке - +1, далее перешёл на другую страницу и кликнул по кнопке - +2

Может подскажите как лучше со счётчикам реализовать ?