Sync with original repository

[ctariel]

Summary by CodeRabbit

Note di rilascio

• Nuove funzionalità

  • Aggiunto supporto per la localizzazione in francese.
  • Introdotto il campo "Version" per i server Z39.50/SRU nell'interfaccia amministrativa.
  • Esteso il salvataggio dati negli importi ISBN per includere numero di serie e dimensioni.
  • Aggiunto supporto per il parser MARCXchange nei server SRU.

• Miglioramenti

  • Aggiornate le opzioni di sintassi disponibili per i server Z39.50/SRU.

[coderabbitai]

Warning

Rate limit exceeded

@fabiodalez-dev has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 13 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: efb51fe5-03d1-49ce-b7e8-5fc702169ec1

📥 Commits

Reviewing files that changed from the base of the PR and between d630ced and 82ae589.

📒 Files selected for processing (5)

• app/Views/admin/plugins.php
• app/Views/libri/partials/book_form.php
• installer/database/data_de_DE.sql
• locale/fr_FR.json
• storage/plugins/z39-server/classes/SruClient.php

📝 Walkthrough

Walkthrough

Aggiunge supporto per il campo Version nei server Z39.50/SRU con estensione parser MARCXchange, espande form importazione ISBN con campi numero_serie e dimensions, e introduce la lingua francese (fr_FR) nei dati di inizializzazione database.

Changes

Cohort / File(s)	Summary
Z39.50/SRU Configuration app/Views/admin/plugins.php, storage/plugins/z39-server/classes/SruClient.php	Aggiunge campo Version in UI amministrazione, preset BnF con versione 1.2 e sintassi unimarcxchange. In SruClient: quote ISBN, namespace adjustment (srw/mxc/ixm), nuovo metodo parseMarcxchangeXml per parsing MARCXchange con estrazione campi bibliografici.
ISBN Import Form app/Views/libri/partials/book_form.php	Aggiunge input nascosti per numero_serie e dimensions, popola campi visibili e hidden dal payload scraper, estende tracking fieldsPopulated con nuovo campo classificazione_dewey.
Database Seeding Languages installer/database/data_de_DE.sql, installer/database/data_en_US.sql, installer/database/data_it_IT.sql	Inserisce riga fr_FR nella tabella languages con path locale/fr_FR.json e metriche completamento traduzioni.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minuti

✨ 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

_{Review rate limit: 0/1 reviews remaining, refill in 8 minutes and 13 seconds.}

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

[fabiodalez-dev]

Grazie per il contributo @ctariel — le feature sono utili e il lavoro è sostanzioso. Alcune note prima del merge:

✅ Approvato

• Locale FR (locale/fr_FR.json): traduzione completa (4082 chiavi). Ottimo lavoro.
• Campo Version per SRU (plugins.php): utile per BnF e altri server SRU 1.2. Il preset BnF con unimarcxchange è corretto.
• Sintassi unimarcxchange nel select: necessaria per i server francesi (SPN, BnF).
• ISBN scraping — numero_serie e dimensions già gestiti nel form: l'aggiunta è allineata con il pattern esistente.

❌ Bug da correggere

installer/database/data_de_DE.sql — manca la virgola prima della riga FR:

-- ATTUALE (SQL invalido):
('de_DE', 'German', 'Deutsch', '🇩🇪', 1, 1, 'locale/de_DE.json', 4009, 4009, 100.00)
('fr_FR', 'French', 'Français', '🇫🇷', 0, 1, 'locale/fr_FR.json', 4081, 4081, 100);

-- CORRETTO:
('de_DE', 'German', 'Deutsch', '🇩🇪', 1, 1, 'locale/de_DE.json', 4009, 4009, 100.00),
('fr_FR', 'French', 'Français', '🇫🇷', 0, 1, 'locale/fr_FR.json', 4081, 4081, 100);

Nota: la stessa riga FR va aggiunta anche in installer/database/data_it_IT.sql e data_en_US.sql per coerenza tra le localizzazioni.

⚠️ Da verificare

• Il campo version del server SRU viene salvato nel JSON di configurazione ma il SruClient.php lo usa effettivamente nella query? Se sì, ottimo — altrimenti il campo sarebbe decorativo.
• Nella PR non vedo modifiche a data_it_IT.sql / data_en_US.sql: il linguaggio FR verrebbe registrato solo installando in DE?