Add cascade deletion for genre groups

[Himura2la]

Implement #269

Summary by CodeRabbit

• Nuove funzionalità

  • Aggiunta l’eliminazione a cascata di un genere e di tutti i relativi sottogeneri.
  • I libri e gli scaffali associati vengono scollegati prima dell’eliminazione.
  • Il pannello di conferma mostra avvisi e azioni specifiche per l’eliminazione a cascata.

• Correzioni

  • Distinti i messaggi di successo per l’eliminazione semplice e a cascata.
  • Aggiunte traduzioni aggiornate in italiano, inglese, tedesco e francese.

• Test

  • Aggiunti test per verificare l’eliminazione a cascata e le associazioni collegate.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0835d7f7-ffa7-4183-9217-e73f9ce9c5e0

📥 Commits

Reviewing files that changed from the base of the PR and between 221f018 and 534f14c.

📒 Files selected for processing (9)

• app/Controllers/GeneriController.php
• app/Models/GenereRepository.php
• app/Views/generi/dettaglio_genere.php
• locale/de_DE.json
• locale/en_US.json
• locale/fr_FR.json
• locale/it_IT.json
• tests/genre-bugs.spec.js
• tests/genre-cascade-delete.unit.php

---

📝 Walkthrough

Walkthrough

La modifica introduce l’eliminazione opzionale a cascata dei generi, con rimozione del sottoalbero, scollegamento di libri e mensole, aggiornamento dell’interfaccia, traduzioni e test di regressione.

Changes

Eliminazione a cascata dei generi

Layer / File(s)	Summary
Cancellazione transazionale del sottoalbero app/Models/GenereRepository.php	Aggiunge cascadeDelete, la raccolta iterativa degli id discendenti e il binding dinamico dei parametri interi; aggiorna libri e mensole, gestisce commit/rollback e invalida la cache.
Selezione e gestione della modalità di eliminazione app/Views/generi/dettaglio_genere.php, app/Controllers/GeneriController.php, locale/*_*.json	La vista abilita la modalità a cascata per i generi con figli; il controller seleziona l’operazione e aggiorna i messaggi localizzati.
Verifica del comportamento a cascata tests/genre-bugs.spec.js, tests/genre-cascade-delete.unit.php	I test verificano il form, il parametro cascade_delete, il rifiuto dell’eliminazione non a cascata e lo scollegamento dei riferimenti nel database.

Estimated code review effort: 4 (Complex) | ~45 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Admin
  participant GeneriController
  participant GenereRepository
  participant Database
  Admin->>GeneriController: invia cascade_delete
  GeneriController->>GenereRepository: cascadeDelete(id)
  GenereRepository->>Database: raccoglie il sottoalbero
  GenereRepository->>Database: scollega libri e mensole
  GenereRepository->>Database: elimina i generi e conferma la transazione
  GeneriController-->>Admin: redirect con messaggio di successo

Loading

Possibly related PRs

• fabiodalez-dev/Pinakes#161: aggiorna i redirect e gli URL nelle azioni e nell’interfaccia di gestione dei generi.

Suggested reviewers: fabiodalez-dev, fabiodalez

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Himura2la]

@fabiodalez-dev btw, I did not test it!!! I'm still working on deploying the app from my fork. It's totally a vibe-coded PR, and I expect you to make sure it's good. This PR did not even pass through @coderabbitai, I think it was too early to merge it...

[coderabbitai]

@Himura2la Ricevuto. Revisionerò la PR #270 concentrandomi su regressioni reali nella cancellazione a cascata, integrità dei dati e casi non gestiti prima del merge.

🐇🔍

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[Himura2la]

prima del merge

if only...