revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-25 16:22:43 +01:00
Code Issues Releases Wiki Activity

Remove unused parameter from PhabricatorMailEmailEngine::newEmailThreadingHeaders() call

Browse source 
Summary:
`Method PhabricatorMailEmailEngine::newEmailThreadingHeaders() invoked with 1 parameter, 0 required.`
Thus remove the parameter from its call in the same class.
The called private function does `$mailer = $this->getMailer()` anyway.

Test Plan: Read the code.

Reviewers: O1 Blessed Committers, valerio.bozzolan

Reviewed By: O1 Blessed Committers, valerio.bozzolan

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D25747
This commit is contained in:
Andre Klapper 2024-07-25 20:33:14 +02:00
parent 4e31cadb54
commit c5c2b8ce5a
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/metamta/engine/PhabricatorMailEmailEngine.php
View file

@ -72,7 +72,7 @@ final class PhabricatorMailEmailEngine
$message->setSubject($subject); $message->setSubject($subject);
$headers = $this->newEmailHeaders(); $headers = $this->newEmailHeaders();
foreach ($this->newEmailThreadingHeaders($mailer) as $threading_header) { foreach ($this->newEmailThreadingHeaders() as $threading_header) {
$headers[] = $threading_header; $headers[] = $threading_header;
} }