Differential D22133

Adapt DBInitializer to behavior change of QPSQL driver in Qt 5.13
ClosedPublic
Actions

Authored by dvratil on Jun 28 2019, 10:58 AM.

Details

Reviewers
vkrause
Commits
R165:74aed9542ba1: Adapt DBInitializer to behavior change of QPSQL driver in Qt 5.13
Summary

Since Qt 5.13 the QPSQL driver escapes table names, which makes them
case-sensitive, therefore QSqlDatabase::record() must be called with
a lower-case table name, otherwise it won't match.

BUG: 409234
FIXED-IN: 5.11.3

Test Plan

Akonadi doesn't attempt to re-initialize all tables on each start when
using Qt 5.13

Diff Detail

Repository
R165 Akonadi
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dvratil created this revision.Jun 28 2019, 10:58 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptJun 28 2019, 10:58 AM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
dvratil requested review of this revision.Jun 28 2019, 10:58 AM
vkrause accepted this revision.Jun 28 2019, 1:46 PM
This revision is now accepted and ready to land.Jun 28 2019, 1:46 PM
Closed by commit R165:74aed9542ba1: Adapt DBInitializer to behavior change of QPSQL driver in Qt 5.13 (authored by dvratil). · Explain WhyJun 28 2019, 2:05 PM
This revision was automatically updated to reflect the committed changes.
dfaure added a subscriber: dfaure.Jul 21 2019, 2:23 PM

This change has been reported to lead to this regression: https://bugs.kde.org/show_bug.cgi?id=409753

Revision Contents
Changeset List

Diff 60784

View Options

src/server/storage/dbintrospector.cpp

Loading...