[ticket/17135] Address review issues but constants

PHPBB-17135

Author: rxu

messenger/method/base.php

@@ -117,8 +117,8 @@ public function __construct(
 		service_collection $twig_extensions_collection,
 		lexer $twig_lexer,
 		user $user,
-		$phpbb_root_path,
-		$template_cache_path,
+		string $phpbb_root_path,
+		string $template_cache_path,
 		?manager $ext_manager = null,
 		?log_interface $log = null
 	)
@@ -154,6 +154,8 @@ abstract public function is_enabled(): bool;
 	/**
 	 * Sets the use of messenger queue flag
 	 *
+	 * @param bool $use_queue Flag indicating if cached queue to be used
+	 *
 	 * @return void
 	 */
 	public function set_use_queue(bool $use_queue = true): void
@@ -426,7 +428,6 @@ public function save_queue(): void
 		if ($this->use_queue && !empty($this->queue))
 		{
 			$this->queue->save();
-			return;
 		}
 	}
 

messenger/method/email.php

@@ -110,9 +110,7 @@ public function init(): void
 	}
 
 	/**
-	 * Sets the use of messenger queue flag
-	 *
-	 * @return void
+	 * {@inheritdoc}
 	 */
 	public function set_use_queue(bool $use_queue = true): void
 	{
@@ -124,7 +122,7 @@ public function set_use_queue(bool $use_queue = true): void
 	 */
 	public function set_addresses(array $user_row): void
 	{
-		if (isset($user_row['user_email']) && $user_row['user_email'])
+		if (!empty($user_row['user_email']))
 		{
 			$this->to($user_row['user_email'], $user_row['username'] ?: '');
 		}
@@ -295,8 +293,8 @@ protected function build_headers(): void
 			'Return-Path'		=> new Address($this->config['board_email']),
 			'Sender'			=> new Address($this->config['board_email']),
 			'X-MSMail-Priority'	=> self::PRIORITY_MAP[$this->mail_priority],
-			'X-Mailer'			=> 'phpBB3',
-			'X-MimeOLE'			=> 'phpBB3',
+			'X-Mailer'			=> 'phpBB',
+			'X-MimeOLE'			=> 'phpBB',
 			'X-phpBB-Origin'	=> 'phpbb://' . str_replace(['http://', 'https://'], ['', ''], generate_board_url()),
 		];
 
@@ -472,20 +470,23 @@ public function send(): bool
 		$this->email->subject($this->subject);
 		$this->email->text($this->msg);
 
+		$break = false;
 		$subject = $this->subject;
 		$msg = $this->msg;
 		$email = $this->email;
 		/**
 		 * Event to send message via external transport
 		 *
 		 * @event core.notification_message_email
+		 * @var	bool	break	Flag indicating if the function return after hook
 		 * @var	string	subject	The message subject
 		 * @var	string	msg		The message text
 		 * @var	string	email	The Symfony Email object
 		 * @since 3.2.4-RC1
-		 * @changed 4.0.0-a1 Added vars: email. Removed vars: addresses, break
+		 * @changed 4.0.0-a1 Added vars: email. Removed vars: addresses
 		 */
 		$vars = [
+			'break',
 			'subject',
 			'msg',
 			'email',
@@ -495,6 +496,11 @@ public function send(): bool
 
 		$this->build_headers();
 
+		if ($break)
+		{
+			return true;
+		}
+
 		// Send message ...
 		if (!$this->use_queue)
 		{

messenger/method/jabber.php

@@ -191,7 +191,7 @@ public function ssl(bool $use_ssl = false): self
 	 */
 	public function port(int $port = 5222): self
 	{
-		$this->port	= ($port) ? $port : 5222;
+		$this->port	= ($port > 0) ? $port : 5222;
 
 		// Change port if we use SSL
 		if ($this->port == 5222 && $this->use_ssl)
@@ -210,14 +210,14 @@ public function port(int $port = 5222): self
 	 */
 	public function username(string $username = ''): self
 	{
-		if (strpos($username, '@') === false)
+		if (str_contains($username, '@'))
 		{
-			$this->username = $username;
+			$this->jid = explode('@', $username, 2);
+			$this->username = $this->jid[0];
 		}
 		else
 		{
-			$this->jid = explode('@', $username, 2);
-			$this->username = $this->jid[0];
+			$this->username = $username;
 		}
 
 		return $this;
@@ -232,7 +232,7 @@ public function username(string $username = ''): self
 	 */
 	public function server(string $server = ''): self
 	{
-		$this->connect_server = ($server) ? $server : 'localhost';
+		$this->connect_server = $server ?: 'localhost';
 		$this->server = $this->jid[1] ?? $this->connect_server;
 
 		return $this;
@@ -407,7 +407,7 @@ public function to(string $address, string $realname = ''): void
 	}
 
 	/**
-	 * {@inheritDoc}
+	 * Resets all the data (address, template file, etc) to default
 	 */
 	public function reset(): void
 	{
@@ -663,7 +663,7 @@ public function listen(int $timeout = 10, bool $wait = false): bool|array
 	}
 
 	/**
-	 * Initiates account registration (based on data used for contructor)
+	 * Initiates account registration (based on data used for constructor)
 	 *
 	 * @return bool|null
 	 */
@@ -714,7 +714,7 @@ function send_presence(string $message = '', string $type = '', bool $unavailabl
 	 */
 	function response(array $xml): bool
 	{
-		if (!is_array($xml) || !count($xml))
+		if (!count($xml))
 		{
 			return false;
 		}

notification/method/messenger_base.php

@@ -112,7 +112,6 @@ protected function notify_using_messenger($notify_method, $template_dir_prefix =
 				continue;
 			}
 
-			/** @psalm-suppress InvalidTemplateParam */
 			$messenger_collection_iterator = $this->messenger->getIterator();
 			foreach ($messenger_collection_iterator as $messenger_method)
 			{

template/twig/extension.php

@@ -14,6 +14,9 @@
 namespace phpbb\template\twig;
 
 use Twig\Error\RuntimeError;
+use Twig\Extension\CoreExtension;
+use Twig\Extension\EscaperExtension;
+use Twig\Runtime\EscaperRuntime;
 
 class extension extends \Twig\Extension\AbstractExtension
 {
@@ -164,8 +167,7 @@ public function loop_subset(\Twig\Environment $env, $item, $start, $end = null,
 		// We always include the last element (this was the past design)
 		$end = ($end == -1 || $end === null) ? null : $end + 1;
 
-		/** @psalm-suppress UndefinedFunction */
-		return twig_slice($env, $item, $start, $end, $preserveKeys);
+		return CoreExtension::slice($env->getCharset(), $item, $start, $end, $preserveKeys);
 	}
 
 	/**
@@ -214,8 +216,7 @@ public function lang_js(): string
 	{
 		$args = func_get_args();
 
-		/** @psalm-suppress UndefinedFunction */
-		return twig_escape_filter($this->environment, call_user_func_array([$this, 'lang'], $args), 'js');
+		return $this->environment->getRuntime(EscaperRuntime::class)->escape(call_user_func_array([$this, 'lang'], $args), 'js');
 	}
 
 	/**