Permalink
Browse files

FIX ReportAdmin report links regression

Regression from 79996a7,
escaped quotes are no longer necessary because the string
doesn't get eval'ed.
  • Loading branch information...
1 parent 30cf08b commit daf92e6ce0313b6a2debef6e1b03550104202e99 @chillu chillu committed Oct 1, 2013
Showing with 16 additions and 1 deletion.
  1. +1 −1 code/controllers/ReportAdmin.php
  2. +15 −0 tests/behat/features/view-reports.feature
@@ -168,7 +168,7 @@ public function getEditForm($id = null, $fields = null) {
$columns->setFieldFormatting(array(
'title' => function($value, &$item) {
return sprintf(
- '<a href=\"%s\" class=\"cms-panel-link\">%s</a>',
+ '<a href="%s" class="cms-panel-link">%s</a>',
Convert::raw2xml($item->Link),
Convert::raw2xml($value)
);
@@ -0,0 +1,15 @@
+Feature: View Reports
+ As an author
+ I want to review aggregate reports on pages
+ So that I can keep an overview on the health of my website data
+
+ Background:
+ Given a "page" "Empty Page"
+ And a "page" "Filled Page" with "Content"="Some Content"
+ And I am logged in with "ADMIN" permissions
+ And I go to "/admin/reports"
+
+ Scenario: I can view the "Pages with no content" report
+ When I follow "Pages with no content"
+ Then I should see "Empty Page"
+ But I should not see "Filled Page"

0 comments on commit daf92e6

Please sign in to comment.