Почему ACF обновляет поле 2 раза?

Question

[Павел]

На странице поста есть кол-во просмотров. Раньше счетчик всегда делал обычным кастомным полем и скрывал его - "_views". Но сейчас клиент захотел в любой момент сам регулировать отображаемое кол-во просмотров поста. Сделал поле через ACF. Вот фрагмент кода из single.php

$views = (int) get_field('views');
        var_dump($views); //n
        $views = $views ? ++$views : 1;
        var_dump($views); //n+1
        var_dump(update_field('views', $views)); //true

var_dump как бы на каждом шаге показывает правильные значения, но при каждом обновлении страницы счетчик увеличивается на 2.

Дебагер показывает только один update запрос

Но при этом в БД записывается значение на 1 больше.

Answer 1

[WordPress WooCommerce]

Мб не стоит совать такой код в single.php. Бог знает кто этот шаблон дергает. Есть миллион возможных причин.
Вероятно проще сделать счетчик на хук template_redirect & is_singular('post'). Технически это более грамотно. И будет точно выполняться 1 раз на запрос.
Далее мб это как то связано с _views. Как видите ACF там хранит какие то свои служебные данные.

Сделайте свое поле как my_count_views. Работайте с ним.
А поле ACF при сохранении синхронизируйте со своим полем.

Это первое что в голову пришло. Либо вообще убрать нафиг ACF. Убрать скрытость поля. И пусть клиент правит это все через типовой метабокс Произвольные поля.

В крайнем случае напишите ему свой метабокс. Это 10 строк кода.

Comments

[Павел]

Только вернулся к этому вопросу. Буду пробовать сделать что-то.
Нашел в документации к ACF, по сути, идентичный пример:

<?php

// get current count value
$count = (int) get_field('views');


// increase
$count++;


// update
update_field('views', $count);
    
?>

https://www.advancedcustomfields.com/resources/upd...
И у меня теперь сомнения. А может ли быть дело в серваке? Потому как у меня на dev'е работало все норм.. отдал клиенту и он пожаловался на этот баг.

[Павел]

нет, дело не в серваке.
Еще попробовал таким образом

$views = get_post_meta($post->ID,'views_counter', true);
        $views = $views ? ++$views : 1;
        update_post_meta($post->ID, 'views_counter', $views);

Результат тот же. Плюсуется 2 раза.

[Павел]

function views_counter() {
    if (is_singular(array('post','blog'))) {
        global $post;
        $views = get_post_meta($post->ID,'views_counter', true);
        $views = $views ? ++$views : 1;
        update_post_meta($post->ID, 'views_counter', $views);
    }
}
add_action( 'template_redirect', 'views_counter' );

Результат тот же. Сильно подозреваю, что проблема глубже. Причем повторюсь, дебаггер показывает только один UPDATE запрос к БД.

[Павел]

И на другом сайте любой из этих примеров работает.. Да и раньше я счетчики делал.

[WordPress WooCommerce]

Павел: странно все это.

Я думаю есть 2 пути:
1. Искать глюк через диагностику и отключение компонентов
2. Переписать механику апдейта на прямой атомарный SQL запрос

Те методы что вы используете - это EAV абстракция над SQL, плюс там есть прослойка из EDA. Обе эти вещи хороши в 99% случаев при умелом обращении, но могут нефигово так взлахматить мозг новичкам или в 1% неких особых задач. Потому в этих 1% случаях бывает уместно и обосновано опуститься до SQL и обойти все эти абстракции стороной.

[Павел]

Макс Ханник: Предлагаете через $wpdb запрос написать? надо попробовать...

[Павел]

Макс Ханник:
Короче, я сдаюсь)

$views_db = $wpdb->get_var("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key = 'views_counter' AND post_id = $post->ID");
    $views_db++;
    $wpdb->query("UPDATE $wpdb->postmeta SET meta_value = $views_db WHERE post_id = $post->ID AND meta_key = 'views_counter'");

Дабы я не выглядел идиотом, вот видео https://www.youtube.com/watch?v=UHxvVAdlNCo