SonarQube MAJOR CODE_SMELL: Class "User" has 21 methods, which is greater than 20 authorized. Split it into #956
Labels
No labels
automated
code-quality
component: admin dashboard
component: backend
component: billing
component: content management
component: faqs
component: game genres
component: game lists
component: game reviews
component: game search
component: games
component: internal notifications
component: platforms
component: security
component: tests
component: user api
component: user badges
component: user blogs
component: user consoles
component: user favorites
component: user friends
component: user notifications
component: user profiles
component: user site notifications
component: user wishlists
component: web design
dependencies
javascript
php
priority
high
priority
low
priority
medium
security-hotspot
source: codex
source: sonarqube
status
awaiting feedback
status
backlog
status
done
status
in progress
status
in queue
status
in review
status
needs codex review
status
needs investigation
status
wontfix
type: bug
type: documentation
type: feature
type: improvement
type: regression
type: task
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
MyVideoGameList/myvideogamelist.com#956
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
SonarQube issue detected.
SonarQube key:
eae83e1b-a69d-49a7-8b43-a898426e0fa1Severity:
MAJORType:
CODE_SMELLLocation:
myvideogamelist:app/Models/User.php:19Message: Class "User" has 21 methods, which is greater than 20 authorized. Split it into smaller classes.
https://sonarqube.linuxbox.ninja/project/issues?id=myvideogamelist&issues=eae83e1b-a69d-49a7-8b43-a898426e0fa1&open=eae83e1b-a69d-49a7-8b43-a898426e0fa1
Extracted friendship relationships into the HasFriends model concern, reducing the methods declared in User. Friendship and profile tests pass. Implemented in PR #957. Pint and PHPStan pass; Forgejo CI and the post-merge SonarQube scan are pending. Leaving this issue open until merge.