From f37ea1c8573bf48ef3c846be46dc20155c43995f Mon Sep 17 00:00:00 2001 From: Anthony Nemirovsky Date: Fri, 15 Aug 2014 17:39:34 -0700 Subject: [PATCH] Fix issue with reports creating invalid URLs when updated. When updating a report, if the Report Settings is set to "Include Report in Navigation Menu", the navigation url that is created is formatted like /civicrm/report/instance/1_reset_1, which is incorrect. The correct URL should be /civicrm/report/instance/1?reset=1. --- CRM/Report/BAO/ReportInstance.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CRM/Report/BAO/ReportInstance.php b/CRM/Report/BAO/ReportInstance.php index 55b36f3a4b..e21a680624 100644 --- a/CRM/Report/BAO/ReportInstance.php +++ b/CRM/Report/BAO/ReportInstance.php @@ -191,7 +191,7 @@ class CRM_Report_BAO_ReportInstance extends CRM_Report_DAO_ReportInstance { if (empty($params['id']) && empty($params['instance_id']) && !empty($navigationParams['id'])) { unset($navigationParams['id']); } - $navigationParams['url'] = "civicrm/report/instance/{$instance->id}&reset=1"; + $navigationParams['url'] = "civicrm/report/instance/{$instance->id}?reset=1"; $navigation = CRM_Core_BAO_Navigation::add($navigationParams); if (!empty($navigationParams['is_active'])) { -- 2.25.1