Using the print module, the review form shows up on the print page. It would be nice if the current reviews were printed on the page, the form shouldn't be showing up.

Comments

dado’s picture

Do you mean Print Friendly Pages module?
http://drupal.org/project/print

It surprises me that the form would show up on the print page, given that it is inside a collapsed field set. I guess when rendered as page for print, it expands all fieldsets. will look for a way to suppress userreview form when printing

dado’s picture

Assigned: Unassigned » dado
dado’s picture

Status: Active » Fixed

I committed code to CVS which witholds printing of the user review form when the request is called by the print module. But to work, the print module must have the small patch which was provided here.
http://drupal.org/node/60817

Pls post here how it works

sami_k’s picture

It works! The only other thing that I might recommend, if you have a bit of time, is a checkbox on the module's page that would enable/disable the printing of the reviews.

sami_k’s picture

It might also be good to disable the user's own review in the print as that's printed twice. Also I think the previous version was hiding edit your review in the second box, I think that should in fact be the case. The reason is because it seems strange when that text appears when you go to print it.

dado’s picture

not sure that i would do these now. some users might want their own reviews to print. i understand the value of configuring these options in settings page but you get to having an awful lot of settings, which means harder maintenance. if you want to prevent feel free to introduce it in the code

//if a review exists by this user, show it...
        if ($userreviewnode) {

becomes

//if a review exists by this user, show it...
        if ($userreviewnode && !$node->printing) {

and

//next, show a list of user reviews if any reviews
        if ($count_votes->value > 0) {  

becomes

//next, show a list of user reviews if any reviews
        if ($count_votes->value > 0  && !$node->printing) {  
sami_k’s picture

Thanks. that sounds fair... I agree with your decision.

Anonymous’s picture

Status: Fixed » Closed (fixed)