Skip to content

refactor: add Laravel wrapper for legacy application - #811

Draft
anonymoususer72041 wants to merge 15 commits into
opencats:masterfrom
anonymoususer72041:refactor/laravel-wrapper
Draft

refactor: add Laravel wrapper for legacy application#811
anonymoususer72041 wants to merge 15 commits into
opencats:masterfrom
anonymoususer72041:refactor/laravel-wrapper

Migrate CI test infrastructure to Laravel Sail

61d1f7e
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Jul 7, 2026 in 0s

20 new issues (0 max.) of at least severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 20
           

Complexity increasing per file
==============================
- app/Http/Controllers/LegacyController.php  47
- app/Models/User.php  1
- bootstrap/app.php  2
- database/migrations/0001_01_01_000002_create_jobs_table.php  2
- legacy/lib/Companies.php  39
- legacy/lib/JobOrders.php  48
- legacy/lib/Mailer.php  31
- legacy/lib/TemplateUtility.php  165
         

See the complete overview on Codacy

Annotations

Check warning on line 33 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L33

The method handle() has a Cyclomatic Complexity of 15. The configured cyclomatic complexity threshold is 10.

Check warning on line 33 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L33

The method handle() has an NPath complexity of 1408. The configured NPath complexity threshold is 200.

Check notice on line 45 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L45

Avoid excessively long variable names like $headersBeforeExecution. Keep variable name length under 20.

Check notice on line 66 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L66

Avoid excessively long variable names like $headersAfterExecution. Keep variable name length under 20.

Check warning on line 131 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L131

prepareLegacyServerState accesses the super-global variable $_SERVER.

Check warning on line 161 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L161

mergeServerBagValues accesses the super-global variable $_SERVER.

Check warning on line 178 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L178

restoreLegacyServerState accesses the super-global variable $_SERVER.

Check warning on line 209 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L209

Avoid using static access to class '\Symfony\Component\HttpFoundation\Cookie' in method 'applyNativeHeadersToResponse'.

Check notice on line 221 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L221

Avoid excessively long variable names like $headersBeforeExecution. Keep variable name length under 20.

Check notice on line 230 in app/Http/Controllers/LegacyController.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

app/Http/Controllers/LegacyController.php#L230

Avoid excessively long variable names like $headersBeforeExecution. Keep variable name length under 20.

Check warning on line 31 in database/factories/UserFactory.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

database/factories/UserFactory.php#L31

Avoid using static access to class '\Illuminate\Support\Facades\Hash' in method 'definition'.

Check warning on line 32 in database/factories/UserFactory.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

database/factories/UserFactory.php#L32

Avoid using static access to class '\Illuminate\Support\Str' in method 'definition'.

Check warning on line 41 in database/factories/UserFactory.php

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

database/factories/UserFactory.php#L41

Avoid unused parameters such as '$attributes'.

Check notice on line 1 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L1

Strings must use doublequote.

Check notice on line 2 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L2

Strings must use doublequote.

Check notice on line 3 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L3

Strings must use doublequote.

Check notice on line 4 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L4

Strings must use doublequote.

Check notice on line 9 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L9

Strings must use doublequote.

Check notice on line 12 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L12

Strings must use doublequote.

Check notice on line 21 in vite.config.js

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

vite.config.js#L21

Strings must use doublequote.