https://bugs.kde.org/show_bug.cgi?id=392112

--- Comment #6 from Jarosław Staniek <stan...@kde.org> ---
Git commit 9978d76a494ca4dc6760158ef742660e2b4989cb by Jaroslaw Staniek.
Committed on 06/01/2019 at 21:14.
Pushed by staniek into branch '3.2'.

Add KDbConnection::drv_getTableNames for low level list of table names, make
tableNames() skip names with non-existing physical tables

Summary:
- KDbTestUtils: add convenience APIs for connecting and using db, support
connection options, use it in parser test
- Add KDbConnection::drv_getTableNames for low level list of table names, make
tableNames() skip names with non-existing physical tables
- This change is backward compatible since metadata without physical table is
not usable anyway.
FIXED-IN:3.2.0

+ Add autotest for handling missing physical tables

Test Plan: Run ctest

Reviewers: piggz

Tags: #kdb

Differential Revision: https://phabricator.kde.org/D11547

M  +3    -0    autotests/CMakeLists.txt
A  +84   -0    autotests/MissingTableTest.cpp     [License: LGPL (v2+)]
A  +-    --    autotests/data/missingTableTest.kexi

https://commits.kde.org/kdb/9978d76a494ca4dc6760158ef742660e2b4989cb

--- Comment #7 from Jarosław Staniek <stan...@kde.org> ---
Git commit e5c3a99d939e9176da8a3d8809a6f48c8186bed8 by Jaroslaw Staniek.
Committed on 06/01/2019 at 21:14.
Pushed by staniek into branch '3.2'.

Add KDbConnection::drv_getTableNames for low level list of table names, make
tableNames() skip names with non-existing physical tables

This change is backward compatible since metadata without physical table is not
usable anyway.

FIXED-IN:3.2.0

M  +44   -2    src/KDbConnection.cpp
M  +20   -1    src/KDbConnection.h
M  +5    -0    src/KDbConnectionProxy.cpp
M  +5    -0    src/KDbConnectionProxy.h
M  +10   -1    src/KDbDriverBehavior.h
M  +4    -3    src/drivers/mysql/MysqlDriver.cpp
M  +3    -0    src/drivers/postgresql/PostgresqlDriver.cpp
M  +2    -0    src/drivers/sqlite/SqliteDriver.cpp

https://commits.kde.org/kdb/e5c3a99d939e9176da8a3d8809a6f48c8186bed8

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

Reply via email to