Skip to content

Conversation

@Mopolo
Copy link
Contributor

@Mopolo Mopolo commented Jan 12, 2026

No description provided.

@Mopolo Mopolo self-assigned this Jan 12, 2026
@Mopolo Mopolo force-pushed the refactor-doctrine-compta-config-category branch from 1642223 to 3fb24eb Compare January 12, 2026 12:53
@Mopolo Mopolo force-pushed the refactor-doctrine-compta-config-category branch from 3fb24eb to 2d65975 Compare January 12, 2026 13:22
@Mopolo Mopolo marked this pull request as ready for review January 12, 2026 15:29
@Mopolo Mopolo requested review from agallou, stakovicz and vgreb January 12, 2026 15:29
@vgreb
Copy link
Collaborator

vgreb commented Jan 13, 2026

Je profilte de la MR pour lancer le débat. Avec la migration Doctrine, on migre sur les public properties pour se soustraire des getter/setter ?

@Mopolo
Copy link
Contributor Author

Mopolo commented Jan 13, 2026

Je profilte de la MR pour lancer le débat. Avec la migration Doctrine, on migre sur les public properties pour se soustraire des getter/setter ?

Pour moi oui, avec les typed properties de toute façon ça revient au même qu'avec les getters/setters, juste plus simple à écrire.

Bien sûr je pense qu'on peut combiner ça avec des méthodes pour certains besoins, par exemple:

class Person
{
	public string $firstName;
	public string $lastName;

	public function fullname(): string { ... }

Copy link
Collaborator

@vgreb vgreb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On déclare les relations entre les entité Doctrine ? Je sais qu'on en avait parlé mais je ne me souviens plus si on l'avait acté.

@Mopolo
Copy link
Contributor Author

Mopolo commented Jan 15, 2026

On déclare les relations entre les entité Doctrine ? Je sais qu'on en avait parlé mais je ne me souviens plus si on l'avait acté.

Hm il me semble qu'on s'était dit qu'on verrait ce qu'on en pense en faisant une PR avec pour voir.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants