When retroactivly awarding points the userpoints_retroactive_do() function is awarding points twice for nodes and comments because it is calling the respective functions twice.

function userpoints_retroactive_do() {
  userpoints_retroactive_nodes();
  userpoints_retroactive_comments();


  set_time_limit(240);
  timer_start('up_retro');
  $num = userpoints_retroactive_nodes();
  $num += userpoints_retroactive_comments();
  $num += userpoints_retroactive_votingapi();
  $ms = timer_read('up_retro');
  watchdog('userpoints', t('Userpoints retroactive processed %count items in %ms milliseconds'), array('%ms' => $ms, '%count' => $num));
}

The first 2 lines of that function are not needed and should be removed for proper functioning.

Comments

kbahey’s picture

Status: Needs review » Fixed

Fixed.

Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.