![]() |
#1 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
KoboTouch and new tolino devices
Since the new tolino devices are based on an adapted Kobo FW, it would be great to have them supported just as the Kobo devices. But since there are some differences (it's firmware version 5.x) it needs some work. What I noticed, for example, is that there seems to be no image tree for the covers anymore.
I did try to adapt the driver to the new devices, and doesn't look too bad. In my case, upoading books and covers seem to work very well. What doesn't work is putting books into collections. I don't know yet, why. Now I'm not sure where to go from here. Perhaps someone who is more experienced could have a look at my changes? At least it seems to not break anything, since my Clara 2E works with the changes as before. ![]() (But of course I don't know what happens when kobo devices go for fw 5.x …) Last edited by beedaddy; 07-19-2024 at 08:44 AM. |
![]() |
![]() |
![]() |
#2 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
Adding books to collections works now. I've no idea, why "false" in the SQL query doesn't work anymore. Using false (no quotation marks) works. But in both databases (Clara 2E and tolino shine 5), the field is BOOL.
![]() I'm also not sure whether more queries need to be adapted accordingly. But for me, it looks ok now. |
![]() |
![]() |
![]() |
#3 |
creator of calibre
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 44,250
Karma: 23661992
Join Date: Oct 2006
Location: Mumbai, India
Device: Various
|
false without quotation marks is correct in modern sqlite. Thoug it is better to use 0 and 1 for false and true as booleans are stored in sqlite databases as integers it doesnt have a separate boolean datatype. See
https://www.sqlite.org/datatype3.html |
![]() |
![]() |
![]() |
#4 |
creator of calibre
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 44,250
Karma: 23661992
Join Date: Oct 2006
Location: Mumbai, India
Device: Various
|
Looking at the rest of the driver, it should probably be 'false'
Try changing Code:
query = 'SELECT Name FROM Shelf WHERE _IsDeleted = "false"' to query = "SELECT Name FROM Shelf WHERE _IsDeleted = 'false'" |
![]() |
![]() |
![]() |
#5 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
|
![]() |
![]() |
![]() |
#6 | |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
Quote:
![]() Last edited by beedaddy; 07-19-2024 at 09:54 AM. |
|
![]() |
![]() |
![]() |
#7 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
And btw, using 0 doesn't work with the Kobo device (Clara 2E).
![]() |
![]() |
![]() |
![]() |
#8 |
creator of calibre
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 44,250
Karma: 23661992
Join Date: Oct 2006
Location: Mumbai, India
Device: Various
|
|
![]() |
![]() |
![]() |
#9 |
creator of calibre
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 44,250
Karma: 23661992
Join Date: Oct 2006
Location: Mumbai, India
Device: Various
|
Hmm well then as far as I can tell the two firmwares are storing different things in that column. Which would imply that kobo is unlikely to ever move to that firmware.
|
![]() |
![]() |
![]() |
#10 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
So, again I'm not sure where to go from here. Perhaps someone who knows more about this driver could have a look a the changes, if they are problematic in any way. Or should I just create a pull request and see what happens?
|
![]() |
![]() |
![]() |
#11 |
creator of calibre
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 44,250
Karma: 23661992
Join Date: Oct 2006
Location: Mumbai, India
Device: Various
|
Wait a few days someone else might have something to say, if not create a PR. I dont have accesss to Kobo hardware it isnt sold in my country so I can only make general comments.
|
![]() |
![]() |
![]() |
#13 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
To sum that up, for me it looks like that, for whatever reason, the kobo firmware 4.x and thus the kobo touch driver use the strings "true" and "false" as boolean values. Which seems not to be correct in a sqlite sense. The (tolino) fw 5.x does it the correct way, using 0 and 1 as boolean values. To not break anything, I had to take care of that different behaviour.
If there are still no objections, I will create a PR soon. |
![]() |
![]() |
![]() |
#14 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 12,545
Karma: 74358018
Join Date: Nov 2007
Location: Toronto
Device: Libra H2O, Libra Colour
|
Has anyone checked to see if the database schema has changed from Kobo to Tolino?
Sent from my Pixel 7a using Tapatalk |
![]() |
![]() |
![]() |
#15 | |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 179
Karma: 131606
Join Date: Oct 2011
Location: Germany (south-west)
Device: Kobo Libra 2 & Clara 2E, Tolino shine 5th Gen
|
Quote:
As far as I can tell, a few columns have been added to the content table (isLcpEpub, familySharing, familySharingRelation, subscriptionType, metaDataJson and isAdobeDrm) as well as to the Bookmark table (Color and markupJson). And then there are a bunch of new tolino_* tables (tolino_author, tolino_bookmark, tolino_bookshelfRevision, tolino_comment, tolino_contentItem, tolino_contentItemPendingDeletions, tolino_dogear, tolino_drmKeysKobo, tolino_skoobeChapter, tolino_tag, tolino_resellerHandshake, tolino_familyMembers, tolino_contentItemExtraData, tolino_recommendations, tolino_contentItemAudioInfo, tolino_downloadInfo, tolino_skoobeReadingTime and tolino_languageConfig). I've attached a diff. |
|
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Bug with Subtitle in KoboTouch Driver 2.5.1 | JSWolf | Devices | 2 | 01-01-2021 09:35 AM |
It would be good between devices add TOLINO. | colo51 | Devices | 3 | 07-28-2017 11:31 AM |
Tolino II und Tolino-Tablet bereits im Herbst? | hamlok | Andere Lesegeräte | 2 | 06-19-2013 08:58 AM |
KoboTouch Firmware | jackastor | Kobo Reader | 7 | 11-14-2011 06:19 AM |
Touch problems with my kobotouch :( | distorteddisco | Kobo Reader | 7 | 07-03-2011 09:00 AM |