SonarQube CRITICAL CODE_SMELL: Define a constant instead of duplicating this literal "Playing a classic" 4 time #953
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#953
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:
560406ab-5b0e-4d64-a278-cd2a26904148Severity:
CRITICALType:
CODE_SMELLLocation:
myvideogamelist:tests/Feature/BadgeEvaluatorTest.php:83Message: Define a constant instead of duplicating this literal "Playing a classic" 4 times.
https://sonarqube.linuxbox.ninja/project/issues?id=myvideogamelist&issues=560406ab-5b0e-4d64-a278-cd2a26904148&open=560406ab-5b0e-4d64-a278-cd2a26904148
Centralized the repeated profile status fixture in BADGE_PROFILE_STATUS. Badge evaluator 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.