Differential D2677

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

	Unknown Object (File)
	Sat, Dec 7, 00:20

	Unknown Object (File)
	Wed, Dec 4, 14:38

	Unknown Object (File)
	Sun, Dec 1, 22:59

	Unknown Object (File)
	Sun, Dec 1, 22:40

	Unknown Object (File)
	Nov 22 2024, 21:04

	Unknown Object (File)
	Nov 22 2024, 16:17

	Unknown Object (File)
	Nov 22 2024, 15:48

	Unknown Object (File)
	Nov 22 2024, 10:37

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 Passed

Unit

Test Failures

Branch

fix-handlertest

Build Status

Buildable 4206
Build 4458: arc lint + arc unit

Unit TestsBroken
View All

	Time	Test
	0 ms	/usr/home/dereckson/dev/nasqueron/notifications/notifications/tests/Exceptions/HandlerTest.php Fatal error: Declaration of Nasqueron\Notifications\Tests\Exceptions\HandlerTest::setUp() must be compatible with Illuminate\Foundation\Testing\TestCase::setUp(): void in /usr/home/dereckson/dev/nasqueron/notifications/notifications/tests/Exceptions/HandlerTest.php on line 25

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 6770

tests/Exceptions/HandlerTest.php

Loading...