{"id":10546874,"date":"2018-08-01T09:58:38","date_gmt":"2018-08-01T09:58:38","guid":{"rendered":"https:\/\/wordpress.org\/support\/topic\/do-shortcode-doesnt-work-without-hack\/"},"modified":"2018-08-01T10:00:37","modified_gmt":"2018-08-01T10:00:37","slug":"do-shortcode-doesnt-work-without-hack","status":"publish","type":"topic","link":"https:\/\/wordpress.org\/support\/topic\/do-shortcode-doesnt-work-without-hack\/","title":{"rendered":"do_shortcode doesn’t work without hack"},"content":{"rendered":"

Hi Aaron,<\/p>\n

We use your plugin and it usually works fine.<\/p>\n

Though on our website Sneakerwijzer.nl, which uses a custom theme, we could not get it to work properly with the standard settings. So our developer used the WP-standard do_shortcode option to show the reviews on the page.<\/p>\n

But, although it shows the reviews, the aggregate shows 0 results and a 0 in the score \ud83d\ude09<\/p>\n

This has to do with the bold part of your function below:<\/p>\n

function get_aggregate_reviews($postid) {
\n global $wpdb;<\/p>\n

$query = $wpdb->prepare(”
\n SELECT
\n COUNT(*) AS aggregate_count, AVG(tmp2.rating) AS aggregate_rating
\n FROM (
\n SELECT pm4.meta_value AS rating
\n FROM (
\n SELECT DISTINCT pm2.post_id
\n FROM {$wpdb->prefix}posts p1
\n INNER JOIN {$wpdb->prefix}postmeta pm1 ON pm1.meta_key = ‘wpcr3_enable’ AND<\/strong> pm1.meta_value = ‘1’ AND pm1.post_id = p1.id
\n INNER JOIN {$wpdb->prefix}postmeta pm2 ON pm2.meta_key = ‘wpcr3_review_post’ AND pm2.meta_value = p1.id
\n WHERE p1.id = %d
\n ) tmp1
\n INNER JOIN {$wpdb->prefix}posts p2 ON p2.id = tmp1.post_id AND p2.post_status = ‘publish’ AND p2.post_type = ‘wpcr3_review’
\n INNER JOIN {$wpdb->prefix}postmeta pm4 ON pm4.post_id = p2.id AND pm4.meta_key = ‘wpcr3_review_rating’ AND pm4.meta_value IS NOT NULL AND pm4.meta_value != ‘0’
\n GROUP BY p2.id
\n ) tmp2
\n “, intval($postid));
\n $results = $wpdb->get_results($query);<\/p>\n

$rtn = new stdClass();<\/p>\n

if (count($results)) {
\n $rtn->aggregate_count = $results[0]->aggregate_count;
\n $rtn->aggregate_rating = $results[0]->aggregate_rating;
\n if ($rtn->aggregate_count == 0) { $rtn->aggregate_rating = 0; }
\n $rtn->stars = $this->get_rating_template($rtn->aggregate_rating, false);
\n }<\/p>\n

return $rtn;
\n }<\/p>\n

So, when leaving the bold part out, all works fine. <\/p>\n

Would it be possible for you to change the code so that, just in case of using the do_shortcode in a template and the checkbox on the page is off, the aggregates are still showing correctly?<\/p>\n

Otherwise, with each update, we’d have to change this manually and that is bound to go wrong.<\/p>\n

Hope you will find the time and if you need more clarification don’t hesitate to contact me.<\/p>\n

Greets,
\nOlaf<\/p>\n","protected":false},"template":"","class_list":["post-10546874","topic","type-topic","status-publish","hentry","topic-tag-shortcode"],"jetpack_sharing_enabled":true,"_links":{"self":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic\/10546874","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic"}],"about":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/types\/topic"}],"version-history":[{"count":2,"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic\/10546874\/revisions"}],"predecessor-version":[{"id":10546886,"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic\/10546874\/revisions\/10546886"}],"wp:attachment":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/media?parent=10546874"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}