You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
47 lines
1.5 KiB
47 lines
1.5 KiB
From 1acd52cff0d516ebfa7f177c38ed7f851c4f472a Mon Sep 17 00:00:00 2001
|
|
From: Michael Orlitzky <michael@orlitzky.com>
|
|
Date: Wed, 16 Aug 2023 21:21:39 -0400
|
|
Subject: [PATCH] includes/batch.inc: synchronize sub and superclass signatures
|
|
|
|
The DrushBatchContext class overrides the offsetSet() method of its
|
|
superclass, ArrayObject, but changes its signature:
|
|
|
|
Error: Return type of DrushBatchContext::offsetSet($name, $value)
|
|
should either be compatible with ArrayObject::offsetSet(mixed $key,
|
|
mixed $value): void, or the #[\ReturnTypeWillChange] attribute should
|
|
be used to temporarily suppress the notice in .../batch.inc, line 37
|
|
|
|
Renaming the parameter to $key and adding type hints fixes the issue.
|
|
|
|
Signed-off-by: Michael Orlitzky <michael@orlitzky.com>
|
|
---
|
|
includes/batch.inc | 8 ++++----
|
|
1 file changed, 4 insertions(+), 4 deletions(-)
|
|
|
|
diff --git a/includes/batch.inc b/includes/batch.inc
|
|
index dd4adc9..d26fabb 100644
|
|
--- a/includes/batch.inc
|
|
+++ b/includes/batch.inc
|
|
@@ -34,14 +34,14 @@
|
|
* @see _drush_batch_worker().
|
|
*/
|
|
class DrushBatchContext extends ArrayObject {
|
|
- function offsetSet($name, $value) {
|
|
- if ($name == 'message') {
|
|
+ function offsetSet(mixed $key, mixed $value): void {
|
|
+ if ($key == 'message') {
|
|
drush_log(strip_tags($value), 'ok');
|
|
}
|
|
- elseif ($name == 'error_message') {
|
|
+ elseif ($key == 'error_message') {
|
|
drush_set_error('DRUSH_BATCH_ERROR', strip_tags($value));
|
|
}
|
|
- parent::offsetSet($name, $value);
|
|
+ parent::offsetSet($key, $value);
|
|
}
|
|
}
|
|
|
|
--
|
|
2.43.2
|
|
|