File browser: sort by metadata#13437
Conversation
|
Books' props are not cached. Can be added if we feel that requesting CoverBrowser database is not fast enough for folders with many files. |
|
Not familiar with sorting, and it's all a lot of refactoring, so trusting you :) (We may have complain by users saying authors and keywords are lists - so sorting/filtering is a bit tedious. May be the title should say "by first keyword", "by first author" - but this is a bit ugly.) |
|
Yes, mostly refactoring based on our flexible collate system. To get document props, collate |
|
|
|
Usually the author's name is in the form "first name-surname". If we sort the files by "Authors", I think it would make more sense to sort by surname. So I think the last name in the author's name order should be used for classification. |
|
Author field is not separated, just a string "John Smith". |
|
I see that this is done in the Pocketbook library when books are sorted by author's name and I assumed that there was a way to set the "last word" in the Author field as a sorting factor. If that's not possible, no problem. |
|
@hius07 https://www.reddit.com/r/koreader/comments/1jkghai/file_search_in_browser_missing/ Haven't tested myself yet, can help later |
|
Thanks, shall fix today. |
|
@ichnilatis-gr if you want to test sorting by author's last name, here is the userpatch: |
Thank you! It's not bad. It even struck me that sometimes, even though the surname is first (surname; first name), it is sometimes correctly classified according to the surname... Will it be merged? |
|
Maybe. If you meet an author that produces incorrect surname, please send this string to me. |
Is this a different file? I see that it has a different name from the booklist.lua you gave me a few days ago. Has something changed? I can save it in the same folder? |
|
booklist.lua is a program module, it will be overwritten by updates. |
Which means what? :-) |
|
So, Can I make a folder koreader\patches and it put the file there? In this case does it function despite the booklist.lua? |
|
Dear @hius07, did I understand correctly about the patches? I am sorry for my ignorance and constant annoyance. |
|
Yep. |
I was thinking of deleting my perhaps silly questions today after seeing the patches work in practice, but you beat me to it. :-) |
Closes #3864. Closes #7875. Closes #10957.
This change is