AJAX update – big 'why' question

Posted in: Newspaper
Post count: 30

Hi.
\td_ajax::on_ajax_update_views static method – it’s clear when and what for it is triggered.
My question is:
why you didn’t use there
td_page_views::update_page_views($td_post_id[0]);
instead of
update_post_meta($td_post_id[0], td_page_views::$post_view_counter_key, $new_post_count);
by using td_page_views::update_page_views($td_post_id[0]); you can easily enable last 7 days counter with cache plugin.
Or I’m missing something?

Post count: 23312

Hello joeydekoning,

Sorry but please notice that was the developer’s decision in this case, sorry! If you want to change the code from this notice that you are free to do this.

Thanks for the message!

Post count: 30

Yes, I understand.
I did change the code. But I’m simply curious. Perhaps there is a some sort of a trap there?
Could you please provide the developer’s feedback?
This potentially can be an improvement for you guys.

Post count: 23312

Hello joeydekoning,

Thanks for your suggestion! We have added this to our list of improvements, our developers will check it and if they will decide to make some changes, in this case, we will consider fixing it in the future update.

Thanks for your understanding!

Post count: 30

Thumbs Up!
there are few minor adjustment to perform, but it’s already working for us, and we did not bumped any obstacles as for now.

thank you very much

Viewing 5 posts - 1 through 5 (of 5 total)
The forum ‘Newspaper’ is closed to new topics and replies.