[Koha-bugs] [Bug 25078] Update DB process - wrap each DBRev inside a transaction and better error handling

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25078

--- Comment #91 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
(In reply to Victor Grousset/tuxayo from comment #90)
> Testing note: always restart services after changing the version syspref.
> 
> ====
> 
> A bit confused about the test plan:
> 
> > - Apply "Bug 25078: [DO NOT PUSH] DB revs for testing" (restart_all)
> > - Read the different DBrevs created as examples
> > - Make sure the different use cases are covered
> > - execute the updatedatabase script (CLI)
> 
> Actually all patches should be applied from the start, right?

Yes, all the patches.

> The Chick-a-boom atomic update is executed after the failure of the previous
> update.
> Is that an issue?

No, after is correct.

> «Wide character in say» Shouldn't it be fixed by «Fix encoding issue with
> db_revs output»

Should, yes. Looks like there is still an issue. Patch coming.

> > - Update the DB from the UI
> ↓↓↓
> > User koha_kohadev doesn't have enough privilege on database koha_kohadev.
> 
> What?! This is obviously unrelated but worth mentioning in case others using
> koha-testing-docker or not also have that.

I have no idea what this is about.

> This time the Chick-a-boom update is first! Does it matter?

Not a big deal in the first step I'd say. Can be fix later if needed.
The atomic updates are executed by updatedatabase.pl and should be moved to
C4::Installer.

-- 
You are receiving this mail because:
You are watching all bug changes.