Skip to content

feat: upgrade project to Symfony 5.4 and PHP 8.1 with complete test coverage - #59

Merged
Mike031289 merged 10 commits into
mainfrom
feature/upgrade-symfony-5.4
Jul 26, 2026
Merged

feat: upgrade project to Symfony 5.4 and PHP 8.1 with complete test coverage#59
Mike031289 merged 10 commits into
mainfrom
feature/upgrade-symfony-5.4

Conversation

@Mike031289

Copy link
Copy Markdown
Owner

Description

This Pull Request marks the complete modernization of the ToDo & Co application, migrating from the legacy stack (Symfony 3 / PHP 5.5) to a modern, maintainable, and secure architecture under Symfony 5.4 and PHP 8.1.


Key Changes Implemented

1. Technical Migration & Upgrade

  • Framework & PHP: Upgraded to Symfony 5.4 (LTS) with PHP 8.1 compatibility.
  • Project Restructuring: Adopted modern Symfony directory layout (removed legacy app/ and web/ directories in favor of config/, public/, and templates/).
  • Quality & Analysis: Cleaned up code using Rector and ensured compliance with coding standards (PSR-12).

2. Security & Access Control

  • Voters: Implemented a custom TaskVoter to secure resource access and prevent unauthorized modifications (strict deletion rules for standard users and administrators regarding anonymous tasks).
  • CLI Commands: Added a maintenance command to safely migrate orphaned tasks to the system anonyme user account.

3. Testing & Quality Assurance (QA)

  • PHPUnit 9.6 Suite: The entire suite of functional and unit tests has been updated and validated (35 tests / 94 assertions passing successfully).
  • Documentation: Fully updated the README.md, technical security documentation (doc/TECHNICAL_SECURITY.md), and integrated the final quality audit report (v1.0.0).

✅ Checklist

  • All unit and functional tests pass successfully (vendor/bin/phpunit).
  • Code style complies with PSR-12 standards.
  • Documentation and audit reports are up to date.

@Mike031289 Mike031289 self-assigned this Jul 26, 2026
@Mike031289 Mike031289 added bug Something isn't working documentation Improvements or additions to documentation feature Extra attention is needed security labels Jul 26, 2026
@codacy-production

codacy-production Bot commented Jul 26, 2026

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 critical · 17 medium

Alerts:
⚠ 18 issues (≤ 0 issues of at least minor severity)

Results:
18 new issues

Category Results
UnusedCode 2 medium
BestPractice 4 medium
ErrorProne 10 medium
Security 1 critical
Complexity 1 medium

View in Codacy

🟢 Metrics 132 complexity · 4 duplication

Metric Results
Complexity 132
Duplication 4

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@Mike031289
Mike031289 force-pushed the feature/upgrade-symfony-5.4 branch from af80fb9 to 7cd2ce2 Compare July 26, 2026 07:44
@Mike031289
Mike031289 merged commit 34fa61b into main Jul 26, 2026
1 of 2 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in @Mike031289's ToDo-Co Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation feature Extra attention is needed security

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant