Deluan 1f3a7efa75 fix(backup): surface real SQLite error when backup step fails ...

The error-check ordering after backupOp.Step(-1) checked !done before
err, which masked the underlying SQLite error (e.g. SQLITE_BUSY, I/O
errors) with a generic "backup not done with step -1" message. On
failure, Step returns done=false together with a non-nil err, so the
!done branch short-circuited before the real error was ever reported.

Swap the checks so the SQLite error is returned first, making failing
backups actually diagnosable.

Refs https://github.com/navidrome/navidrome/issues/5305#issuecomment-4230470593

2026-04-11 21:14:52 -04:00

..

migrations

fix(subsonic): always emit required created field on AlbumID3 (#5340)

2026-04-10 19:29:20 -04:00

backup_test.go

feat(bfr): Big Refactor: new scanner, lots of new fields and tags, improvements and DB schema changes (#2709)

2025-02-19 20:35:17 -05:00

backup.go

fix(backup): surface real SQLite error when backup step fails

2026-04-11 21:14:52 -04:00

db_test.go

feat(bfr): Big Refactor: new scanner, lots of new fields and tags, improvements and DB schema changes (#2709)

2025-02-19 20:35:17 -05:00

db.go

refactor: run Go modernize (#5002)

2026-02-08 09:57:30 -05:00

export_test.go

feat(bfr): Big Refactor: new scanner, lots of new fields and tags, improvements and DB schema changes (#2709)

2025-02-19 20:35:17 -05:00