[ticket/17489] Ignore Psalm UndefinedMethod issue

Psalm fails with UndefinedMethod issue when a method of a child class
does not exist in parent/abstract/interface class. Ignore the issue.

PHPBB-17489
This commit is contained in:
rxu 2025-04-03 20:57:24 +07:00
parent c23504e3c7
commit d99c3838df
No known key found for this signature in database
GPG key ID: 955F0567380E586A
7 changed files with 36 additions and 9 deletions

View file

@ -227,7 +227,10 @@ class acp_email
$used_lang = $email_list[$i][0]['lang']; $used_lang = $email_list[$i][0]['lang'];
$used_method = $email_list[$i][0]['method']; $used_method = $email_list[$i][0]['method'];
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
$notify_method = $messenger_method->get_id(); $notify_method = $messenger_method->get_id();
@ -273,7 +276,10 @@ class acp_email
if ($use_queue) if ($use_queue)
{ {
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
$messenger_method->save_queue(); $messenger_method->save_queue();

View file

@ -205,7 +205,10 @@ class acp_inactive
do do
{ {
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
if ($messenger_method->get_id() == $user_row['user_notify_type'] || $user_row['user_notify_type'] == $messenger_method::NOTIFY_BOTH) if ($messenger_method->get_id() == $user_row['user_notify_type'] || $user_row['user_notify_type'] == $messenger_method::NOTIFY_BOTH)
@ -228,7 +231,10 @@ class acp_inactive
} }
while ($row = $db->sql_fetchrow($result)); while ($row = $db->sql_fetchrow($result));
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
$messenger_method->save_queue(); $messenger_method->save_queue();

View file

@ -134,7 +134,10 @@ class ucp_activate
$messenger = $phpbb_container->get('messenger.method_collection'); $messenger = $phpbb_container->get('messenger.method_collection');
$messenger_collection_iterator = $messenger->getIterator(); $messenger_collection_iterator = $messenger->getIterator();
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
if ($messenger_method->get_id() == $user_row['user_notify_type'] || $user_row['user_notify_type'] == $messenger_method::NOTIFY_BOTH) if ($messenger_method->get_id() == $user_row['user_notify_type'] || $user_row['user_notify_type'] == $messenger_method::NOTIFY_BOTH)

View file

@ -138,7 +138,10 @@ class ucp_resend
$messenger_collection_iterator = $messenger->getIterator(); $messenger_collection_iterator = $messenger->getIterator();
while ($row = $db->sql_fetchrow($result)) while ($row = $db->sql_fetchrow($result))
{ {
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
$messenger_method->set_use_queue(false); $messenger_method->set_use_queue(false);

View file

@ -250,7 +250,10 @@ class message
/** @psalm-suppress InvalidTemplateParam */ /** @psalm-suppress InvalidTemplateParam */
$messenger_collection_iterator = $messenger->getIterator(); $messenger_collection_iterator = $messenger->getIterator();
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
$messenger_method->set_use_queue(false); $messenger_method->set_use_queue(false);

View file

@ -117,7 +117,10 @@ class queue
/** @psalm-suppress InvalidTemplateParam */ /** @psalm-suppress InvalidTemplateParam */
$messenger_collection_iterator = $this->messenger_method_collection->getIterator(); $messenger_collection_iterator = $this->messenger_method_collection->getIterator();
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
if (isset($this->queue_data[$messenger_method->get_queue_object_name()])) if (isset($this->queue_data[$messenger_method->get_queue_object_name()]))

View file

@ -114,7 +114,10 @@ abstract class messenger_base extends \phpbb\notification\method\base
continue; continue;
} }
/** @var \phpbb\messenger\method\base $messenger_method */ /**
* @var \phpbb\messenger\method\messenger_interface $messenger_method
* @psalm-suppress UndefinedMethod
*/
foreach ($messenger_collection_iterator as $messenger_method) foreach ($messenger_collection_iterator as $messenger_method)
{ {
if ($messenger_method->get_id() == $notify_method || $notify_method == $messenger_method::NOTIFY_BOTH) if ($messenger_method->get_id() == $notify_method || $notify_method == $messenger_method::NOTIFY_BOTH)