From 4b1fdf53692d695dc7d58b4601689cfaf985c8c2 Mon Sep 17 00:00:00 2001
From: Seamus Lee <seamuslee001@gmail.com>
Date: Sat, 16 Nov 2019 09:26:55 +1100
Subject: [PATCH] [REF] Convert CRM_Core_Error::fatal to
 CRM_Core_Error::statusBounce in Extension folder

---
 CRM/Extension/Downloader.php      | 6 +++---
 CRM/Extension/Manager/Payment.php | 6 +++---
 CRM/Extension/Manager/Report.php  | 9 ++-------
 CRM/Extension/Manager/Search.php  | 2 +-
 4 files changed, 9 insertions(+), 14 deletions(-)

diff --git a/CRM/Extension/Downloader.php b/CRM/Extension/Downloader.php
index f06fba6e8a31..8f2ca9dd99a3 100644
--- a/CRM/Extension/Downloader.php
+++ b/CRM/Extension/Downloader.php
@@ -71,7 +71,7 @@ public function checkRequirements($extensionInfo = NULL) {
 
     if (empty($errors) && !CRM_Utils_HttpClient::singleton()->isRedirectSupported()) {
       CRM_Core_Session::setStatus(ts('WARNING: The downloader may be unable to download files which require HTTP redirection. This may be a configuration issue with PHP\'s open_basedir or safe_mode.'));
-      CRM_Core_Error::debug_log_message('WARNING: The downloader may be unable to download files which require HTTP redirection. This may be a configuration issue with PHP\'s open_basedir or safe_mode.');
+      Civi::log()->debug('WARNING: The downloader may be unable to download files which require HTTP redirection. This may be a configuration issue with PHP\'s open_basedir or safe_mode.');
     }
 
     if ($extensionInfo) {
@@ -105,7 +105,7 @@ public function download($key, $downloadUrl) {
     $destDir = $this->containerDir . DIRECTORY_SEPARATOR . $key;
 
     if (!$downloadUrl) {
-      CRM_Core_Error::fatal(ts('Cannot install this extension - downloadUrl is not set!'));
+      CRM_Core_Error::statusBounce(ts('Cannot install this extension - downloadUrl is not set!'));
     }
 
     if (!$this->fetch($downloadUrl, $filename)) {
@@ -213,7 +213,7 @@ public function validateFiles($key, $extractedZipPath) {
     }
 
     if ($newInfo->key != $key) {
-      CRM_Core_Error::fatal(ts('Cannot install - there are differences between extdir XML file and archive XML file!'));
+      CRM_Core_Error::statusBounce(ts('Cannot install - there are differences between extdir XML file and archive XML file!'));
     }
 
     return TRUE;
diff --git a/CRM/Extension/Manager/Payment.php b/CRM/Extension/Manager/Payment.php
index f8b7ec092718..ed0983324e8d 100644
--- a/CRM/Extension/Manager/Payment.php
+++ b/CRM/Extension/Manager/Payment.php
@@ -39,12 +39,12 @@ public function onPreInstall(CRM_Extension_Info $info) {
     $paymentProcessorTypes = $this->_getAllPaymentProcessorTypes('class_name');
 
     if (array_key_exists($info->key, $paymentProcessorTypes)) {
-      CRM_Core_Error::fatal(ts('This payment processor type is already installed.'));
+      CRM_Core_Error::statusBounce(ts('This payment processor type is already installed.'));
     }
 
     $ppByName = $this->_getAllPaymentProcessorTypes('name');
     if (array_key_exists($info->name, $ppByName)) {
-      CRM_Core_Error::fatal(ts('This payment processor type already exists.'));
+      CRM_Core_Error::statusBounce(ts('This payment processor type already exists.'));
     }
 
     $dao = new CRM_Financial_DAO_PaymentProcessorType();
@@ -82,7 +82,7 @@ public function onPreInstall(CRM_Extension_Info $info) {
         break;
 
       default:
-        CRM_Core_Error::fatal(ts('Billing mode in info file has wrong value.'));
+        CRM_Core_Error::statusBounce(ts('Billing mode in info file has wrong value.'));
     }
 
     $dao->is_recur = trim($info->typeInfo['isRecur']);
diff --git a/CRM/Extension/Manager/Report.php b/CRM/Extension/Manager/Report.php
index eeba44919b7a..0f715bde53a7 100644
--- a/CRM/Extension/Manager/Report.php
+++ b/CRM/Extension/Manager/Report.php
@@ -37,7 +37,7 @@ public function __construct() {
   public function onPreInstall(CRM_Extension_Info $info) {
     $customReports = $this->getCustomReportsByName();
     if (array_key_exists($info->key, $customReports)) {
-      CRM_Core_Error::fatal(ts('This report is already registered.'));
+      CRM_Core_Error::statusBounce(ts('This report is already registered.'));
     }
 
     if ($info->typeInfo['component'] === 'Contact') {
@@ -48,7 +48,7 @@ public function onPreInstall(CRM_Extension_Info $info) {
       $compId = $comp->componentID;
     }
     if (empty($compId)) {
-      CRM_Core_Error::fatal(ts('Component for which you are trying to install the extension (%1) is currently disabled.', [1 => $info->typeInfo['component']]));
+      CRM_Core_Error::statusBounce(ts('Component for which you are trying to install the extension (%1) is currently disabled.', [1 => $info->typeInfo['component']]));
     }
     $weight = CRM_Utils_Weight::getDefaultWeight('CRM_Core_DAO_OptionValue',
       ['option_group_id' => $this->groupId]
@@ -74,11 +74,6 @@ public function onPreInstall(CRM_Extension_Info $info) {
    * @return bool
    */
   public function onPreUninstall(CRM_Extension_Info $info) {
-
-    //        if( !array_key_exists( $info->key, $this->customReports ) ) {
-    //            CRM_Core_Error::fatal( 'This report is not registered.' );
-    //        }
-
     $customReports = $this->getCustomReportsByName();
     $cr = $this->getCustomReportsById();
     $id = $cr[$customReports[$info->key]];
diff --git a/CRM/Extension/Manager/Search.php b/CRM/Extension/Manager/Search.php
index 3fc8167c9ac7..f0f22169bb61 100644
--- a/CRM/Extension/Manager/Search.php
+++ b/CRM/Extension/Manager/Search.php
@@ -38,7 +38,7 @@ public function __construct() {
   public function onPreInstall(CRM_Extension_Info $info) {
     $customSearchesByName = $this->getCustomSearchesByName();
     if (array_key_exists($info->key, $customSearchesByName)) {
-      CRM_Core_Error::fatal(ts('This custom search is already registered.'));
+      CRM_Core_Error::statusBounce(ts('This custom search is already registered.'));
     }
 
     $weight = CRM_Utils_Weight::getDefaultWeight('CRM_Core_DAO_OptionValue',