[REF][PHP8.2] Declare properties in CRM_Queue_ErrorPolicy
authorBradley Taylor <hello@brad-taylor.co.uk>
Sat, 23 Sep 2023 10:38:11 +0000 (11:38 +0100)
committerBradley Taylor <hello@brad-taylor.co.uk>
Sat, 23 Sep 2023 10:38:11 +0000 (11:38 +0100)
CRM/Queue/ErrorPolicy.php

index 178591bbd209927dbea5a36fcc2660e8f883afad..9246036f9bc272877ca4ca8d863d85aa76b008d0 100644 (file)
  * will be necessary to get reuse from the other parts of this class.
  */
 class CRM_Queue_ErrorPolicy {
-  public $active;
+
+  /**
+   * @var bool
+   */
+  protected $active;
+
+  /**
+   * @var int
+   */
+  protected $level;
+
+  /**
+   * @var array
+   */
+  protected $backup;
 
   /**
    * @param null|int $level
@@ -43,7 +57,7 @@ class CRM_Queue_ErrorPolicy {
   /**
    * Enable the error policy.
    */
-  public function activate() {
+  protected function activate() {
     $this->active = TRUE;
     $this->backup = [];
     foreach ([
@@ -55,14 +69,12 @@ class CRM_Queue_ErrorPolicy {
       ini_set($key, 0);
     }
     set_error_handler([$this, 'onError'], $this->level);
-    // FIXME make this temporary/reversible
   }
 
   /**
    * Disable the error policy.
    */
-  public function deactivate() {
-    $this->errorScope = NULL;
+  protected function deactivate() {
     restore_error_handler();
     foreach ([
       'display_errors',
@@ -137,7 +149,7 @@ class CRM_Queue_ErrorPolicy {
    * @param array $error
    *   The PHP error (with "type", "message", etc).
    */
-  public function reportError($error) {
+  protected function reportError($error) {
     $response = [
       'is_error' => 1,
       'is_continue' => 0,
@@ -158,7 +170,7 @@ class CRM_Queue_ErrorPolicy {
    * @param Exception $e
    *   The unhandled exception.
    */
-  public function reportException(Exception $e) {
+  protected function reportException(Exception $e) {
     CRM_Core_Error::debug_var('CRM_Queue_ErrorPolicy_reportException', CRM_Core_Error::formatTextException($e));
 
     $response = [