Keeping Your Legacy Codebase Alive!

Having to manage “legacy” applications is an inevitable step in the life of developers, especially when they are critical to the business. Over the years and the many changes made to the code, technical debt accumulates to the point that it can even become a brake on the provision of value for the business. It can therefore be tempting for developers to completely rewrite an application with the latest trendy tools. Spoiler, this is rarely the most economically rational solution! Therefore, how can we reconcile the production of value for the business while maintaining technical debt at an acceptable level? The solution lies in an approach of progressive modernization of the code thanks to the many professional tools offered by the PHP community. This conference will give you an overview of best practices, techniques and tools to help you modernize your code base.

Transcript

1. Keeping your Legacy Codebase Alive! March, 21st 2025 - Dutch PHP Conference - Amsterdam, The Netherlands
2. @hhamon / hello@kodero.fr Hugo HAMON Consultant PHP / Symfony @ KODERO Symfony Certi fi ed Expert Developer
3. Why modernizing a “legacy” codebase?
4. What’s “legacy code” ? The one you haven’t written in first place 🫣 The one you’ve just written (and goes to production) 🥳 The one that is not tested (and goes to production) 😭 The one that is based on no longer supported dependencies 😞 The one that “works” on prod but no one wants to change 😰
5. Reasons for operating a change Validating a new added feature Fixing a persistent or business impacting bug Improving or simplifying the overall “shape” of the code Detecting and optimizing performance issues Maintaining your application for the years to come Replacing an old piece of software for a modern one Easing infrastructure & dependencies upgrades … https://unsplash.com/@clemono
6. Leaving the code in a better shape than it was before changing it What are the benefits for the tech team? Easing the upcoming evolutions and changes to the code base Lowering the risk of introducing new regressions and bugs when changing the code Keeping focusing on bringing value to the business and end users on the long run https://unsplash.com/@orrbarone 🏆 🥇 🥈 🥉
7. What kind of change to operate? Purely syntactic change (“code style”) Bug fix Duplicated code extraction No longer supported dependency replacement New feature integration Components abstraction / encapsulation Automated tests addition Infrastructure level change
8. Impactful changes! Impacts on third party services https://unsplash.com/@vinic_ Impacts on the data model Changes that introduce a breaking change for others Impacts on the business
9. Keeping delivery value for the business https://unsplash.com/@nathan_cima Define a roadmap with the stake holders Plan an action plan ahead Communicate with all involved parties Dedicate a testing / preprod environnement Use feature flags for beta testers Integrate tools to automate complex tasks
10. Sentry https://sentry.io
11. Automated Tests at the Heart of Software Development
12. “Code without tests is bad code. It doesn ’ t matter how well-written it is; it doesn ’ t matter how pretty or object-oriented or well-encapsulated it is.” “With tests, we can change the behavior of our code quickly and veri fi ably. Without them, we really don ’ t know if our code is getting better or worse.” — Michael C. Feathers https://www.pmi.org/learning/library/legacy-it-systems-upgrades-11443
13. Different types of tests https://unsplash.com/@imchenyf Unit Tests Integration Tests E2E Tests Others Application Tests
15. final class LoanTest extends TestCase { public function testGetLoanInstallmentValue(): void { $customer = new Customer(); $loan = new Loan(); $loan->setCustomer($customer); $loan->setBorrowedValue(10_000); $loan->setTotalInstallments(60); $loan->setMonthlyFee(3.25); self::assertSame($customer, $loan->getCustomer()); self::assertSame(10_000, $loan->getBorrowedValue()); self::assertSame(3.25, $loan->getMonthlyFee()); self::assertSame(60, $loan->getTotalInstallments()); // Expecting a decimal amount with cents self::assertSame(172.08, $loan->getInstallmentsValue()); } }
17. use Symfony\Component\Panther\Client; require __DIR__.'/vendor/autoload.php'; $client = Client::createChromeClient(); // Interact with the UI elements $client->request('GET', 'https://api-platform.com'); $client->clickLink('Getting started'); // Wait for an element to be present in the DOM (even if hidden) $crawler = $client->waitFor('#installing-the-framework'); // Alternatively, wait for an element to be visible $crawler = $client->waitForVisibility('#installing-the-framework'); echo $crawler->filter('#installing-the-framework')->text(); // Yeah, screenshot! $client->takeScreenshot('screen.png'); Panther Library
18. final class PostAppointmentsCancellationControllerTest extends WebTestCase { public function testCannotCancelPastAppointment(): void { // ... $this->browser() ->post( url: \sprintf('/api/appointments/%s/cancellation', $appointment->getId()), options: HttpOptions::json([ 'referenceNumber' => 'E4N6ST', 'lastName' => 'SMITH', 'reason' => 'I booked another one earlier.', ]), ) ->assertStatus(422) ->assertJson() ->assertJsonMatches('violations[0].title', 'Appointment is no longer cancellable.') ->use(function (MailerComponent $component): void { $component->assertNoEmailSent(); }) ; } } Symfony + Zenstruck Browser
19. Automate Legacy Code Modernization with PHP Tools
20. “ Dependencies management, what a hell!” “ How to get rid of all these include / require statements ?! ” https://unsplash.com/@nate_dumlao “ If I only could execute all these scripts within a one line command line… ”
21. Composer > Dependencies Management { "require": { "php": ">=7.2", "league/flysystem": "^2.3.2", "moneyphp/money": "^3", "pagerfanta/core": "*" } }
22. { "require": { "php": ">=7.2", "league/flysystem": "^2.5.0", "moneyphp/money": "^3.3.3", "pagerfanta/core": "^3.8.0" } } $ php composer bump
23. Dependabot https://github.com/dependabot Renovate.js https://github.com/renovatebot
24. { "require": { "php": ">=7.2", "ext-iconv": "*", "ext-imap": "*", "ext-mbstring": "*", "ext-pdo": "*", "ext-xsl": "*", ... } } $ php composer require ext-iconv ext-imap ext-mbstring …
25. { "scripts": { "app:install-dev": [ "@composer database:reset-dev", "@composer frontend:build-dev" ], "database:reset-dev": [ "@php bin/console d:d:d --if-exists --force -n -e dev", "@php bin/console d:d:c --if-not-exists -n -e dev", "@php bin/console d:m:m --all-or-nothing -n -e dev", "@php bin/console d:f:l --purge-with-truncate -n -e dev" ], "frontend:build-dev": [ "@php bin/console tailwind:build -e dev", "@php bin/console sass:build -e dev", "@php bin/console asset-map:compile -e dev" ] } } $ php composer app:install-dev
26. “ WOW ! I love these new functions (or classes) added to PHP 8.x. They would be very helpful for our code base! ” https://unsplash.com/@1nimidiffa_ “ Sadly, we’re still running production with PHP 7… ”
27. Polyfills symfony/polyfill-php54 symfony/polyfill-php55 symfony/polyfill-php56 symfony/polyfill-php70 symfony/polyfill-php71 symfony/polyfill-php72 symfony/polyfill-php73 symfony/polyfill-php74 symfony/polyfill-php80 symfony/polyfill-php81 symfony/polyfill-php82 symfony/polyfill-php83 symfony/polyfill-php84 symfony/polyfill-apcu symfony/polyfill-ctype symfony/polyfill-iconv symfony/polyfill-intl-grapheme symfony/polyfill-intl-idn symfony/polyfill-intl-icu symfony/polyfill-intl-messageformatter symfony/polyfill-mbstring symfony/polyfill-util symfony/polyfill-uuid
28. $text = <<<TEXT PHP is a popular general-purpose scripting language that is especially suited to web development. Fast, flexible and pragmatic, PHP powers everything from your blog 📰 to the most popular websites in the world. TEXT; if (strncmp($text, 'PHP is a popular', strlen('PHP is a popular')) === 0) { echo 'Text starts with "PHP is a popular scripting language" string.'; } if (strpos($text, 'from your blog 📰 to the most') !== false) { echo 'Text contains "from your blog 📰 to the most" string.'; } $needle = 'in the world.'; $needleLength = strlen($needle); if ($needleLength <= strlen($text) && 0 === substr_compare($text, $needle, -$needleLength)) { echo 'Text ends with "in the world." string.'; } Symfony Polyfills
29. { "require": { "php": ">=7.1", "symfony/polyfill-php72": "^1.31.0", "symfony/polyfill-php73": "^1.31.0", "symfony/polyfill-php74": "^1.31.0", "symfony/polyfill-php80": "^1.31.0", "symfony/polyfill-php81": "^1.31.0", "symfony/polyfill-php82": "^1.31.0", "symfony/polyfill-php83": "^1.31.0", "symfony/polyfill-php84": "^1.31.0" } } Symfony Polyfills
30. if (str_starts_with($text, 'PHP is a popular')) { echo 'Text starts with "PHP is a popular scripting language"…'; } if (str_contains($text, 'from your blog 📰 to the most')) { echo 'Text contains "from your blog 📰 to the most" string.'; } if (str_ends_with($text, 'in the world.')) { echo 'Text ends with "in the world." string.'; } $ php71 polyfill-php80.php Text starts with "PHP is a popular scripting language" string. Text contains "from your blog 📰 to the most" string. Text ends with "in the world." string.
31. “There is absolutely no consistency from one file to another! ” https://unsplash.com/@xavi_cabrera “ Let’s make things consistent once and for all for everyone ”
33. Configuration (1/3) return ECSConfig::configure() ->withPaths([ __DIR__ . '/config', __DIR__ . '/src', __DIR__ . '/tests', ]) ->withSkip([ __DIR__ . '/config/secrets/', ArrayOpenerAndCloserNewlineFixer::class, MethodChainingNewlineFixer::class, ]) ; ecs.php
34. return ECSConfig::configure() ->withPreparedSets( psr12: true, symplify: true, arrays: true, comments: true, docblocks: true, spaces: true, namespaces: true, controlStructures: true, phpunit: true, strict: true, cleanCode: true, ) ; ecs.php Configuration (2/3)
35. return ECSConfig::configure() ->withRules([ NoUnusedImportsFixer::class, ]) ->withSkip([ DeclareStrictTypesFixer::class, ]) ->withConfiguredRule(GlobalNamespaceImportFixer::class, [ 'import_classes' => true, ]) ->withConfiguredRule(MethodArgumentSpaceFixer::class,[ 'on_multiline' => 'ensure_fully_multiline', 'attribute_placement' => 'same_line', ]) ; ecs.php Configuration (3/3)
37. “ I’d love to modernize my codebase but how should I start? ” https://unsplash.com/@brookecagle “ If I only could be able to have a clear view on what is worth being fixed… ”
38. PHPStan https://phpstan.org/
39. { ... "require-dev": { ... "phpstan/phpstan": "^1.12.3", "phpstan/extension-installer": "^1.4.3", "phpstan/phpstan-deprecation-rules": "^1.2.1", "phpstan/phpstan-doctrine": "^1.5.3", "phpstan/phpstan-phpunit": "^1.4.0", "phpstan/phpstan-strict-rules": "^1.6.0", "phpstan/phpstan-symfony": "^1.4.9" }, "config": { "allow-plugins": { "phpstan/extension-installer": true } } } PHPStan & Plugins
40. includes: - phpstan-baseline.neon parameters: level: max paths: - bin/ - config/ - public/ - src/ excludePaths: - src/Migrations/ checkAlwaysTrueInstanceof: true checkAlwaysTrueStrictComparison: true checkExplicitMixedMissingReturn: true reportWrongPhpDocTypeInVarTag: true treatPhpDocTypesAsCertain: false PHPStan > Configuration > phpstan.neon file
41. $ php vendor/bin/phpstan --generate-baseline
42. parameters: ignoreErrors: - message: "#^Method App\\\\Controller\\\\Authenticator\\:\\:login\\(\\) has no return type specified\\.$#" count: 1 path: src/Controller/Authenticator.php - message: "#^Method App\\\\Controller\\\\Authenticator\\:\\:logout\\(\\) has no return type specified\\.$#" count: 1 path: src/Controller/Authenticator.php - message: "#^Method App\\\\Controller\\\\Customer\\:\\:edit\\(\\) has no return type specified\\.$#" count: 1 path: src/Controller/Customer.php - message: "#^Method App\\\\Controller\\\\Customer\\:\\:edit\\(\\) has parameter \\$id with no type specified\\.$#" count: 1 path: src/Controller/Customer.php - message: "#^Method App\\\\Controller\\\\Customer\\:\\:handleCreationFormSubmission\\(\\) has no return type specified\\.$#" count: 1 path: src/Controller/Customer.php - message: "#^Method App\\\\Controller\\\\Customer\\:\\:handleCreationFormSubmission\\(\\) has parameter \\$form with ...” count: 1 path: src/Controller/Customer.php - message: "#^Method App\\\\Controller\\\\Customer\\:\\:handleEditFormSubmission\\(\\) has no return type specified\\.$#" count: 1 path: src/Controller/Customer.php PHPStan > Generated Baseline File
43. “ It’s going to take so much time and effort to fix everything! ” https://unsplash.com/@odissei “ Could we have the opportunity to automate some of these changes? ”
44. PHPStorm
45. Rector https://getrector.com/ https://getrector.com
46. Rector ˒ Adding properties & methods type hints ˒ Auto reindentation ˒ Convert PHP annotations to PHP attributes ˒ Detect and remove dead code ˒ Leverage constructor property promotion ˒ Use modern syntax (array, closures, etc.) ˒ Leverage “ early returns ” ˒ Make properties & methods private ˒ Rename functions and methods ˒ Migrate framework based code (Symfony, Twig, PHPUnit, Laravel, etc.) ˒ Extensibility by creating (or importing third party) new rules ˒ …
47. { ... "require-dev": { ... "rector/rector": "^1.2.0" } } Rector > Installation
48. C declare(strict_types=1); use Rector\Config\RectorConfig; return RectorConfig::configure() ->withPaths([ __DIR__ . '/src', ]) ->withSkip([ __DIR__ . '/src/Migrations', ]) ->withAttributesSets(symfony: true, doctrine: true) ->withPhp74Sets() ->withPreparedSets(typeDeclarations: true); Rector > Configuration File rector.php
49. $ php vendor/bin/rector ——dry—run
50. Code Refactoring
51. Refactoring ˒ Organize code by business / domain / feature ˒ Reuse domain terminology in code ˒ Expose semantic class constructors ˒ Leverage use of native enumeration ˒ Enable dependency inversion principle ˒ Define short and focused interfaces ˒ Avoir anemic domain model for entities ˒ Assert values for arguments / parameters ˒ Reuse trusted third party PHP dependencies ˒ …
52. Organize code by domain https://unsplash.com/@odissei In order to better identify modules that coexist within the codebase.
53. src ├ ─ ─ Controller │ ├ ─ ─ Authenticator.php │ ├ ─ ─ ... │ └ ─ ─ User.php ├ ─ ─ Entity │ ├ ─ ─ Customer.php │ ├ ─ ─ Helper.php │ ├ ─ ─ Installment.php │ ├ ─ ─ InstallmentPeriod.php │ ├ ─ ─ InstallmentStatus.php │ ├ ─ ─ Loan.php │ ├ ─ ─ Role.php │ └ ─ ─ User.php ├ ─ ─ Repository │ ├ ─ ─ CustomersRepository.php │ ├ ─ ─ HelperRepository.php │ ├ ─ ─ InstallmentPeriodsRepository.php │ ├ ─ ─ InstallmentStatusRepository.php │ ├ ─ ─ InstallmentsRepository.php │ ├ ─ ─ LoansRepository.php │ ├ ─ ─ RolesRepository.php │ └ ─ ─ UserRepository.php └ ─ ─ Service ├ ─ ─ Calculator.php ├ ─ ─ Customer.php ├ ─ ─ Helper.php ├ ─ ─ Installment.php ├ ─ ─ Loan.php ├ ─ ─ Profit.php └ ─ ─ User.php 🤔
54. src ├ ─ ─ Application │ ├ ─ ─ Controller │ │ ├ ─ ─ Authenticator.php │ │ ├ ─ ─ Customer.php │ │ ├ ─ ─ CustomerHistoric.php │ │ ├ ─ ─ Installment.php │ │ ├ ─ ─ Loan.php │ │ ├ ─ ─ Profile.php │ │ ├ ─ ─ Profit.php │ │ └ ─ ─ User.php │ ├ ─ ─ DataFixtures │ │ └ ─ ─ ORM │ │ └ ─ ─ AppFixtures.php │ ├ ─ ─ Kernel.php │ └ ─ ─ Migrations │ ├ ─ ─ Version20190103002150.php │ ├ ─ ─ Version20190113221724.php │ ├ ─ ─ Version20190124120652.php │ ├ ─ ─ Version20190212012115.php │ └ ─ ─ Version20190212013033.php ├ ─ ─ BankingServices ├ ─ ─ FinancialReporting └ ─ ─ User
55. src ├ ─ ─ FinancialServices │ ├ ─ ─ Calculator.php │ ├ ─ ─ Customer.php │ ├ ─ ─ Entity │ │ ├ ─ ─ Customer.php │ │ ├ ─ ─ Helper.php │ │ ├ ─ ─ Installment.php │ │ ├ ─ ─ InstallmentPeriod.php │ │ ├ ─ ─ InstallmentStatus.php │ │ └ ─ ─ Loan.php │ ├ ─ ─ Helper.php │ ├ ─ ─ Installment.php │ ├ ─ ─ Loan.php │ └ ─ ─ Repository │ ├ ─ ─ CustomersRepository.php │ ├ ─ ─ HelperRepository.php │ ├ ─ ─ InstallmentPeriodsRepository.php │ ├ ─ ─ InstallmentStatusRepository.php │ ├ ─ ─ InstallmentsRepository.php │ └ ─ ─ LoansRepository.php ├ ─ ─ FinancialReporting │ └ ─ ─ Profit.php └ ─ ─ User ├ ─ ─ Entity │ ├ ─ ─ Role.php │ └ ─ ─ User.php ├ ─ ─ Repository │ ├ ─ ─ RolesRepository.php │ └ ─ ─ UserRepository.php └ ─ ─ User.php
56. Reuse domain terminology https://unsplash.com/@waldemarbrandt67w To name code data structures according to their real terminology within the business they belong to.
57. // Financial Reporting class Profit { // ... public function findAll(): array { $installmentsPerMonth = []; foreach ($this->installmentRepository->findAllPaid() as $installment) { $month = date('n', $installment->getDueDate()->getTimestamp()); if (! isset($installmentsPerMonth[$month])) { $installmentsPerMonth[$month] = []; } $installmentsPerMonth[$month][] = $installment; } $formattedData = []; foreach ($installmentsPerMonth as $month => $installments) { $formattedData[] = [ 'month' => $month, 'year' => date('Y', $installments[0]->getDueDate()->getTimestamp()), 'profit' => $this->sumInstallments($installments), ]; } return $formattedData; } }
58. class ComputeYearlyLoanInstallmentReporting { // ... /** * @return array<int, array{ * month: string, * year: string, * profit: float * }> */ public function __invoke(string $year): array { // ... } }
59. class ComputeYearlyLoanInstallmentReporting { // ... /** * @return array<int, array{month: string, year: string, profit: float}> */ public function __invoke(string $year): array { $reporting = new YearlyLoanInstallmentReporting($year); foreach ($this->installmentRepository->findYearlyPaid($year) as $installment) { $reporting->increaseMonthlyRevenue( $installment->getDueDate()->format('m'), $installment->getValue(), // to be renamed to getAmount? ); } return $reporting->toArray(); } }
60. Avoid anemic domain models https://unsplash.com/@pattib Expose explicit domain focused methods from the domain entities
61. // Customer has subscribed to a loan $loan = new Loan(...); $installmentAmount = $this->loanService->getInstallmentAmount($loan); // On loan settlement, all its installments are calculated $installment = (new Installment()) ->setLoan($loan) ->setDueDate(new DateTimeImmutable('2024-10-05')) ->setAmount($installmentAmount) ->setStatus(InstallmentStatus::DUE); // Each day, a cronjob runs to update the status of the due/overdue installments $receivedPaymentDate = ...; if ($receivedPaymentDate instanceof DateTimeImmutable) { $installment->setPaymentDate($receivedPaymentDate); $installment->setStatus(InstallmentStatus::PAID); } // If payment is still unpaid after due date, its marked overdue $today = new DateTimeImmutable('today'); if ($installment->getStatus() === InstallmentStatus::DUE && $today > $installment->getDueDate()) { $installment->setStatus(InstallmentStatus::OVERDUE); }
62. // Customer has subscribed to a loan $loan = new Loan(...); $installmentAmount = $this->loanService->getInstallmentAmount($loan); // On loan settlement, all its installments are calculated $installment = (new Installment()) ->setLoan($loan) ->setDueDate(new DateTimeImmutable('2024-10-05')) ->setAmount($installmentAmount) ->setStatus(InstallmentStatus::DUE);
63. class Installment { public function __construct( private readonly Loan $loan, private readonly InstallmentStatus $status, private readonly DateTimeImmutable $dueDate, private readonly float $amount, ) { } }
64. class Installment { public static function due( Loan $loan, DateTimeImmutable $dueDate, float $amount ): self { return new self( loan: $loan, status: InstallmentStatus::DUE, dueDate: $dueDate, Amount: $amount, ); } }
65. class Installment { private function __construct( private readonly Loan $loan, private readonly InstallmentStatus $status, private readonly DateTimeImmutable $dueDate, private readonly float $amount, ) { } }
66. // Customer has subscribed to a loan $loan = new Loan(...); $amount = $this->loanService->getInstallmentAmount($loan); // On loan settlement, all its installments are calculated $installment = Installment::due($loan, '2024-10-05', $amount);
67. class Installment { // ... public function isPaid(): bool { return $this->status === InstallmentStatus::PAID; } public function pay(DateTimeImmutable $paymentDate): void { if ($this->isPaid()) { throw new DomainException('Already paid!'); } $this->status = InstallmentStatus::PAID; $this->paymentDate = $paymentDate; } }
68. // Each day, a cronjob runs to update the status // of the due/overdue installments $receivedPaymentDate = new DateTimeImmutable('2024-10-03'); try { if ($receivedPaymentDate instanceof DateTimeImmutable) { $installment->pay($receivedPaymentDate); } } catch (DomainException $e) { $this->logger->warning($e->getMessage()); }
69. Leverage built-in enumerations https://unsplash.com/@glenncarstenspeters Encapsulate lists of predefined values within the same data structure.
70. enum InstallmentStatus: string { case DUE = 'due'; case OVERDUE = 'overdue'; case PAID = 'paid'; public function isPaid(): bool { return self::PAID->value === $this->value; } public function pay(): self { if ($this->isPaid()) { throw new LogicException('Installment already paid!'); } return self::PAID; } }
71. class Installment { // ... private InstallmentStatus $status = InstallmentStatus::DUE; public function isPaid(): bool { return $this->status->isPaid(); } public function pay(DateTimeImmutable $paymentDate): void { $this->status = $this->status->pay(); $this->paymentDate = $paymentDate; } }
72. Dependency inversion https://unsplash.com/@cmycatistufa200gsugden Reducing coupling between high-level modules and low-level modules.
73. ˒ High level modules should not depend on low level modules. ˒ Abstractions should not depend on implementation details ; implementations should depend on abstractions. Dependency Inversion
74. // ... use App\FinancialServices\Repository\DoctrineInstallmentRepository; use Doctrine\ORM\EntityManager; class ComputeYearlyLoanInstallmentReporting { private DoctrineInstallmentRepository $installmentRepository; public function __construct(EntityManager $entityManager) { /** @var DoctrineInstallmentRepository $installmentRepository */ $installmentRepository = $entityManager->getRepository(Installment::class); $this->installmentRepository = $installmentRepository; } // ... } detail detail detail
75. namespace App\FinancialServices\Repository; use App\FinancialServices\Entity\Installment; use App\FinancialServices\Entity\Loan; interface InstallmentRepository { public function byId(string $id): Installement; /** * @return Installment[] */ public function findByLoan(Loan $loan): array; /** * @return Installment[] */ public function findYearlyPaid(string $year): array; public function save(Installment $installment): void; }
76. // ... use App\FinancialServices\Repository\InstallmentRepository; class ComputeYearlyLoanInstallmentReporting { public function __construct( private readonly InstallmentRepository $installmentRepository, ) { } // ... } detail abstraction
77. final class InMemoryInstallmentRepository implements InstallmentRepository { /** @var array<string, Installment> */ private array $store = []; public function byId(string $id): Installement { $installment = $this->store[$id] ?? null; if (!$installment instanceof Installment) { throw new DomainException('Installment not found!'); } return $installment; } public function save(Installment $installment): void { $this->store[(string) $installment->getId()] = $installment; } } detail abstraction
78. final class DoctrineInstallmentRepository implements InstallmentRepository { public function __construct( private readonly EntityManagerInterface $entityManager, ) { } public function byId(string $id): Installement { $installment = $this->getRepository()->find($id); if (!$installment instanceof Installment) { throw new DomainException('Installment not found!'); } return $installment; } public function save(Installment $installment): void { $this->entityManager->persist($installment); $this->entityManager->flush(); } private function getRepository(): EntityRepository { return $this->entityManager->getRepository(Installment::class); } } detail abstraction abstraction
79. final class DoctrineInstallmentRepository implements InstallmentRepository { // ... public function findByLoan(Loan $loan): array { return $this->getRepository()->findBy(['loan' => $loan]); } public function findYearlyPaid(string $year): array { $qb = $this->getRepository()->createQueryBuilder('i'); $qb ->andWhere($qb->expr()->between('i.paymentDate', ':from', ':to')) ->andWhere($qb->expr()->eq('i.status', ':status')) ->setParameter('from', $year . '-01-01') ->setParameter('to', $year . '-12-31') ->setParameter('status', InstallmentStatus::PAID->value); /** @var Installment[] */ return $qb->getQuery()->getResult(); } }
80. // Unit testing case $computer = new ComputeYearlyLoanInstallmentReporting( new InMemoryInstallmentRepository(), ); // General use case $computer = new ComputeYearlyLoanInstallmentReporting( new DoctrineInstallmentRepository( new EntityManager(...) ), );
81. @hhamon / hello@kodero.fr Thank You!