Paths

Table of Contentst

Differential D2674

Update Laravel 8
ClosedPublic
Actions

Authored by DorianWinty on Apr 16 2022, 22:51.

Tags

Referenced Files

	Unknown Object (File)
	Fri, Dec 20, 04:41

	Unknown Object (File)
	Fri, Dec 20, 04:41

	Unknown Object (File)
	Fri, Dec 20, 04:40

	Unknown Object (File)
	Fri, Dec 20, 04:40

	Unknown Object (File)
	Fri, Dec 20, 04:39

	Unknown Object (File)
	Fri, Dec 20, 04:39

	Unknown Object (File)
	Fri, Dec 20, 04:39

	Unknown Object (File)
	Thu, Dec 19, 15:26

Subscribers

Tokens

"Yellow Medal" token, awarded by dereckson.

Details

Reviewers

Maniphest Tasks

T1725: Upgrade Laravel

Commits

rNOTIF327396a753b0: Update Laravel 8

Summary

Update to Laravel 8

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Errors

Severity	Location	Code	Message
Error	tests/Analyzers/GitHub/Events/EventTest.php:58	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/EventTest.php:72	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/IssueCommentEventTest.php:38	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/IssueCommentEventTest.php:39	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/PullRequestEventTest.php:38	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/PullRequestEventTest.php:39	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/PullRequestEventTest.php:41	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/PushEventTest.php:42	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/PushEventTest.php:57	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Analyzers/GitHub/Events/PushEventTest.php:58	PHPCS.E.Generic.Files.LineLength.MaxExceeded	Generic.Files.LineLength.MaxExceeded
Error	tests/Http/PlaceholderTest.php:12	PHPCS.E.Generic.Functions.OpeningFunctionBraceKernighanRitchie.BraceOnNewLine	Generic.Functions.OpeningFunctionBraceKernighanRitchie.BraceOnNewLine
Error	tests/Http/StatusTest.php:12	PHPCS.E.Generic.Functions.OpeningFunctionBraceKernighanRitchie.BraceOnNewLine	Generic.Functions.OpeningFunctionBraceKernighanRitchie.BraceOnNewLine
Error	tests/Providers/ConfigTest.php:38	PHPCS.E.Generic.Strings.UnnecessaryStringConcat.Found	Generic.Strings.UnnecessaryStringConcat.Found
Error	tests/Providers/EventServiceProviderTest.php:34	PHPCS.E.Generic.Strings.UnnecessaryStringConcat.Found	Generic.Strings.UnnecessaryStringConcat.Found
Error	tests/TestCase.php:27	PHPCS.E.Generic.Functions.OpeningFunctionBraceKernighanRitchie.BraceOnNewLine	Generic.Functions.OpeningFunctionBraceKernighanRitchie.BraceOnNewLine
Warning	app/Providers/RouteServiceProvider.php:37	PHPCS.W.Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClass	Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClass

Unit

Tests Passed

Branch

arcpatch-D2674

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Harbormaster completed remote builds in B4197: Diff 6758.Apr 18 2022, 17:39

correct arc lint bug

Harbormaster completed remote builds in B4198: Diff 6759.Apr 18 2022, 20:37

DorianWinty added a project: Restricted Project.Apr 19 2022, 07:43

DorianWinty edited the summary of this revision. (Show Details)Apr 19 2022, 07:59

DorianWinty added a task: T1725: Upgrade Laravel.

adding codestyle
adding on the up of file the class that is use
adding using str_ instead of str::

Harbormaster completed remote builds in B4199: Diff 6760.Apr 19 2022, 18:45

wip

Harbormaster completed remote builds in B4200: Diff 6763.Apr 19 2022, 20:09

removing last str::

Harbormaster completed remote builds in B4202: Diff 6766.Apr 19 2022, 22:29

dereckson added a subscriber: dereckson.Apr 20 2022, 08:44

dereckson added inline comments.

