Page MenuHomeIn-Portal Phabricator

INP-1808 - Create "E-mail Log" records using corresponding unit
ClosedPublic

Authored by alex on Jul 1 2021, 11:49 AM.

Details

Test Plan
  1. login to the Admin Console
  2. remove all records from Logs & ReportsE-mail Log and User ManagementMailingsEmail Queue (tab)
  3. go to User ManagementUsers section
  4. create a record if none present
  5. try disabling any enabled record (or enabled any disabled record if all records are disabled)
  6. go to User ManagementMailingsEmail Queue (tab) section
  7. confirm, that a queue record was added into it
  8. press Process Queue toolbar button
  9. confirmed that queued record went away
  10. go to Logs & ReportsE-mail Log section
  11. confirm, that e-mail record was created as part of queued record processing

Diff Detail

Repository
rINP In-Portal
Branch
/in-portal/branches/5.2.x
Lint
Lint ErrorsExcuse: Not fixing issues introduced before these changes.
SeverityLocationCodeMessage
Errorcore/kernel/utility/email_send.php:1215PHPCS.E.CodingStandard.Arrays.Array.SpaceAfterKeywordCodingStandard.Arrays.Array.SpaceAfterKeyword
Errorcore/kernel/utility/email_send.php:1215PHPCS.E.Generic.PHP.LowerCaseKeyword.FoundGeneric.PHP.LowerCaseKeyword.Found
Errorcore/units/logs/email_logs/email_logs_config.php:92PHPCS.E.CodingStandard.Arrays.Array.SpaceAfterKeywordCodingStandard.Arrays.Array.SpaceAfterKeyword
Errorcore/units/logs/email_logs/email_logs_config.php:92PHPCS.E.Generic.PHP.LowerCaseConstant.FoundGeneric.PHP.LowerCaseConstant.Found
Errorcore/units/logs/email_logs/email_logs_config.php:92PHPCS.E.Generic.PHP.LowerCaseKeyword.FoundGeneric.PHP.LowerCaseKeyword.Found
Errorcore/units/logs/email_logs/email_logs_config.php:93PHPCS.E.CodingStandard.Arrays.Array.SpaceAfterKeywordCodingStandard.Arrays.Array.SpaceAfterKeyword
Errorcore/units/logs/email_logs/email_logs_config.php:93PHPCS.E.Generic.PHP.LowerCaseKeyword.FoundGeneric.PHP.LowerCaseKeyword.Found
Unit
No Unit Test Coverage
Build Status
Buildable 1049
Build 1049: arc lint + arc unit

Event Timeline

alex created this revision.Jul 1 2021, 11:49 AM
alex requested review of this revision.Jul 1 2021, 11:49 AM
alex updated this revision to Diff 1021.Jul 1 2021, 12:01 PM

Remove temporary object used for e-mail log record creation to avoid memory leaks.

erik accepted this revision.Aug 6 2021, 9:19 AM
This revision is now accepted and ready to land.Aug 6 2021, 9:19 AM
alex retitled this revision from INP-1808 - Create "E-mail Log" records unit corresponding unit to INP-1808 - Create "E-mail Log" records using corresponding unit.Jul 8 2022, 7:56 AM
This revision was landed with ongoing or failed builds.Jul 8 2022, 7:59 AM
This revision was automatically updated to reflect the committed changes.