Page MenuHomePhorge

replace usage of each() with foreach() for smtp

Authored by MacFan4000 on Nov 27 2022, 21:20.
Referenced Files
Unknown Object (File)
Mon, May 8, 13:29
Unknown Object (File)
Mon, May 8, 10:33
Unknown Object (File)
Apr 30 2023, 18:51
Unknown Object (File)
Apr 26 2023, 20:46
Unknown Object (File)
Apr 26 2023, 20:46
Unknown Object (File)
Apr 25 2023, 21:32
Unknown Object (File)
Apr 23 2023, 13:14
Unknown Object (File)
Mar 28 2023, 08:38



this replaces uses with warning suppression of each() (depreccated in
PHP7, removed in PHP8) with foreach

Test Plan

can verify that these chages do work (tested on my own install)

Diff Detail

rP Phorge
No Lint Coverage
No Test Coverage
Build Status
Buildable 104
Build 104: arc lint + arc unit

Event Timeline

  1. Is this even used?
  2. Looks like the file is currently in Windows mode; can you please split this to 2 diffs, or just keep the original line-ending?

Yes it is used for mailing via smtp, so probably most installs use it

This revision is now accepted and ready to land.Feb 2 2023, 19:05

If somebody could land this at some point, that would be great!

I will land since the author asked this kind of help, and avivey confirmed, and also asked to keep the same "no newline at end of file" and that's respected.