From a2f99248cb5550ac7641676457a8450c8d499d1a Mon Sep 17 00:00:00 2001
From: rxu <rxu@mail.ru>
Date: Wed, 8 Jan 2025 10:27:55 +0700
Subject: [PATCH] [ticket/17135] Optimize save_queue() method call for
 acp_inactive

PHPBB-17135
---
 phpBB/includes/acp/acp_inactive.php | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/phpBB/includes/acp/acp_inactive.php b/phpBB/includes/acp/acp_inactive.php
index b9ff966b6d..2436277e15 100644
--- a/phpBB/includes/acp/acp_inactive.php
+++ b/phpBB/includes/acp/acp_inactive.php
@@ -227,6 +227,11 @@ class acp_inactive
 						}
 						while ($row = $db->sql_fetchrow($result));
 
+						foreach ($messenger_collection_iterator as $messenger_method)
+						{
+							$messenger_method->save_queue();
+						}
+
 						// Add the remind state to the database and increase activation expiration by one day
 						$sql = 'UPDATE ' . USERS_TABLE . '
 							SET user_reminded = user_reminded + 1,
@@ -241,11 +246,6 @@ class acp_inactive
 					}
 					$db->sql_freeresult($result);
 
-					foreach ($messenger_collection_iterator as $messenger_method)
-					{
-						$messenger_method->save_queue();
-					}
-
 					// For remind we really need to redirect, else a refresh can result in more than one reminder
 					$u_action = $this->u_action . "&amp;$u_sort_param&amp;start=$start";
 					$u_action .= ($per_page != $config['topics_per_page']) ? "&amp;users_per_page=$per_page" : '';