From d2860fc3749524d4942f0808a856ef543b5caa5f Mon Sep 17 00:00:00 2001 From: Ozzie Isaacs Date: Thu, 15 Aug 2024 18:35:48 +0200 Subject: [PATCH] Final fix for cover size during kobo sync (fix for #2523) Update Teststatus --- cps/kobo.py | 9 +- test/Calibre-Web TestSummary_Linux.html | 694 ++++++++++++++++-------- 2 files changed, 475 insertions(+), 228 deletions(-) diff --git a/cps/kobo.py b/cps/kobo.py index 03fc29c3..8903056e 100644 --- a/cps/kobo.py +++ b/cps/kobo.py @@ -47,7 +47,7 @@ import requests from . import config, logger, kobo_auth, db, calibre_db, helper, shelf as shelf_lib, ub, csrf, kobo_sync_status from . import isoLanguages from .epub import get_epub_layout -from .constants import COVER_THUMBNAIL_SMALL +from .constants import COVER_THUMBNAIL_SMALL, COVER_THUMBNAIL_MEDIUM from .helper import get_download_link from .services import SyncToken as SyncToken from .web import download_required @@ -903,7 +903,12 @@ def get_current_bookmark_response(current_bookmark): @requires_kobo_auth def HandleCoverImageRequest(book_uuid, width, height, Quality, isGreyscale): try: - resolution = None if int(height) > 1000 else COVER_THUMBNAIL_SMALL + if int(height) > 1000: + resolution = None + elif int(height) > 500: + resolution = COVER_THUMBNAIL_MEDIUM + else: + resolution = COVER_THUMBNAIL_SMALL except ValueError: log.error("Requested height %s of book %s is invalid" % (book_uuid, height)) resolution = COVER_THUMBNAIL_SMALL diff --git a/test/Calibre-Web TestSummary_Linux.html b/test/Calibre-Web TestSummary_Linux.html index fc1ed2e3..726e2f9b 100644 --- a/test/Calibre-Web TestSummary_Linux.html +++ b/test/Calibre-Web TestSummary_Linux.html @@ -37,20 +37,20 @@
-

Start Time: 2024-08-04 21:24:21

+

Start Time: 2024-08-14 19:34:04

-

Stop Time: 2024-08-05 04:56:20

+

Stop Time: 2024-08-15 02:42:49

-

Duration: 6h 17 min

+

Duration: 6h 0 min

@@ -930,43 +930,84 @@ - - TestEbookConvertKepubify - 3 - 3 + + _FailedTest + 2 0 0 + 2 0 - Detail + Detail - + -
TestEbookConvertKepubify - test_convert_deactivate
+
_FailedTest - test_ebook_convert_kepubify
+ + +
+ ERROR +
+ + + - PASS - + -
TestEbookConvertKepubify - test_convert_only
+
_FailedTest - test_pip_install
- PASS - - - - - - -
TestEbookConvertKepubify - test_convert_wrong_excecutable
+ +
+ ERROR +
+ + + - PASS @@ -1228,13 +1269,13 @@ TestEditBooks - 38 + 37 36 0 0 - 2 + 1 - Detail + Detail @@ -1437,33 +1478,7 @@ - - -
TestEditBooks - test_rename_uppercase_lowercase
- - -
- SKIP -
- - - - - - - - - +
TestEditBooks - test_typeahead_author
@@ -1472,7 +1487,7 @@ - +
TestEditBooks - test_typeahead_functions
@@ -1481,7 +1496,7 @@ - +
TestEditBooks - test_typeahead_language
@@ -1490,7 +1505,7 @@ - +
TestEditBooks - test_typeahead_publisher
@@ -1499,7 +1514,7 @@ - +
TestEditBooks - test_typeahead_series
@@ -1508,7 +1523,7 @@ - +
TestEditBooks - test_typeahead_tag
@@ -1517,7 +1532,7 @@ - +
TestEditBooks - test_upload_book_cb7
@@ -1526,7 +1541,7 @@ - +
TestEditBooks - test_upload_book_cbr
@@ -1535,7 +1550,7 @@ - +
TestEditBooks - test_upload_book_cbt
@@ -1544,7 +1559,7 @@ - +
TestEditBooks - test_upload_book_cbz
@@ -1553,7 +1568,7 @@ - +
TestEditBooks - test_upload_book_epub
@@ -1562,7 +1577,7 @@ - +
TestEditBooks - test_upload_book_fb2
@@ -1571,7 +1586,7 @@ - +
TestEditBooks - test_upload_book_lit
@@ -1580,7 +1595,7 @@ - +
TestEditBooks - test_upload_book_mobi
@@ -1589,7 +1604,7 @@ - +
TestEditBooks - test_upload_book_pdf
@@ -1598,7 +1613,7 @@ - +
TestEditBooks - test_upload_cbz_coverformats
@@ -1607,7 +1622,7 @@ - +
TestEditBooks - test_upload_cover_hdd
@@ -3921,48 +3936,6 @@ IndexError: list index out of range - - TestPipInstall - 3 - 3 - 0 - 0 - 0 - - Detail - - - - - - - -
TestPipInstall - test_command_start
- - PASS - - - - - - -
TestPipInstall - test_foldername_database_location
- - PASS - - - - - - -
TestPipInstall - test_module_start
- - PASS - - - - - TestReader 8 @@ -3971,13 +3944,13 @@ IndexError: list index out of range 0 1 - Detail + Detail - +
TestReader - test_cb7_reader
@@ -3986,7 +3959,7 @@ IndexError: list index out of range - +
TestReader - test_comic_MACOS_files
@@ -3995,7 +3968,7 @@ IndexError: list index out of range - +
TestReader - test_comic_reader
@@ -4004,7 +3977,7 @@ IndexError: list index out of range - +
TestReader - test_epub_reader
@@ -4013,7 +3986,7 @@ IndexError: list index out of range - +
TestReader - test_pdf_reader
@@ -4022,7 +3995,7 @@ IndexError: list index out of range - +
TestReader - test_single_file_comic
@@ -4031,7 +4004,7 @@ IndexError: list index out of range - +
TestReader - test_sound_listener
@@ -4040,7 +4013,7 @@ IndexError: list index out of range - +
TestReader - test_txt_reader
@@ -4058,13 +4031,13 @@ IndexError: list index out of range 0 0 - Detail + Detail - +
TestReadOnlyDatabase - test_readonly_path
@@ -4082,13 +4055,13 @@ IndexError: list index out of range 0 0 - Detail + Detail - +
TestRegister - test_forgot_password
@@ -4097,7 +4070,7 @@ IndexError: list index out of range - +
TestRegister - test_illegal_email
@@ -4106,7 +4079,7 @@ IndexError: list index out of range - +
TestRegister - test_limit_domain
@@ -4115,7 +4088,7 @@ IndexError: list index out of range - +
TestRegister - test_register_no_server
@@ -4124,7 +4097,7 @@ IndexError: list index out of range - +
TestRegister - test_registering_only_email
@@ -4133,7 +4106,7 @@ IndexError: list index out of range - +
TestRegister - test_registering_user
@@ -4142,7 +4115,7 @@ IndexError: list index out of range - +
TestRegister - test_registering_user_fail
@@ -4151,7 +4124,7 @@ IndexError: list index out of range - +
TestRegister - test_user_change_password
@@ -4169,13 +4142,13 @@ IndexError: list index out of range 0 0 - Detail + Detail - +
TestReverseProxy - test_logout
@@ -4184,7 +4157,7 @@ IndexError: list index out of range - +
TestReverseProxy - test_move_page
@@ -4193,7 +4166,7 @@ IndexError: list index out of range - +
TestReverseProxy - test_next
@@ -4202,7 +4175,7 @@ IndexError: list index out of range - +
TestReverseProxy - test_reverse_about
@@ -4220,13 +4193,13 @@ IndexError: list index out of range 0 1 - Detail + Detail - +
TestShelf - test_access_shelf
@@ -4235,7 +4208,7 @@ IndexError: list index out of range - +
TestShelf - test_add_shelf_from_search
@@ -4244,7 +4217,7 @@ IndexError: list index out of range - +
TestShelf - test_adv_search_shelf
@@ -4253,7 +4226,7 @@ IndexError: list index out of range - +
TestShelf - test_arrange_shelf
@@ -4262,7 +4235,7 @@ IndexError: list index out of range - +
TestShelf - test_create_public_shelf
@@ -4271,7 +4244,7 @@ IndexError: list index out of range - +
TestShelf - test_create_public_shelf_no_permission
@@ -4280,7 +4253,7 @@ IndexError: list index out of range - +
TestShelf - test_delete_book_of_shelf
@@ -4289,7 +4262,7 @@ IndexError: list index out of range - +
TestShelf - test_private_shelf
@@ -4298,7 +4271,7 @@ IndexError: list index out of range - +
TestShelf - test_public_private_shelf
@@ -4307,7 +4280,7 @@ IndexError: list index out of range - +
TestShelf - test_public_shelf
@@ -4316,7 +4289,7 @@ IndexError: list index out of range - +
TestShelf - test_rename_shelf
@@ -4325,7 +4298,7 @@ IndexError: list index out of range - +
TestShelf - test_shelf_action_non_shelf_edit_role
@@ -4334,7 +4307,7 @@ IndexError: list index out of range - +
TestShelf - test_shelf_anonymous
@@ -4343,19 +4316,19 @@ IndexError: list index out of range - +
TestShelf - test_shelf_database_change
- SKIP + SKIP
-