Merge pull request #24162 from colemanw/savedSearchLabel

[civicrm-core.git] / CRM / Core / Page / Redirect.php

diff --git a/CRM/Core/Page/Redirect.php b/CRM/Core/Page/Redirect.php
index f5f484e708ec0cd1b5ef78c6da3ff2c2cf8362bf..4f0b83dde4b60267b614c0890064518c8b989ea6 100644 (file)
--- a/CRM/Core/Page/Redirect.php
+++ b/CRM/Core/Page/Redirect.php
@@ -3,18 +3,24 @@
 /**
  * Placeholder page which generates a redirect
  *
- * @code
+ * ```
  * <item>
  *   <path>civicrm/admin/options/case_type</path>
  *   <page_callback>CRM_Core_Page_Redirect</page_callback>
  *   <page_arguments>url=civicrm/foo/bar?whiz=bang&amp;passthru=%%passthru%%</page_arguments>
  * </item>
- * @endcoe
+ * ```
  */
 class CRM_Core_Page_Redirect extends CRM_Core_Page {
-  public function run($path = NULL, $pageArgs = array()) {
+
+  /**
+   * Run page.
+   *
+   * @param string $path
+   * @param array $pageArgs
+   */
+  public function run($path = NULL, $pageArgs = []) {
     $url = self::createUrl($path, $_REQUEST, $pageArgs, TRUE);
-    // return $url;
     CRM_Utils_System::redirect($url);
   }
 
@@ -32,10 +38,10 @@ class CRM_Core_Page_Redirect extends CRM_Core_Page {
    */
   public static function createUrl($requestPath, $requestArgs, $pageArgs, $absolute) {
     if (empty($pageArgs['url'])) {
-      CRM_Core_Error::fatal('This page is configured as a redirect, but it does not have a target.');
+      CRM_Core_Error::statusBounce(ts('This page is configured as a redirect, but it does not have a target.'));
     }
 
-    $vars = array();
+    $vars = [];
     // note: %% isn't legal in a well-formed URL, so it's not a bad variable-delimiter
     foreach ($requestPath as $pathPos => $pathPart) {
       $vars["%%{$pathPos}%%"] = urlencode($pathPart);