From 44ca57d8ea66e0de63d327f9d1a15c6054d4a2dd Mon Sep 17 00:00:00 2001
From: Camilo Rodriguez <camilo@compucorp.co.uk>
Date: Thu, 21 Dec 2017 11:59:39 +0000
Subject: [PATCH] CRM-21520: Remove Unneeded Change to Menu XML and Refactor
 Redirect

---
 CRM/Case/Form/AddToCaseAsRole.php | 7 ++++++-
 CRM/Case/xml/Menu/Case.xml        | 7 -------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/CRM/Case/Form/AddToCaseAsRole.php b/CRM/Case/Form/AddToCaseAsRole.php
index 1fa811d399..d62dd92ad1 100644
--- a/CRM/Case/Form/AddToCaseAsRole.php
+++ b/CRM/Case/Form/AddToCaseAsRole.php
@@ -80,7 +80,12 @@ class CRM_Case_Form_AddToCaseAsRole extends CRM_Contact_Form_Task {
 
     $url = CRM_Utils_System::url(
       'civicrm/contact/view/case',
-      sprintf('cid=%d&id=%d', $clients[0], $caseId)
+      array(
+        'cid' => $clients[0],
+        'id' => $caseId,
+        'reset' => 1,
+        'action' => 'view',
+      )
     );
     CRM_Utils_System::redirect($url);
   }
diff --git a/CRM/Case/xml/Menu/Case.xml b/CRM/Case/xml/Menu/Case.xml
index 976c04aeb6..386cdc131d 100644
--- a/CRM/Case/xml/Menu/Case.xml
+++ b/CRM/Case/xml/Menu/Case.xml
@@ -136,11 +136,4 @@
     <path>civicrm/ajax/delcaserole</path>
     <page_callback>CRM_Case_Page_AJAX::deleteCaseRoles</page_callback>
   </item>
-  <item>
-    <path>civicrm/case/add-contact</path>
-    <page_callback>CRM_Case_Form_AddContact</page_callback>
-    <title>AddContact</title>
-    <access_arguments>access CiviCRM</access_arguments>
-    <page_type>0</page_type>
-  </item>
 </menu>
-- 
2.25.1