[Done v1.3.9b] 1.3.9a Random Reviews Running Rampant

[ToniScraparoni]

Tony, can you specify which version you upgraded from?
I'm puzzled as to why it would be showing 100 reviews if you have only specified 1 as the maximum. I can't recreate that problem in 1.3.9

1.3.8a, as indicated in my original post. ;)

The issue of the "text" on product reviews showing the entire review can be quickly fixed by making this small change to the tpl_reviews_random.php template file:

Find this:
zen_break_string(nl2br(zen_output_string_protected(stripslashes($random_review_s idebox_pr oduct->fields['reviews_text']))), 60, '-<br />') . ' ..</a><br /><br />')
and replace with this:
zen_trunc_string(nl2br(zen_output_string_protected(stripslashes($random_review_s idebox_pr oduct->fields['reviews_text']))), 60)

Not that easily, apparently. Hehe!

PHP Parse error: syntax error, unexpected T_STRING in /sideboxes/tpl_reviews_random.php on line 16

Thanks very much for the help. Glad to see I'm not alone. ;)

[ToniScraparoni]

Updating to say I put Random Reviews back to 1, which displays only one review in the box as it should. The reviews seem to rotate when I refresh the page. Yay!

However, in 1.3.8a, it seems like there was an additional setting; one for the number to display at one time, another for how many to rotate. I remember changing that when I received a complaint the same reviews were always showing. Assuming then, 1.3.9a will automatically rotate among every review.

So, if we can fix the trunc issue, I/we all should be good to go. Thank you! :)

[DrByte]

Oops ... sorry ... I left off a tiny bit, which I've corrected in my earlier post:

. '</a><br /><br />')

The corrections are just the parts highlighted.

[ToniScraparoni]

Thanks, Dr. Byte. Still not working for me, though.

[DrByte]

Fine.
Replace the whole line with this then:

    $content .= '<a href="' . zen_href_link(FILENAME_PRODUCT_REVIEWS_INFO, 'products_id=' . $random_review_sidebox_product->fields['products_id'] . '&reviews_id=' . $random_review_sidebox_product->fields['reviews_id']) . '">' . zen_image(DIR_WS_IMAGES . $random_review_sidebox_product->fields['products_image'], $random_review_sidebox_product->fields['products_name'], SMALL_IMAGE_WIDTH, SMALL_IMAGE_HEIGHT) . '<br />' . zen_trunc_string(nl2br(zen_output_string_protected(stripslashes($random_review_sidebox_product->fields['reviews_text']))), 60) . '</a><br /><br />' . zen_image(DIR_WS_TEMPLATE_IMAGES . 'stars_' . $random_review_sidebox_product->fields['reviews_rating'] . '.gif' , sprintf(BOX_REVIEWS_TEXT_OF_5_STARS, $random_review_sidebox_product->fields['reviews_rating']));

[ToniScraparoni]

Now THAT worked! TYVM!!

But you aren't getting rid of me just yet, I'm afraid. Hehe! I just noticed another related issue. I have a custom sidebox below the reviews sidebox. The header in the box below is now mirroring the reviews sidebox header. It wasn't a linking image before. Now it links to reviews and has the [more] link underneath.

Haven't touched that box since putting it in almost a year ago. Checked the templates, all look normal. Any ideas?

Thank you again for all of your help!

[DrByte]

That suggests that your custom sidebox isn't properly resetting the variables before execution. Compare its /includes/modules/sideboxes/ file with other built-in sideboxes to see what you've missed.

That's not a bug with ZC core files. If you have specific problems with it, please start another thread for it outside of the Bug Reports area.

[ToniScraparoni]

Great suggestion, Dr. B. I dunno why it changed behavior after the upgrade, but adding in $title_link = false; fixed it.

Thank you again for all of your help! :)