Fix the check to see if the financialAclExtension is installed
authorJohn Twyman <john.twyman@greens.org.au>
Tue, 10 Aug 2021 06:48:00 +0000 (16:48 +1000)
committerJohn Twyman <john.twyman@greens.org.au>
Tue, 10 Aug 2021 07:14:02 +0000 (17:14 +1000)
CRM/Upgrade/Incremental/General.php

index 115e47a7d01e2d7fa04a5e38775f65a96f9610b1..9023a670c80090d355d5ee38d8ad92f16ca4a3a4 100644 (file)
@@ -122,8 +122,8 @@ class CRM_Upgrade_Incremental_General {
     }
 
     $ftAclSetting = Civi::settings()->get('acl_financial_type');
-    $financialAclExtension = civicrm_api3('extension', 'get', ['key' => 'biz.jmaconsulting.financialaclreport']);
-    if ($ftAclSetting && (($financialAclExtension['count'] == 1 && $financialAclExtension['status'] != 'Installed') || $financialAclExtension['count'] !== 1)) {
+    $financialAclExtension = civicrm_api3('extension', 'get', ['key' => 'biz.jmaconsulting.financialaclreport', 'sequential' => 1]);
+    if ($ftAclSetting && (($financialAclExtension['count'] == 1 && $financialAclExtension['values'][0]['status'] != 'Installed') || $financialAclExtension['count'] !== 1)) {
       $preUpgradeMessage .= '<br />' . ts('CiviCRM will in the future require the extension %1 for CiviCRM Reports to work correctly with the Financial Type ACLs. The extension can be downloaded <a href="%2">here</a>', [
         1 => 'biz.jmaconsulting.financialaclreport',
         2 => 'https://github.com/JMAConsulting/biz.jmaconsulting.financialaclreport',