app/Actions/AMQPAction.php
34 ↗	(On Diff #6766)	Not in the scope of the upgrade. Those esthetic changes are welcome, but: it's the other way around, ie the PREVIOUS `public function __construct )` was the correct one it should be done in a separate change against main branch (we can merge esthetic changes before upgrade without any issue)
app/Actions/Action.php
20 ↗	(On Diff #6766)	Probably created by a `phpcbf` with MediaWiki setting. The previous code was correct.
33 ↗	(On Diff #6766)	Same issue, previous file was fully correct in style.

correction err

Harbormaster completed remote builds in B4203: Diff 6767.Apr 22 2022, 15:50

correct tests
update some packages

Harbormaster completed remote builds in B4204: Diff 6768.Apr 22 2022, 22:10

correcting some Tests

Harbormaster completed remote builds in B4214: Diff 6779.May 4 2022, 19:39

correct to make work the tests

Harbormaster completed remote builds in B4215: Diff 6781.May 7 2022, 19:24

DorianWinty published this revision for review.May 7 2022, 19:31

DorianWinty retitled this revision from WIP Update Laravel 8 to Update Laravel 8.

modify test
remove unused use

Harbormaster completed remote builds in B4217: Diff 6785.May 7 2022, 20:44

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/253/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/253/console

adding back one removed file

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/254/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/254/console

Harbormaster failed remote builds in B4222: Diff 6790!May 8 2022, 14:52

dereckson mentioned this in D2683: Modernize build.xml.May 8 2022, 15:17

rebased

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/256/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/256/console

Harbormaster failed remote builds in B4225: Diff 6796!May 8 2022, 17:01

dereckson requested changes to this revision.May 8 2022, 17:33

dereckson added inline comments.

app/Analyzers/GitHub/Events/Event.php
34	Extraneous added line
48	extraneous added line
app/Analyzers/GitHub/Events/IssueCommentEvent.php
20	MediaWiki style
app/Analyzers/GitHub/Events/PullRequestEvent.php
18	MediaWiki
tests/TestCase.php
5	Restore that division Our app External libs Global space

This revision now requires changes to proceed.May 8 2022, 17:33

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/257/ for more details.

DorianWinty retitled this revision from Update Laravel 8 to WIP Update Laravel 8.May 9 2022, 18:14

modify folowing comments but need more to reading

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/258/ for more details.

Harbormaster completed remote builds in B4228: Diff 6799.May 9 2022, 18:31

DorianWinty retitled this revision from WIP Update Laravel 8 to Update Laravel 8.May 9 2022, 18:31

correcting style problem due to mediawiki phpcs style

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/259/ for more details.

Harbormaster completed remote builds in B4229: Diff 6800.May 10 2022, 17:38

correcting MW style to old style

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/260/ for more details.

Harbormaster completed remote builds in B4230: Diff 6801.May 10 2022, 17:54

phpcbf

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/261/ for more details.

Harbormaster completed remote builds in B4231: Diff 6802.May 10 2022, 18:05

dereckson requested changes to this revision.May 12 2022, 11:37

dereckson added inline comments.

app/Config/Reporting/BaseReportEntry.php
11–12	"public abstract" is more convenient as it's coherent with "public static": That would mean an order `<visibility> [special status] function <function name>`. More generally, run automated tools and accept ANY modification they do is a probably not a good idea, `git add -p` to filter the useful ones is very convenient with those "magic" tools.
app/Providers/RouteServiceProvider.php
36–37	MediaWiki style
composer.json
23–24	Two needs to be restored now we're up to date: phploc phpcpd
53	For phpmd, it's well ruleset.xml
tests/Analyzers/GitHub/GitHubPayloadAnalyzerTest.php
68	MediaWiki style
tests/Analyzers/Phabricator/PhabricatorPayloadAnalyzerTest.php
90	MW
tests/Config/FeaturesTest.php
25	MW
tests/Console/Commands/NotificationsPayloadTest.php
49	MW (the two lines)
tests/Notifications/DockerHubNotificationTest.php
12	Extraneous line

This revision now requires changes to proceed.May 12 2022, 11:37

correction with the comments

Build is green
See https://ci.nasqueron.org/job/test-notifications-phab/262/ for more details.

Harbormaster completed remote builds in B4235: Diff 6806.May 13 2022, 06:54

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

dereckson awarded a token.

This revision is now accepted and ready to land.May 21 2022, 19:25

This revision was landed with ongoing or failed builds.May 21 2022, 20:02

Closed by commit rNOTIF327396a753b0: Update Laravel 8 (authored by DorianWinty). · Explain Why

This revision was automatically updated to reflect the committed changes.

DorianWinty added a commit: rNOTIF327396a753b0: Update Laravel 8.

DorianWinty mentioned this in D2673: WIP Update Laravel 5.8.May 21 2022, 20:10

Large Diff

This large diff affects 116 files. Files without inline comments have been collapsed. Expand All Files

Revision Contents
Changeset List

Path

Size

8 lines

app/

Analyzers/

BasePayloadAnalyzer.php

9 lines

DockerHub/

BuildFailureEvent.php

2 lines

GitHub/

Events/

20 lines

IssueCommentEvent.php

3 lines

PullRequestEvent.php

2 lines

2 lines

4 lines

WithRepoAndBranch.php

2 lines

GitHubPayloadAnalyzer.php

2 lines

ItemGroupMapping.php

4 lines

Jenkins/

JenkinsPayloadAnalyzer.php

2 lines

Config/

2 lines

Reporting/

BaseReportEntry.php

4 lines

ConfigReport.php

4 lines

ServiceReportEntry.php

2 lines

Services/

2 lines

Console/

Commands/

ConfigValidate.php

3 lines

PhabricatorProjectsMap.php

4 lines

Contracts/

4 lines

2 lines

Events/

DockerHubPayloadEvent.php

1 line

GitHubPayloadEvent.php

1 line

JenkinsPayloadEvent.php

1 line

NotificationEvent.php

1 line

PhabricatorPayloadEvent.php

2 lines

ReportEvent.php

2 lines

Exceptions/

15 lines

Facades/

3 lines

Http/

Controllers/

8 lines

Gate/

DockerHubGateController.php

7 lines

GateController.php

13 lines

GitHubGateController.php

8 lines

JenkinsGateController.php

7 lines

NotificationGateController.php

14 lines

PhabricatorGateController.php

7 lines

4 lines

Jobs/

FireDockerHubNotification.php

9 lines

FireGitHubNotification.php

9 lines

FireJenkinsNotification.php

9 lines

FirePhabricatorNotification.php

11 lines

NotifyNewCommitsToDiffusion.php

9 lines

SendMessageToBroker.php

9 lines

TriggerDockerHubBuild.php

7 lines

Listeners/

AMQPEventListener.php

24 lines

DockerHubListener.php

25 lines

LastPayloadSaver.php

26 lines

NotificationListener.php

40 lines

PhabricatorListener.php

21 lines

Phabricator/

PhabricatorAPI.php

3 lines

PhabricatorStory.php

2 lines

ProjectsMap.php

4 lines

ProjectsMapFactory.php

4 lines

Providers/

DockerHubServiceProvider.php

5 lines

EventServiceProvider.php

55 lines

MailgunServiceProvider.php

6 lines

PhabricatorAPIServiceProvider.php

2 lines

PhabricatorProjectsMapServiceProvider.php

2 lines

ReportServiceProvider.php

2 lines

RouteServiceProvider.php

10 lines

35 lines

config/

20 lines

8 lines

tests/

Actions/

AMQPActionTest.php

4 lines

ActionErrorTest.php

4 lines

ActionsReportTest.php

12 lines

NotifyNewCommitsActionTest.php

4 lines

TriggerDockerHubBuildActionTest.php

4 lines

Analyzers/

GitHub/

Events/

CreateEventTest.php

8 lines

DeleteEventTest.php

8 lines

6 lines

IssueCommentEventTest.php

2 lines

PullRequestEventTest.php

2 lines

PushEventTest.php

6 lines

RepositoryEventTest.php

10 lines

StatusEventTest.php

2 lines

UnknownEventTest.php

2 lines

GitHubPayloadAnalyzerTest.php

14 lines

Jenkins/

JenkinsPayloadAnalyzerConfigurationTest.php

6 lines

JenkinsPayloadAnalyzerTest.php

5 lines

PayloadAnalyzerConfigurationTest.php

6 lines

Phabricator/

PhabricatorGroupMappingTest.php

2 lines

PhabricatorPayloadAnalyzerTest.php

8 lines

WithConfiguration.php

1 line

Config/

FeaturesTest.php

2 lines

Reporting/

FeatureReportEntryTest.php

2 lines

IntegrationTest.php

2 lines

ServiceReportEntryTest.php

3 lines

Services/

ServiceTest.php

2 lines

ServicesTest.php

2 lines

Console/

Commands/

ConfigShowTest.php

4 lines

ConfigValidateTest.php

4 lines

NotificationsPayloadTest.php

8 lines

PhabricatorProjectsMapTest.php

3 lines

9 lines

19 lines

Exceptions/

HandlerTest.php

7 lines

Facades/

DockerHubTest.php

4 lines

MailgunTest.php

4 lines

4 lines

Http/

Controllers/

GitHubGateControllerTest.php

2 lines

PayloadFullTest.php

2 lines

PlaceholderTest.php

7 lines

7 lines

Jobs/

NotifyNewCommitsToDiffusionTest.php

6 lines

Notifications/

DockerHubNotificationTest.php

1 line

Phabricator/

PhabricatorAPIExceptionTest.php

2 lines

PhabricatorAPIFactoryTest.php

2 lines

PhabricatorAPITest.php

8 lines

ProjectsMapFactoryTest.php

2 lines

ProjectsMapTest.php

17 lines

Providers/

6 lines

DockerHubServiceProviderTest.php

2 lines

EventServiceProviderTest.php

24 lines

ServicesServiceProviderTest.php

2 lines

9 lines

Diff 6801

.arclint

app/Analyzers/BasePayloadAnalyzer.php

app/Analyzers/DockerHub/BuildFailureEvent.php

app/Analyzers/GitHub/Events/Event.php

Loading...

app/Analyzers/GitHub/Events/IssueCommentEvent.php

Loading...

app/Analyzers/GitHub/Events/PullRequestEvent.php

Loading...

app/Analyzers/GitHub/Events/PushEvent.php

app/Analyzers/GitHub/Events/WithRef.php

app/Analyzers/GitHub/Events/WithRepoAndBranch.php

app/Analyzers/GitHub/GitHubPayloadAnalyzer.php

app/Analyzers/ItemGroupMapping.php

app/Analyzers/Jenkins/JenkinsPayloadAnalyzer.php

app/Config/Features.php

app/Config/Reporting/BaseReportEntry.php

Loading...

app/Config/Reporting/ConfigReport.php

app/Config/Reporting/ServiceReportEntry.php

app/Config/Services/Services.php

app/Console/Commands/ConfigValidate.php

app/Console/Commands/PhabricatorProjectsMap.php

app/Contracts/APIClient.php

app/Contracts/APIFactory.php

app/Events/DockerHubPayloadEvent.php

app/Events/GitHubPayloadEvent.php

app/Events/JenkinsPayloadEvent.php

app/Events/NotificationEvent.php

app/Events/PhabricatorPayloadEvent.php

app/Events/ReportEvent.php

app/Exceptions/Handler.php

app/Facades/Raven.php

app/Http/Controllers/Controller.php

app/Http/Controllers/Gate/DockerHubGateController.php

app/Http/Controllers/Gate/GateController.php

app/Http/Controllers/Gate/GitHubGateController.php

app/Http/Controllers/Gate/JenkinsGateController.php

app/Http/Controllers/Gate/NotificationGateController.php

app/Http/Controllers/Gate/PhabricatorGateController.php

app/Http/routes.php

app/Jobs/FireDockerHubNotification.php

app/Jobs/FireGitHubNotification.php

app/Jobs/FireJenkinsNotification.php

app/Jobs/FirePhabricatorNotification.php

app/Jobs/NotifyNewCommitsToDiffusion.php

app/Jobs/SendMessageToBroker.php

app/Jobs/TriggerDockerHubBuild.php

app/Listeners/AMQPEventListener.php

app/Listeners/DockerHubListener.php

app/Listeners/LastPayloadSaver.php

app/Listeners/NotificationListener.php

app/Listeners/PhabricatorListener.php

app/Phabricator/PhabricatorAPI.php

app/Phabricator/PhabricatorStory.php

app/Phabricator/ProjectsMap.php

app/Phabricator/ProjectsMapFactory.php

app/Providers/DockerHubServiceProvider.php

app/Providers/EventServiceProvider.php

app/Providers/MailgunServiceProvider.php

app/Providers/PhabricatorAPIServiceProvider.php

app/Providers/PhabricatorProjectsMapServiceProvider.php

app/Providers/ReportServiceProvider.php

app/Providers/RouteServiceProvider.php

Loading...

composer.json

Loading...

config/app.php

phpcs.xml

tests/Actions/AMQPActionTest.php

tests/Actions/ActionErrorTest.php

tests/Actions/ActionsReportTest.php

tests/Actions/NotifyNewCommitsActionTest.php

tests/Actions/TriggerDockerHubBuildActionTest.php

tests/Analyzers/GitHub/Events/CreateEventTest.php

tests/Analyzers/GitHub/Events/DeleteEventTest.php

tests/Analyzers/GitHub/Events/EventTest.php

tests/Analyzers/GitHub/Events/IssueCommentEventTest.php

tests/Analyzers/GitHub/Events/PullRequestEventTest.php

tests/Analyzers/GitHub/Events/PushEventTest.php

tests/Analyzers/GitHub/Events/RepositoryEventTest.php

tests/Analyzers/GitHub/Events/StatusEventTest.php

tests/Analyzers/GitHub/Events/UnknownEventTest.php

tests/Analyzers/GitHub/GitHubPayloadAnalyzerTest.php

Loading...

tests/Analyzers/Jenkins/JenkinsPayloadAnalyzerConfigurationTest.php

tests/Analyzers/Jenkins/JenkinsPayloadAnalyzerTest.php

tests/Analyzers/PayloadAnalyzerConfigurationTest.php

tests/Analyzers/Phabricator/PhabricatorGroupMappingTest.php

tests/Analyzers/Phabricator/PhabricatorPayloadAnalyzerTest.php

Loading...

tests/Analyzers/Phabricator/WithConfiguration.php

tests/Config/FeaturesTest.php

Loading...

tests/Config/Reporting/FeatureReportEntryTest.php

tests/Config/Reporting/IntegrationTest.php

tests/Config/Reporting/ServiceReportEntryTest.php

tests/Config/Services/ServiceTest.php

tests/Config/Services/ServicesTest.php

tests/Console/Commands/ConfigShowTest.php

tests/Console/Commands/ConfigValidateTest.php

tests/Console/Commands/NotificationsPayloadTest.php

Loading...

tests/Console/Commands/PhabricatorProjectsMapTest.php

tests/Console/Commands/TestCase.php

tests/Console/KernelTest.php

tests/Exceptions/HandlerTest.php

tests/Facades/DockerHubTest.php

tests/Facades/MailgunTest.php

tests/Facades/RavenTest.php

tests/Http/Controllers/GitHubGateControllerTest.php

tests/Http/PayloadFullTest.php

tests/Http/PlaceholderTest.php

tests/Http/StatusTest.php

tests/Jobs/NotifyNewCommitsToDiffusionTest.php

tests/Notifications/DockerHubNotificationTest.php

Loading...

tests/Phabricator/PhabricatorAPIExceptionTest.php

tests/Phabricator/PhabricatorAPIFactoryTest.php

tests/Phabricator/PhabricatorAPITest.php

tests/Phabricator/ProjectsMapFactoryTest.php

tests/Phabricator/ProjectsMapTest.php

tests/Providers/ConfigTest.php

tests/Providers/DockerHubServiceProviderTest.php

tests/Providers/EventServiceProviderTest.php

tests/Providers/ServicesServiceProviderTest.php

tests/TestCase.php

Loading...