From dcc271b636eacf79861508376195a8d1f57443c4 Mon Sep 17 00:00:00 2001
From: Daimona Eaytoy <daimona.wiki@gmail.com>
Date: Wed, 21 Aug 2024 14:31:46 +0200
Subject: [PATCH] Api: Avoid type error in AbuseLogPrivateDetails

Make the `reason` parameter default to the empty string, so that we
don't end up passing null to ManualLogEntry::setComment.

Bug: T373010
Change-Id: Ifca828401628368bdddae14df2bbeb7391b2c02d
---
 includes/Api/AbuseLogPrivateDetails.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/includes/Api/AbuseLogPrivateDetails.php b/includes/Api/AbuseLogPrivateDetails.php
index 9378eb9a8..53d44f289 100644
--- a/includes/Api/AbuseLogPrivateDetails.php
+++ b/includes/Api/AbuseLogPrivateDetails.php
@@ -124,6 +124,7 @@ class AbuseLogPrivateDetails extends ApiBase {
 			'reason' => [
 				ParamValidator::PARAM_TYPE => 'string',
 				ParamValidator::PARAM_REQUIRED => $this->getConfig()->get( 'AbuseFilterPrivateDetailsForceReason' ),
+				ParamValidator::PARAM_DEFAULT => '',
 			]
 		];
 	}