Bug 411920

Summary:	SQL connections does not save nor retrieve
Product:	[Applications] kate	Reporter:	tepostlakatl <horus.sandoval>
Component:	sessions	Assignee:	Artyom Kirnev <kirnevartem30>
Status:	RESOLVED FIXED
Severity:	normal	CC:	christoph, justin.zobel, kirnevartem30, mircea.dragota, ms27, salvi.loris
Priority:	NOR
Version First Reported In:	18.08.0
Target Milestone:	---
Platform:	Debian stable
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:	22.04.0
Sentry Crash Report:
Attachments:	attachment-22501-0.html

Description tepostlakatl 2019-09-14 19:12:48 UTC

SUMMARY

Using the SQL plugin, when you create a new connection, and activate the "save and restore connections in the kate session", connections does not retrieve back

STEPS TO REPRODUCE
1. Enable SQL plugin.
2. Create a new session.
3. Create a new connection.
4. Save session.
5. Close kate.
6. Open again kate and restore the saved session.

OBSERVED RESULT
Connections aren't restored.

EXPECTED RESULT
Be able to use the connections I'd created before.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Debian 10, Debian 4.19.67-2 (2019-08-28) x86_64 GNU/Linux
KDE Plasma Version: 5.14.5
KDE Frameworks Version: 5.54.0
Qt Version: 5.11.3

ADDITIONAL INFORMATION

Comment 1 Loris 2019-11-11 11:55:38 UTC

I confirm, same problem.

Debian GNU/Linux 10 4.19.0-6-amd64
KDE Plasma: 5.14.5
Qt: 5.11.3
KDE Frameworks: 5.54.0

Comment 2 Mircea D 2020-01-24 16:08:00 UTC

Same problem here.

OpenSuse Leap 15.1
KDE Plasma Version : 5.12.8
Qt Version : 5.9.7
KDE Frameworks Version : 5.55.0

Comment 3 Justin Zobel 2020-10-30 04:27:50 UTC

Setting to confirmed as reported by 3 users.

Comment 4 tepostlakatl 2020-10-31 20:01:38 UTC

I was checking the source code and noticed than the class KateSQLPlugin doesn't implement the KTextEditor::SessionConfigInterface interface and also, it is not implementen neither method to save the configuration. I'd tryed to implement the class but until now, I havent succeded on it.

Comment 5 Mircea D 2020-10-31 20:04:07 UTC

Created attachment 132924 [details]
attachment-22501-0.html

Good to know somebody s taking care of it after such a long time Sent from my Samsung Galaxy smartphone.
-------- Original message --------From: tepostlakatl <bugzilla_noreply@kde.org> Date: 10/31/20  22:07  (GMT+02:00) To: mircea.dragota@ccms.ro Subject: [kate] [Bug 411920] SQL connections does not save nor retrieve https://bugs.kde.org/show_bug.cgi?id=411920--- Comment #4 from tepostlakatl <horus.sandoval@carpathialab.com> ---I was checking the source code and noticed than the class KateSQLPlugin doesn'timplement the KTextEditor::SessionConfigInterface interface and also, it is notimplementen neither method to save the configuration. I'd tryed to implementthe class but until now, I havent succeded on it.-- You are receiving this mail because:You are on the CC list for the bug.

Comment 6 Christoph Cullmann 2021-03-16 20:57:16 UTC

(In reply to tepostlakatl from comment #4)
> I was checking the source code and noticed than the class KateSQLPlugin
> doesn't implement the KTextEditor::SessionConfigInterface interface and
> also, it is not implementen neither method to save the configuration. I'd
> tryed to implement the class but until now, I havent succeded on it.

If you need some help, just ask what is the issue, would be cool to have this fixed!

Comment 7 Christoph Cullmann 2021-03-16 20:57:38 UTC

Updated build docs are here:

https://kate-editor.org/build-it/

Comment 8 Christoph Cullmann 2021-12-03 20:37:45 UTC

Implement SessionConfigInterface in sql plugin (and expanded it's functionality to store SQLite connections)

https://invent.kde.org/utilities/kate/-/merge_requests/522