Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V13: Add rcl files to static files tree #17671

Open
wants to merge 2 commits into
base: v13/dev
Choose a base branch
from

Obsolete ctor

93ad7bd
Select commit
Loading
Failed to load commit list.
Open

V13: Add rcl files to static files tree #17671

Obsolete ctor
93ad7bd
Select commit
Loading
Failed to load commit list.
CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (v13/dev) failed Nov 28, 2024 in 56s

CodeScene PR Check

Code Health Quality Gates: FAILED

Change in average Code Health of affected files: -0.29 (9.24 -> 8.96)

  • Declining Code Health: 2 findings(s) 🚩
  • Improving Code Health: 2 findings(s) ✅

View detailed results in CodeScene

Details

🚩 Declining Code Health (highest to lowest):

  • Code Duplication StaticFilesTreeController.cs
  • Constructor Over-Injection StaticFilesTreeController.cs: StaticFilesTreeController

✅ Improving Code Health:

  • Complex Method StaticFilesTreeController.cs: GetTreeNodes
  • Complex Conditional StaticFilesTreeController.cs: GetTreeNodes

Annotations

Check warning on line 38 in src/Umbraco.Web.BackOffice/Trees/StaticFilesTreeController.cs

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (v13/dev)

❌ New issue: Code Duplication

The module contains 2 functions with similar structure: StaticFilesTreeController,StaticFilesTreeController. Avoid duplicated, aka copy-pasted, code inside the module. More duplication lowers the code health.

Check notice on line 136 in src/Umbraco.Web.BackOffice/Trees/StaticFilesTreeController.cs

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (v13/dev)

✅ No longer an issue: Complex Method

GetTreeNodes is no longer above the threshold for cyclomatic complexity

Check notice on line 67 in src/Umbraco.Web.BackOffice/Trees/StaticFilesTreeController.cs

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (v13/dev)

✅ No longer an issue: Complex Conditional

GetTreeNodes no longer has a complex conditional

Check warning on line 38 in src/Umbraco.Web.BackOffice/Trees/StaticFilesTreeController.cs

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (v13/dev)

❌ New issue: Constructor Over-Injection

StaticFilesTreeController has 6 arguments, threshold = 5. This constructor has too many arguments, indicating an object with low cohesion or missing function argument abstraction. Avoid adding more arguments.