I don't know for sure if this is a bug or due to the fact that I've hacked the module a bit to make it work in my nodeprofile, but I'm submitting it because the line in question is in the the original code:

  $output .= '<div class="content guestbook-message">'. check_markup($entry['message'], variable_get('guestbook_input_format',1)) .'</div>';

According to http://api.drupal.org/api/5/function/check_markup if you don't explicitly send FALSE for the last parameter, it will check to see if the current user has access to that input filter. So if you have an input filter for the guestbook that not all roles can use, you get N/A instead of text. If you change the line to:

  $output .= '<div class="content guestbook-message">'. check_markup($entry['message'], variable_get('guestbook_input_format',1,FALSE)) .'</div>';

then it works fine. I ran into this because the variable_get isn't returning a number in my case (which may be due to the hacking around... I don't know) and defaulted to 1, which only admins have access to.

Michelle

Comments

hba’s picture

hba commented
Assigned: Unassigned » hba
Status: Active » Fixed

Thanks for your bug report, I just commited your fix to the 5.x branch.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
ekrispin’s picture

ekrispin commented

The above fix doesn't work due to a small error: The FALSE should be the last parameter in the check_markup() function and not in the variable_get() function (should be placed one parenthesis ahead).

Hence the line should change to:

$output .= '<div class="content guestbook-message">'. check_markup($entry['message'], variable_get('guestbook_input_format',1),FALSE) .'</div>';