MokoConsulting/MokoSuiteBackup

fix(restore): surface DatabaseImporter errors + quote-aware split #294

Merged

jmiller merged 1 commits from fix/databaseimporter-surface-errors into dev

2026-07-16 18:06:57 +00:00

Author	SHA1	Message	Date
jmiller	028e1c7e80	fix(restore): surface DatabaseImporter errors + quote-aware split (#188 ) Universal: Pre-Release / Build Pre-Release (${{ inputs.stability \|\| github.ref_name }}) (push) Failing after 7s Details Universal: PR Check / Branch Policy (pull_request) Successful in 2s Details Universal: PR Check / Secret Scan (pull_request) Successful in 8s Details Universal: PR Check / Validate PR (pull_request) Failing after 12s Details Generic: Project CI / Lint & Validate (pull_request) Successful in 15s Details Branch Cleanup / Delete merged branch (pull_request) Successful in 1s Details RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped Details Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 12s Details Generic: Project CI / Tests (pull_request) Has been cancelled Details Universal: PR Check / Build RC Package (pull_request) Has been cancelled Details Universal: PR Check / Report Issues (pull_request) Has been cancelled Details RestoreEngine ignored DatabaseImporter::hasErrors(), so the non-stepped restore always reported success even when SQL statements failed. It now logs the failures and returns a warning ("database may be incomplete"). DatabaseImporter's splitter is also quote/backtick/escape-aware so a ; or newline inside a value no longer splits a statement, and comments are only skipped between statements — matching the stepped-engine fix (#291). Closes #188. Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i	2026-07-16 13:06:44 -05:00