Page MenuHomeDevCentral

Update Laravel 8
ClosedPublic

Authored by DorianWinty on Apr 16 2022, 22:51.
Tags
  • Restricted Project
Referenced Files
F3762864: D2674.id6785.largetrue.diff
Thu, Nov 21, 16:47
F3761426: D2674.id6763.diff
Wed, Nov 20, 18:51
Unknown Object (File)
Wed, Nov 20, 05:37
Unknown Object (File)
Tue, Nov 19, 16:56
Unknown Object (File)
Tue, Nov 19, 13:04
Unknown Object (File)
Tue, Nov 19, 13:01
Unknown Object (File)
Tue, Nov 19, 08:56
Unknown Object (File)
Tue, Nov 19, 05:06
Subscribers
Tokens
"Yellow Medal" token, awarded by dereckson.

Details

Summary

Update to Laravel 8

Ref T1725

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Errors
SeverityLocationCodeMessage
Errortests/Actions/ActionsReportTest.php:59PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/EventTest.php:56PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/EventTest.php:70PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/IssueCommentEventTest.php:38PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/IssueCommentEventTest.php:39PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/PullRequestEventTest.php:38PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/PullRequestEventTest.php:39PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/PullRequestEventTest.php:41PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/PushEventTest.php:41PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/PushEventTest.php:56PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Analyzers/GitHub/Events/PushEventTest.php:57PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Providers/ConfigTest.php:38PHPCS.E.Generic.Strings.UnnecessaryStringConcat.FoundGeneric.Strings.UnnecessaryStringConcat.Found
Errortests/Providers/EventServiceProviderTest.php:33PHPCS.E.Generic.Strings.UnnecessaryStringConcat.FoundGeneric.Strings.UnnecessaryStringConcat.Found
Errortests/Providers/EventServiceProviderTest.php:36PHPCS.E.Generic.Files.LineLength.MaxExceededGeneric.Files.LineLength.MaxExceeded
Errortests/Providers/EventServiceProviderTest.php:36PHPCS.E.Generic.Strings.UnnecessaryStringConcat.FoundGeneric.Strings.UnnecessaryStringConcat.Found
Warningapp/Providers/RouteServiceProvider.php:37PHPCS.W.Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassGeneric.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClass
Warningtests/Http/PayloadFullTest.php:144PHPCS.W.Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassGeneric.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClass
Warningtests/Phabricator/ProjectsMapTest.php:178PHPCS.W.Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassAfterLastUsedGeneric.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassAfterLastUsed
Warningtests/Phabricator/ProjectsMapTest.php:178PHPCS.W.Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassAfterLastUsedGeneric.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassAfterLastUsed
Unit
Test Failures
Branch
arcpatch-D2674
Build Status
Buildable 4202
Build 4454: arc lint + arc unit

Unit TestsFailed

TimeTest
45 msNasqueron\Notifications\Tests\Actions\ActionsReportTest
Nasqueron\Notifications\Tests\Actions\ActionsReportTest::testReport data/report.json and rendered report differ too much. Try $this->assertEquals($expectedReport, $actualReport) to see a diff.
220 msNasqueron\Notifications\Tests\Analyzers\GitHubPayloadAnalyzerTest
Nasqueron\Notifications\Tests\Analyzers\GitHubPayloadAnalyzerTest::testGetGroupOnPushToMappedRepository Failed asserting that two strings are identical.
103 msNasqueron\Notifications\Tests\Analyzers\ItemGroupMappingTest
Nasqueron\Notifications\Tests\Analyzers\ItemGroupMappingTest::testDoesItemMatch Failed asserting that false is true.
197 msNasqueron\Notifications\Tests\Analyzers\Phabricator\PhabricatorGroupMappingTest
Nasqueron\Notifications\Tests\Analyzers\Phabricator\PhabricatorGroupMappingTest::testDoesProjectBelong Failed asserting that true is false.
103 msNasqueron\Notifications\Tests\Config\Reporting\IntegrationTest
Nasqueron\Notifications\Tests\Config\Reporting\IntegrationTest::testConfig ErrorException: Undefined property: Illuminate\Http\JsonResponse::$response
View Full Test Results (4 Failed · 6 Broken · 56 Passed)

Event Timeline

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

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

dereckson added inline comments.
app/Actions/AMQPAction.php
34

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

Probably created by a phpcbf with MediaWiki setting.

The previous code was correct.

33

Same issue, previous file was fully correct in style.

correct tests
update some packages

correct to make work the tests

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

modify test
remove unused use

adding back one removed file

dereckson requested changes to this revision.May 8 2022, 17:33
dereckson added inline comments.
app/Analyzers/GitHub/Events/Event.php
36

Extraneous added line

51

extraneous added line

app/Analyzers/GitHub/Events/IssueCommentEvent.php
18–20

MediaWiki style

app/Analyzers/GitHub/Events/PullRequestEvent.php
18

MediaWiki

tests/TestCase.php
5

Restore that division

  1. Our app
  2. External libs
  3. Global space
This revision now requires changes to proceed.May 8 2022, 17:33
DorianWinty retitled this revision from Update Laravel 8 to WIP Update Laravel 8.May 9 2022, 18:14
DorianWinty marked 5 inline comments as done.

modify folowing comments but need more to reading

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

correcting MW style to old style

dereckson requested changes to this revision.May 12 2022, 11:37
dereckson added inline comments.
app/Config/Reporting/BaseReportEntry.php
11–13

"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
35–36

MediaWiki style

composer.json
23–24

Two needs to be restored now we're up to date:

  • phploc
  • phpcpd
51

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
24–25

MW

tests/Console/Commands/NotificationsPayloadTest.php
49

MW (the two lines)

tests/Notifications/DockerHubNotificationTest.php
10

Extraneous line

This revision now requires changes to proceed.May 12 2022, 11:37
DorianWinty marked 9 inline comments as done.

correction with the comments

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
This revision was automatically updated to reflect the committed changes.