calibredb-list (wrong-type-argument char-or-string-p nil) about calibredb.el

[chenyanming]

Fix in 85dc26c * master origin/master Fix bug comments that have new lines can not be decode.

[akirak]

I encountered the same error. It would be better to use emacsql-sqlite for accessing the sqlite database.

[chenyanming]

I found the same issue, it is because some books' comments have new line charaters, the split-string function did not work. Sorry about that, I will fix them asap.

[chenyanming]

I encountered the same error. It would be better to use emacsql-sqlite for accessing the sqlite database.

I checked emacssql-sqlite, it seems can not decode some tables of calibredb metadata.db, for example the name column in data table. Any other better solutions?

[akirak]

I'm sorry for that. EmacsQL wasn't suitable for the purpose:

EmacSQL is not intended to interact with arbitrary databases. All text values must be stored as printed s-expressions.

magit/emacsql#30 (comment)

Rather than directly interacting with the database, why not use calibredb command to query the library? You can use the following command to get items in one's library in JSON:

calibredb list --for-machine [--with-library=DIR]

[akirak]

It's already been fixed. Thanks.

Why don't you close this issue?

[yuchen-lea]

Oh... I thought you could close the issue. Sorry for that.

[chenyanming]

I'm sorry for that. EmacsQL wasn't suitable for the purpose:

EmacSQL is not intended to interact with arbitrary databases. All text values must be stored as printed s-expressions.

skeeto/emacsql#30 (comment)

Rather than directly interacting with the database, why not use calibredb command to query the library? You can use the following command to get items in one's library in JSON:

calibredb list --for-machine [--with-library=DIR]

Changing tocalibredb --list could be next TODO item.

[akirak]

Changing tocalibredb --list could be next TODO item.

I don't think it's necessary. It's already working, and calibredb list doesn't produce sufficient information for the current features of this package. Thanks.