From 99390f48dc9a74f41744afed006adb77c2737079 Mon Sep 17 00:00:00 2001
From: Matthew Wire <mjw@mjwconsult.co.uk>
Date: Mon, 10 Aug 2020 19:29:43 +0100
Subject: [PATCH] Regenerate civix code
---
CRM/Stripe/Upgrader/Base.php | 73 +++++++++++++++++++++---------------
stripe.civix.php | 22 +++++------
2 files changed, 54 insertions(+), 41 deletions(-)
diff --git a/CRM/Stripe/Upgrader/Base.php b/CRM/Stripe/Upgrader/Base.php
index 3749a426..9bdc78c1 100644
--- a/CRM/Stripe/Upgrader/Base.php
+++ b/CRM/Stripe/Upgrader/Base.php
@@ -9,9 +9,9 @@ use CRM_Stripe_ExtensionUtil as E;
class CRM_Stripe_Upgrader_Base {
/**
- * @var varies, subclass of this
+ * @var CRM_Stripe_Upgrader_Base subclass of this
*/
- static $instance;
+ public static $instance;
/**
* @var CRM_Queue_TaskContext
@@ -19,22 +19,23 @@ class CRM_Stripe_Upgrader_Base {
protected $ctx;
/**
- * @var string, eg 'com.example.myextension'
+ * @var string eg 'com.example.myextension'
*/
protected $extensionName;
/**
- * @var string, full path to the extension's source tree
+ * @var string full path to the extension's source tree
*/
protected $extensionDir;
/**
- * @var array(revisionNumber) sorted numerically
+ * @var revisionNumber[]
+ * sorted numerically
*/
private $revisions;
/**
- * @var boolean
+ * @var bool
* Flag to clean up extension revision data in civicrm_setting
*/
private $revisionStorageIsDeprecated = FALSE;
@@ -42,12 +43,12 @@ class CRM_Stripe_Upgrader_Base {
/**
* Obtain a reference to the active upgrade handler.
*/
- static public function instance() {
+ public static function instance() {
if (!self::$instance) {
// FIXME auto-generate
self::$instance = new CRM_Stripe_Upgrader(
'com.drastikbydesign.stripe',
- realpath(__DIR__ . '/../../../')
+ E::path()
);
}
return self::$instance;
@@ -63,15 +64,21 @@ class CRM_Stripe_Upgrader_Base {
* CRM_Stripe_Upgrader_Base::_queueAdapter($ctx, 'methodName', 'arg1', 'arg2');
* @endcode
*/
- static public function _queueAdapter() {
+ public static function _queueAdapter() {
$instance = self::instance();
$args = func_get_args();
$instance->ctx = array_shift($args);
$instance->queue = $instance->ctx->queue;
$method = array_shift($args);
- return call_user_func_array(array($instance, $method), $args);
+ return call_user_func_array([$instance, $method], $args);
}
+ /**
+ * CRM_Stripe_Upgrader_Base constructor.
+ *
+ * @param $extensionName
+ * @param $extensionDir
+ */
public function __construct($extensionName, $extensionDir) {
$this->extensionName = $extensionName;
$this->extensionDir = $extensionDir;
@@ -97,7 +104,7 @@ class CRM_Stripe_Upgrader_Base {
*
* @return bool
*/
- protected static function executeCustomDataFileByAbsPath($xml_file) {
+ protected function executeCustomDataFileByAbsPath($xml_file) {
$import = new CRM_Utils_Migrate_Import();
$import->run($xml_file);
return TRUE;
@@ -119,10 +126,14 @@ class CRM_Stripe_Upgrader_Base {
}
/**
+ * Run the sql commands in the specified file.
+ *
* @param string $tplFile
* The SQL file path (relative to this extension's dir).
* Ex: "sql/mydata.mysql.tpl".
+ *
* @return bool
+ * @throws \CRM_Core_Exception
*/
public function executeSqlTemplate($tplFile) {
// Assign multilingual variable to Smarty.
@@ -141,17 +152,19 @@ class CRM_Stripe_Upgrader_Base {
* Run one SQL query.
*
* This is just a wrapper for CRM_Core_DAO::executeSql, but it
- * provides syntatic sugar for queueing several tasks that
+ * provides syntactic sugar for queueing several tasks that
* run different queries
+ *
+ * @return bool
*/
- public function executeSql($query, $params = array()) {
+ public function executeSql($query, $params = []) {
// FIXME verify that we raise an exception on error
CRM_Core_DAO::executeQuery($query, $params);
return TRUE;
}
/**
- * Syntatic sugar for enqueuing a task which calls a function in this class.
+ * Syntactic sugar for enqueuing a task which calls a function in this class.
*
* The task is weighted so that it is processed
* as part of the currently-pending revision.
@@ -163,11 +176,11 @@ class CRM_Stripe_Upgrader_Base {
$args = func_get_args();
$title = array_shift($args);
$task = new CRM_Queue_Task(
- array(get_class($this), '_queueAdapter'),
+ [get_class($this), '_queueAdapter'],
$args,
$title
);
- return $this->queue->createItem($task, array('weight' => -1));
+ return $this->queue->createItem($task, ['weight' => -1]);
}
// ******** Revision-tracking helpers ********
@@ -200,23 +213,23 @@ class CRM_Stripe_Upgrader_Base {
$currentRevision = $this->getCurrentRevision();
foreach ($this->getRevisions() as $revision) {
if ($revision > $currentRevision) {
- $title = ts('Upgrade %1 to revision %2', array(
+ $title = E::ts('Upgrade %1 to revision %2', [
1 => $this->extensionName,
2 => $revision,
- ));
+ ]);
// note: don't use addTask() because it sets weight=-1
$task = new CRM_Queue_Task(
- array(get_class($this), '_queueAdapter'),
- array('upgrade_' . $revision),
+ [get_class($this), '_queueAdapter'],
+ ['upgrade_' . $revision],
$title
);
$this->queue->createItem($task);
$task = new CRM_Queue_Task(
- array(get_class($this), '_queueAdapter'),
- array('setCurrentRevision', $revision),
+ [get_class($this), '_queueAdapter'],
+ ['setCurrentRevision', $revision],
$title
);
$this->queue->createItem($task);
@@ -231,7 +244,7 @@ class CRM_Stripe_Upgrader_Base {
*/
public function getRevisions() {
if (!is_array($this->revisions)) {
- $this->revisions = array();
+ $this->revisions = [];
$clazz = new ReflectionClass(get_class($this));
$methods = $clazz->getMethods();
@@ -256,7 +269,7 @@ class CRM_Stripe_Upgrader_Base {
private function getCurrentRevisionDeprecated() {
$key = $this->extensionName . ':version';
- if ($revision = CRM_Core_BAO_Setting::getItem('Extension', $key)) {
+ if ($revision = \Civi::settings()->get($key)) {
$this->revisionStorageIsDeprecated = TRUE;
}
return $revision;
@@ -302,7 +315,7 @@ class CRM_Stripe_Upgrader_Base {
$this->executeCustomDataFileByAbsPath($file);
}
}
- if (is_callable(array($this, 'install'))) {
+ if (is_callable([$this, 'install'])) {
$this->install();
}
}
@@ -315,7 +328,7 @@ class CRM_Stripe_Upgrader_Base {
if (!empty($revisions)) {
$this->setCurrentRevision(max($revisions));
}
- if (is_callable(array($this, 'postInstall'))) {
+ if (is_callable([$this, 'postInstall'])) {
$this->postInstall();
}
}
@@ -330,7 +343,7 @@ class CRM_Stripe_Upgrader_Base {
$this->executeSqlTemplate($file);
}
}
- if (is_callable(array($this, 'uninstall'))) {
+ if (is_callable([$this, 'uninstall'])) {
$this->uninstall();
}
$files = glob($this->extensionDir . '/sql/*_uninstall.sql');
@@ -346,7 +359,7 @@ class CRM_Stripe_Upgrader_Base {
*/
public function onEnable() {
// stub for possible future use
- if (is_callable(array($this, 'enable'))) {
+ if (is_callable([$this, 'enable'])) {
$this->enable();
}
}
@@ -356,7 +369,7 @@ class CRM_Stripe_Upgrader_Base {
*/
public function onDisable() {
// stub for possible future use
- if (is_callable(array($this, 'disable'))) {
+ if (is_callable([$this, 'disable'])) {
$this->disable();
}
}
@@ -364,7 +377,7 @@ class CRM_Stripe_Upgrader_Base {
public function onUpgrade($op, CRM_Queue_Queue $queue = NULL) {
switch ($op) {
case 'check':
- return array($this->hasPendingRevisions());
+ return [$this->hasPendingRevisions()];
case 'enqueue':
return $this->enqueuePendingRevisions($queue);
diff --git a/stripe.civix.php b/stripe.civix.php
index 90b91595..b4c715a2 100644
--- a/stripe.civix.php
+++ b/stripe.civix.php
@@ -193,8 +193,9 @@ function _stripe_civix_civicrm_disable() {
* @param $op string, the type of operation being performed; 'check' or 'enqueue'
* @param $queue CRM_Queue_Queue, (for 'enqueue') the modifiable list of pending up upgrade tasks
*
- * @return mixed based on op. for 'check', returns array(boolean) (TRUE if upgrades are pending)
- * for 'enqueue', returns void
+ * @return mixed
+ * based on op. for 'check', returns array(boolean) (TRUE if upgrades are pending)
+ * for 'enqueue', returns void
*
* @link https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_upgrade
*/
@@ -225,7 +226,7 @@ function _stripe_civix_upgrader() {
* @param string $dir base dir
* @param string $pattern , glob pattern, eg "*.txt"
*
- * @return array(string)
+ * @return array
*/
function _stripe_civix_find_files($dir, $pattern) {
if (is_callable(['CRM_Utils_File', 'findFiles'])) {
@@ -244,7 +245,7 @@ function _stripe_civix_find_files($dir, $pattern) {
if ($dh = opendir($subdir)) {
while (FALSE !== ($entry = readdir($dh))) {
$path = $subdir . DIRECTORY_SEPARATOR . $entry;
- if ($entry{0} == '.') {
+ if ($entry[0] == '.') {
}
elseif (is_dir($path)) {
$todos[] = $path;
@@ -255,6 +256,7 @@ function _stripe_civix_find_files($dir, $pattern) {
}
return $result;
}
+
/**
* (Delegated) Implements hook_civicrm_managed().
*
@@ -362,7 +364,7 @@ function _stripe_civix_civicrm_themes(&$themes) {
* @link http://php.net/glob
* @param string $pattern
*
- * @return array, possibly empty
+ * @return array
*/
function _stripe_civix_glob($pattern) {
$result = glob($pattern);
@@ -470,14 +472,12 @@ function _stripe_civix_civicrm_alterSettingsFolders(&$metaDataFolders = NULL) {
*
* @link https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_entityTypes
*/
-
function _stripe_civix_civicrm_entityTypes(&$entityTypes) {
- $entityTypes = array_merge($entityTypes, array (
- 'CRM_Stripe_DAO_StripePaymentintent' =>
- array (
+ $entityTypes = array_merge($entityTypes, [
+ 'CRM_Stripe_DAO_StripePaymentintent' => [
'name' => 'StripePaymentintent',
'class' => 'CRM_Stripe_DAO_StripePaymentintent',
'table' => 'civicrm_stripe_paymentintent',
- ),
- ));
+ ],
+ ]);
}
--
GitLab