INP-1804 - Don't reset current user after all Scheduled Tasks were executed
Needs RevisionPublic
Actions

Authored by alex on May 13 2021, 4:33 AM.

Details

Reviewers

erik

Test Plan

login to Admin Console
go to Configuration → Website → Advanced section
enable Use Cron to run Scheduled Tasks setting and save changes
in IDE change contents of \AdminEventsHandler::OnBeforeShutdown method (file: core/units/admin/admin_events_handler.php) to be die('user_id: ' . $this->Application->RecallVar('user_id')); and save changes
execute php tools/cron.php from root folder of a project
confirm, that you'll see either user_id: -1 or user_id: -3 as output of that script

Repository

rINP In-Portal

Branch

/in-portal/branches/5.2.x

Lint

Lint OK

Unit

No Unit Test Coverage

Build Status

Buildable 1043
Build 1043: arc lint + arc unit

alex requested review of this revision.May 13 2021, 4:33 AM

Harbormaster completed remote builds in B1043: Diff 1015.

alex added a project: Restricted Project.May 13 2021, 4:33 AM

No output detected after doing test plan.

This revision now requires changes to proceed.May 18 2021, 5:09 AM

			Path	Packages
M			core/kernel/managers/scheduled_task_manager.php (2 lines)

Commit	Parents	Author	Summary	Date
#fake-hash			Ignore it :) (Show More…)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	1015	16691		May 13 2021, 4:33 AM	★	★