Refer to service container property instead of using global helper
ClosedPublic
Actions

Authored by dereckson on Apr 23 2022, 08:34.

Tags

None

Referenced Files

	F2830999: D2677.id6770.diff
	Fri, Apr 19, 14:38

	Unknown Object (File)
	Thu, Apr 18, 16:27

	Unknown Object (File)
	Thu, Apr 18, 13:19

	Unknown Object (File)
	Mon, Apr 15, 05:22

	Unknown Object (File)
	Mon, Apr 15, 02:56

	Unknown Object (File)
	Sat, Apr 13, 08:07

	Unknown Object (File)
	Wed, Apr 10, 13:11

	Unknown Object (File)
	Tue, Apr 9, 18:38

View All Files

Subscribers

None

Details

Reviewers

DorianWinty
dereckson

Commits

rNOTIF5365c197439c: Refer to service container property instead of using global helper

Summary

The Illuminate\Foundation\Exceptions\Handler constructor requires
the full container in the constructor, using a service locator pattern.

Test Plan

Run tests

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Apr 23 2022, 08:34

dereckson created this revision.

Harbormaster completed remote builds in B4206: Diff 6770.Apr 23 2022, 08:34

DorianWinty accepted this revision.Apr 23 2022, 12:08

This revision is now accepted and ready to land.Apr 23 2022, 12:08

dereckson accepted this revision.May 7 2022, 19:25

This revision was landed with ongoing or failed builds.May 7 2022, 19:31

Closed by commit rNOTIF5365c197439c: Refer to service container property instead of using global helper (authored by dereckson). · Explain Why

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rNOTIF5365c197439c: Refer to service container property instead of using global helper.

Revision Contents
Changeset List

Path

Size

tests/

Exceptions/

HandlerTest.php

2 lines

Diff 6783

View Options

tests/Exceptions/HandlerTest.php

Refer to service container property instead of using global helperClosedPublicActions