VirtueMart Forum

VirtueMart 2 + 3 + 4 => Virtuemart Development and bug reports => Topic started by: fexnok on October 24, 2012, 00:52:57 AM

Title: Reviews not shown
Post by: fexnok on October 24, 2012, 00:52:57 AM
Hi,

I'm using VM 2.0.12f, when you publish or unpublish reviews, the value in the database remains 0, so the review is never shown.

Thanks
Title: Re: Reviews not shown
Post by: John2400 on October 24, 2012, 11:29:13 AM
Hi fexnok,

I tried this issue on several sites. I just cannot reproduce the problem.? sorry all 2.0.12f
Title: Re: Reviews not shown
Post by: fexnok on October 24, 2012, 11:59:31 AM
Thanks for your answer. Let me explain a little bit better.

This is my config: Only registered users can post reviews and reviews are not published automatically, it needs to be manually published from the admin panel.

Now, from what I can see in the file administrator/components/com_virtuemart/controllers/ratings.php line 97

$product_id= JRequest::getInt('virtuemart_product_id',0);

the function publish is expecting a parameter called 'virtuemart_product_id' but checking the variables passed from the form I can't see that variable being passed.

My post variables for publish are the following

filter_ratings=&cid%5B%5D=3&limit=10&limitstart=0&filter_order=created_on&filter_order_Dir=DESC&task=publish&option=com_virtuemart&boxchecked=1&controller=ratings&view=ratings&ccdbef72b3089465dfab911deeed3a01=1

and unpublish

filter_ratings=&cid%5B%5D=3&limit=10&limitstart=0&filter_order=created_on&filter_order_Dir=DESC&task=unpublish&option=com_virtuemart&boxchecked=1&controller=ratings&view=ratings&ccdbef72b3089465dfab911deeed3a01=1

The only difference between both are task=publish/unpublish


Also if I check the tables, the table virtuemart_ratings is correctly updated when I publish/unpublish items and I can see from the front-end that the icon correctly changes from red to green and viceversa. On the other hand, the table virtuemart_rating_reviews is not updated correctly, as published field doesn't change accordingly.

I hope it explains it better.

Thanks