Passing a variable by reference via call_user_func is deprecated. I've fixed it, but I've got a mess of other code in there so I can't post a patch right now. However, replacing call_user_func with call_user_func_array on lines 604 and 610 like this:

       // Calculate weekly totals using the appropriate scoring engine.
        call_user_func_array('get_user_totals_'. $league->scoring_engine['type'], array(&$totals, $row));

      }

      // If there is a get_user_totals_finalize_* function for this scoring engine, run it.
      if ( function_exists('get_user_totals_finalize_'. $league->scoring_engine['type']) ) {
        call_user_func_array('get_user_totals_finalize_'. $league->scoring_engine['type'], array(&$totals));
      }

And then changing the function definitions so that $totals becomes &$totals makes the warning go away, and the code works fine.

Comments

jvandervort’s picture

Status: Active » Fixed

Committed to dev, thanks.

Status: Fixed » Closed (fixed)

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