Page MenuHomeIn-Portal Phabricator

INP-1960 - Exceptions thrown in CLI aren't logged into the System Log
ClosedPublic

Authored by alex on Apr 13 2026, 3:52 AM.

Details

Test Plan

Preparations

  • in the IDE:
    1. open the core/units/admin/admin_events_handler.php file for editing
    2. in the \AdminEventsHandler::OnDeploy method add the throw new LogicException('Testing.'); line before the current code
    3. save changes

Test Plan

  • in CLI:
    1. execute the ./in-portal event:run adm:OnDeploy -v command
    2. confirm that:
      • exception class/message/backtrace is displayed
  • login to the Admin Console
    1. go to the Logs & ReportsSystem Log section
    2. confirm that the above-thrown exception was properly logged

Diff Detail

Repository
rINP In-Portal
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alex created this revision.Apr 13 2026, 3:52 AM
alex requested review of this revision.Apr 13 2026, 3:52 AM
alex updated this revision to Diff 1468.Apr 13 2026, 4:00 AM

Removed the visuals-related part.

alex edited the test plan for this revision. (Show Details)Apr 13 2026, 4:22 AM
erik accepted this revision.Apr 13 2026, 5:49 AM
This revision is now accepted and ready to land.Apr 13 2026, 5:49 AM
alex updated this revision to Diff 1471.Apr 13 2026, 6:04 AM

Changed comment phrasing.

This revision was automatically updated to reflect the committed changes.