Skip to content

Commit

Permalink
Fix for silverstripe#1018 "'Export To CSV' for Reports not working"
Browse files Browse the repository at this point in the history
The call to LeftAndMain::Link() was appending the Action as opposed to passing it as a parameter
This resulted in the double forward slash (/),

There is still possibly an unresolved issue with LeftAndMain::Link() about appending the slash at the end??
  • Loading branch information
oilee80 authored and Damian Mooyman committed Jul 15, 2014
1 parent 681645a commit 7c9c8e1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion code/ReportAdmin.php
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ public function reportEditFormFor($id) {
unset($filteredCriteria[$notAParam]);
}

$formLink = $this->Link() . '/EditForm';
$formLink = $this->Link( 'EditForm' );
if($filteredCriteria) $formLink .= '?' . http_build_query($filteredCriteria);
$form->setFormAction($formLink);
$form->setTemplate('ReportAdminForm');
Expand Down

0 comments on commit 7c9c8e1

Please sign in to comment.