Skip to content

Fix tkindex als oo.sqlite3 afwezig is#121

Merged
berthubert merged 1 commit into
berthubert:mainfrom
vtraag:fix/oo-tkindex
Apr 24, 2026
Merged

Fix tkindex als oo.sqlite3 afwezig is#121
berthubert merged 1 commit into
berthubert:mainfrom
vtraag:fix/oo-tkindex

Conversation

@vtraag
Copy link
Copy Markdown
Contributor

@vtraag vtraag commented Apr 23, 2026

Volgens de README.md hoeft ooget en oopull momenteel niet gerund te worden. In dat geval bestaat oo.sqlite3 echter niet, en daardoor werkt tkindex momenteel niet correct. Helaas werkt ooget momenteel ook niet correct (zie hieronder). Dit is een simpele exception handling om ervoor te zorgen dat tkindex in ieder geval werkt als de oo.sqlite3 database mist.

Ik heb ook even gekeken waarom ooget momenteel niet lijkt te werken. In

tkconv/ooget.cc

Line 247 in 0712e1e

sqlw.queryT("CREATE table if not exists OOEntry (id TEXT)");

en

tkconv/ooget.cc

Line 251 in 0712e1e

sqlw.queryT("CREATE table if not exists OODocument (id TEXT NOT NULL REFERENCES OOEntry(id) ON DELETE CASCADE)");

worden momenteel nieuwe tabellen aangemaakt, maar die definities zijn niet correct. Latere regels (L341-349 en L464-480) doen vermoeden waat de correcte definities moeten zijn. Maar ik heb dit hier niet aangepast omdat ik dat niet zeker weet. Na de correcte definities werkt het in principe wel, maar begint het proces vanaf 1900, en ik ben daar nog niet verder ingedoken om het werkend te krijgen.

@berthubert
Copy link
Copy Markdown
Owner

Uitstekend, dank je! Heb ook gefixed dat dit minder een probleem zou moet zijn.

@berthubert berthubert merged commit a325333 into berthubert:main Apr 24, 2026
1 check passed
@vtraag
Copy link
Copy Markdown
Contributor Author

vtraag commented Apr 24, 2026

Super, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants