It seems that the number of posts to show is mixed up. The entered numbers for page are taken for blocks and the other way around.

I changed lines 241 ff in shoutbox.module from:

  // Determine the post count
  if ($block) {
    $show_amount = variable_get('shoutbox_showamount_page', '30');
  }
  else {
    $show_amount = variable_get('shoutbox_showamount_block', '10');
  }

to

  // Determine the post count
  if ($block) {
    $show_amount = variable_get('shoutbox_showamount_block', '10');
  }
  else {
    $show_amount = variable_get('shoutbox_showamount_page', '30');
  }

bacause your comment "TRUE if the output desired is for a block, otherwise FALSE" makes more sense then.

Comments

mstef’s picture

mstef commented

thanks..good find..

damm i knew i rushed the 2.2 release

mstef’s picture

mstef commented

Wow..didn't realize I'm not using _shoutbox_block_view() any more..

There were just so many changes it's hard to keep up. Especially because I didn't write the original 1.x version.

mstef’s picture

mstef commented
Status: Active » Closed (fixed)

Alright fixed..wonder if I should put up a 2.3? What do ya think...?

JThan’s picture

JThan commented

I dont know... for me it is working now ;) - and I had to do some other hacks for a special use. So I won't update anyway.

If you are interested in the little 'special' hacks: I added the last three chars of the sid to the anonymous because mainly anonymous people just dropping by will use this as a way to ask questions on a doctors page. Also the name link should not go the users page (the only logged in chatter will be the doctor) but to their public profile and I hardcoded that link (I know, this is bad style).

Well, thx again for the module :)

mstef’s picture

mstef commented

You're very welcome..keep the feedback coming

mstef’s picture

mstef commented

If you can tomorrow, please pull in the latest dev and test it out. I want to tag a 2.3 version but need to make sure it's definitely good to go..

Thanks