r86581 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r86580‎ | r86581 | r86582 >
Date:00:07, 21 April 2011
Author:awjrichards
Status:deferred
Tags:
Comment:
Fixed order of params for array_push in Paypal_Audit::getCustomReport()
Modified paths:
  • /civicrm/trunk/sites/all/modules/paypal_audit/paypal_audit.php (modified) (history)

Diff [purge]

Index: civicrm/trunk/sites/all/modules/paypal_audit/paypal_audit.php
@@ -11,14 +11,14 @@
1212 * @param string $end_date
1313 * @param array $options
1414 */
15 - public function getCustomReport( $start_date, $end_date, $options ) {
 15+ public function getCustomReport( $start_date, $end_date, $options=array() ) {
1616 require_once( variable_get( 'paypal_audit_dir', CONTRIBUTION_AUDIT_PAYFLOW_AUDIT_DIR ) . "PayflowReportTypes.php");
1717
1818 $time_bounds = array( $this->fixTime( $start_date ), $this->fixTime( $endDate ));
1919
2020 $report = new CustomReport( $time_bounds );
2121 $report_options = $report->getOptions( true );
22 - $report->setOptions( array_push( $options, $report_options ));
 22+ $report->setOptions( array_push( $report_options, $options ));
2323 $report->runReport();
2424
2525 return $results;

Status & tagging log