Skip to content

Chapter link#61

Closed
LucBlocker01 wants to merge 2 commits into
devfrom
chapter_link
Closed

Chapter link#61
LucBlocker01 wants to merge 2 commits into
devfrom
chapter_link

Conversation

@LucBlocker01

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/Entity/Chapter.php
Comment on lines +32 to +33
#[ORM\OneToMany(targetEntity: self::class, mappedBy: 'chapter')]
private Collection $link;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

link est un mot singulier dont la sémantique ne représente pas une collection : le nom de variable en convient pas

Comment on lines +22 to +25
// this up() migration is auto-generated, please modify it to your needs
$this->addSql('ALTER TABLE chapter ADD linked_by_id INT DEFAULT NULL');
$this->addSql('ALTER TABLE chapter ADD CONSTRAINT FK_F981B52E1AE3CFF3 FOREIGN KEY (linked_by_id) REFERENCES chapter (id)');
$this->addSql('CREATE INDEX IDX_F981B52E1AE3CFF3 ON chapter (linked_by_id)');

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tu dois faire valider tes noms de champs avant de faire une migration : là tu dois supprimer cette migration vu que le nom du champ ne convient pas

@LucBlocker01 LucBlocker01 deleted the chapter_link branch October 16, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants