]> Kevux Git Server - koopa/commitdiff
Update: standard path output format test should be available at the standard path...
authorKevin Day <thekevinday@gmail.com>
Sat, 3 Feb 2018 02:21:49 +0000 (20:21 -0600)
committerKevin Day <thekevinday@gmail.com>
Sat, 3 Feb 2018 02:21:49 +0000 (20:21 -0600)
The output format is not specific to the user-level pages.
The output format check should be a standard page-level check.

common/standard/classes/standard_path.php
common/standard/classes/standard_path_user.php

index 42bf0d11c4fbd7d259c4b5bf30ca49f207daee05..e8c5293fa7d13823dec039d2591f57aa8d8a32c0 100644 (file)
@@ -2674,5 +2674,33 @@ class c_standard_path extends c_base_path {
   protected function pr_process_access_denied(&$executed) {
     return FALSE;
   }
+
+  /**
+   * Provides a standard access check handler function, specific for output format.
+   *
+   * This is generally intended to be called by do_execute().
+   *
+   * @param c_base_path_executed &$executed
+   *   The execution array for making changes to.
+   *   Any detected errors are assigned to this.
+   *   This often may have the output settings altered by this class implementation.
+   *
+   * @return bool
+   *   FALSE on access granted, TRUE on access denied.
+   */
+  protected function pr_process_output_format_denied(&$executed) {
+
+    // only support HTML output unless otherwise needed.
+    // @todo: eventually all HTML output will be expected to support at least print and PDF formats (with print being the string 'print').
+    if ($this->output_format !== c_base_mime::TYPE_TEXT_HTML) {
+      $error = c_base_error::s_log(NULL, ['arguments' => [':{path_name}' => static::PATH_SELF . '/' . implode('/', $this->arguments), ':{function_name}' => __CLASS__ . '->' . __FUNCTION__]], i_base_error_messages::NOT_FOUND_PATH);
+      $executed->set_error($error);
+      unset($error);
+
+      return TRUE;
+    }
+
+    return FALSE;
+  }
 }
 
index 23a140aef27b5ec9851c8a55db256c44d43ddf5d..200c18c7bd34e5bd26c4a72154bbff9052a16083 100644 (file)
@@ -319,34 +319,6 @@ class c_standard_path_user extends c_standard_path {
   }
 
   /**
-   * Provides a standard access check handler function, specific for output format.
-   *
-   * This is generally intended to be called by do_execute().
-   *
-   * @param c_base_path_executed &$executed
-   *   The execution array for making changes to.
-   *   Any detected errors are assigned to this.
-   *   This often may have the output settings altered by this class implementation.
-   *
-   * @return bool
-   *   FALSE on access granted, TRUE on access denied.
-   */
-  protected function pr_process_output_format_denied(&$executed) {
-
-    // only support HTML output unless otherwise needed.
-    // @todo: eventually all HTML output will be expected to support at least print and PDF formats (with print being the string 'print').
-    if ($this->output_format !== c_base_mime::TYPE_TEXT_HTML) {
-      $error = c_base_error::s_log(NULL, ['arguments' => [':{path_name}' => static::PATH_SELF . '/' . implode('/', $this->arguments), ':{function_name}' => __CLASS__ . '->' . __FUNCTION__]], i_base_error_messages::NOT_FOUND_PATH);
-      $executed->set_error($error);
-      unset($error);
-
-      return TRUE;
-    }
-
-    return FALSE;
-  }
-
-  /**
    * Execution of the view path.
    *
    * @param c_base_path_executed &$executed