From cbb9edac1950e5d4c50c9b719a74a2914f440f80 Mon Sep 17 00:00:00 2001 From: Ozzie Isaacs Date: Sun, 21 Nov 2021 13:23:27 +0100 Subject: [PATCH] Bugfix search in books list Fix renaming upper to lowercase letters Update test results --- cps/editbooks.py | 2 +- cps/web.py | 2 +- test/Calibre-Web TestSummary_Linux.html | 2721 ++++++++++++++++++++--- 3 files changed, 2425 insertions(+), 300 deletions(-) diff --git a/cps/editbooks.py b/cps/editbooks.py index 3403f46f..929aa1a7 100644 --- a/cps/editbooks.py +++ b/cps/editbooks.py @@ -114,7 +114,7 @@ def search_objects_add(db_book_object, db_type, input_elements): type_elements = c_elements.value else: type_elements = c_elements.name - if inp_element.lower() == type_elements.lower(): # Lowercase check + if inp_element == type_elements: found = True break if not found: diff --git a/cps/web.py b/cps/web.py index cff0036e..c061af30 100644 --- a/cps/web.py +++ b/cps/web.py @@ -831,7 +831,7 @@ def list_books(): books = calibre_db.session.query(db.Books).filter(calibre_db.common_filters()).all() entries = calibre_db.get_checkbox_sorted(books, state, off, limit, order) elif search: - entries, filtered_count, __ = calibre_db.get_search_results(search, off, order, limit, *join) + entries, filtered_count, __ = calibre_db.get_search_results(search, off, [order,''], limit, *join) else: entries, __, __ = calibre_db.fill_indexpage((int(off) / (int(limit)) + 1), limit, db.Books, True, order, *join) diff --git a/test/Calibre-Web TestSummary_Linux.html b/test/Calibre-Web TestSummary_Linux.html index 1733a51a..95505cc4 100644 --- a/test/Calibre-Web TestSummary_Linux.html +++ b/test/Calibre-Web TestSummary_Linux.html @@ -37,20 +37,20 @@
-

Start Time: 2021-10-30 19:49:15

+

Start Time: 2021-11-21 07:25:47

-

Stop Time: 2021-10-30 23:31:02

+

Stop Time: 2021-11-21 10:30:41

-

Duration: 3h 2 min

+

Duration: 2h 25 min

@@ -102,12 +102,12 @@ - + TestAnonymous 13 - 13 - 0 + 12 0 + 1 0 Detail @@ -215,11 +215,33 @@ - +
TestAnonymous - test_guest_restricted_settings_visibility
- PASS + +
+ ERROR +
+ + + + @@ -872,11 +894,11 @@ - + TestEditBooks 35 - 34 - 0 + 30 + 4 0 1 @@ -904,11 +926,35 @@ - +
TestEditBooks - test_edit_category
- PASS + +
+ FAIL +
+ + + + @@ -1012,11 +1058,35 @@ - +
TestEditBooks - test_edit_publisher
- PASS + +
+ FAIL +
+ + + + @@ -1039,11 +1109,35 @@ - +
TestEditBooks - test_edit_series
- PASS + +
+ FAIL +
+ + + + @@ -1209,25 +1303,45 @@ - +
TestEditBooks - test_upload_cover_hdd
- PASS + +
+ FAIL +
+ + + + - + TestEditBooksList - 18 - 18 - 0 - 0 + 10 + 8 + 1 + 1 0 - Detail + Detail @@ -1253,7 +1367,7 @@ -
TestEditBooksList - test_bookslist_edit_comment
+
TestEditBooksList - test_bookslist_edit_languages
PASS @@ -1262,7 +1376,7 @@ -
TestEditBooksList - test_bookslist_edit_cust_category
+
TestEditBooksList - test_bookslist_edit_publisher
PASS @@ -1271,7 +1385,7 @@ -
TestEditBooksList - test_bookslist_edit_cust_comment
+
TestEditBooksList - test_bookslist_edit_series
PASS @@ -1280,7 +1394,7 @@ -
TestEditBooksList - test_bookslist_edit_cust_enum
+
TestEditBooksList - test_bookslist_edit_seriesindex
PASS @@ -1289,7 +1403,7 @@ -
TestEditBooksList - test_bookslist_edit_cust_float
+
TestEditBooksList - test_bookslist_edit_title
PASS @@ -1297,78 +1411,6 @@ - -
TestEditBooksList - test_bookslist_edit_cust_int
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_cust_ratings
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_cust_text
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_languages
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_publisher
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_series
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_seriesindex
- - PASS - - - - - - -
TestEditBooksList - test_bookslist_edit_title
- - PASS - - - - -
TestEditBooksList - test_list_visibility
@@ -1377,30 +1419,84 @@ - +
TestEditBooksList - test_restricted_rights
- PASS + +
+ ERROR +
+ + + + - +
TestEditBooksList - test_search_books_list
- PASS + +
+ FAIL +
+ + + + - + TestEditBooksOnGdrive 20 - 20 - 0 + 15 + 5 0 0 @@ -1428,11 +1524,35 @@ - +
TestEditBooksOnGdrive - test_edit_category
- PASS + +
+ FAIL +
+ + + + @@ -1518,11 +1638,35 @@ - +
TestEditBooksOnGdrive - test_edit_publisher
- PASS + +
+ FAIL +
+ + + + @@ -1536,11 +1680,35 @@ - +
TestEditBooksOnGdrive - test_edit_series
- PASS + +
+ FAIL +
+ + + + @@ -1572,20 +1740,60 @@ - +
TestEditBooksOnGdrive - test_upload_cover_hdd
- PASS + +
+ FAIL +
+ + + + - +
TestEditBooksOnGdrive - test_watch_metadata
- PASS + +
+ FAIL +
+ + + + @@ -1759,15 +1967,15 @@ - + TestFilePicker + 3 + 2 + 0 + 0 1 - 1 - 0 - 0 - 0 - Detail + Detail @@ -1781,6 +1989,41 @@ + + + +
TestFilePicker - test_filepicker_new_file
+ + +
+ SKIP +
+ + + + + + + + + + +
TestFilePicker - test_two_filepickers
+ + PASS + + + @@ -1849,12 +2092,12 @@ - + TestKoboSync 10 - 10 - 0 - 0 + 5 + 4 + 1 0 Detail @@ -1881,29 +2124,91 @@ - +
TestKoboSync - test_kobo_sync_selected_shelfs
- PASS + +
+ ERROR +
+ + + + - +
TestKoboSync - test_shelves_add_remove_books
- PASS + +
+ FAIL +
+ + + + - +
TestKoboSync - test_sync_changed_book
- PASS + +
+ FAIL +
+ + + + @@ -1926,11 +2231,31 @@ - +
TestKoboSync - test_sync_shelf
- PASS + +
+ FAIL +
+ + + + @@ -1944,22 +2269,42 @@ - +
TestKoboSync - test_sync_upload
- PASS + +
+ FAIL +
+ + + + - + TestKoboSyncBig 4 - 4 - 0 - 0 + 1 + 1 + 2 0 Detail @@ -1968,20 +2313,62 @@ - +
TestKoboSyncBig - test_kobo_sync_selected_shelfs
- PASS + +
+ ERROR +
+ + + + - +
TestKoboSyncBig - test_sync_changed_book
- PASS + +
+ ERROR +
+ + + + @@ -1995,22 +2382,42 @@ - +
TestKoboSyncBig - test_sync_shelf
- PASS + +
+ FAIL +
+ + + + - + TestLdapLogin 13 - 13 - 0 + 11 0 + 2 0 Detail @@ -2055,20 +2462,72 @@ - +
TestLdapLogin - test_LDAP_import
- PASS + +
+ ERROR +
+ + + + - +
TestLdapLogin - test_LDAP_import_memberfield
- PASS + +
+ ERROR +
+ + + + @@ -2400,12 +2859,12 @@ - + TestMergeBooksList 2 - 2 - 0 + 1 0 + 1 0 Detail @@ -2414,11 +2873,31 @@ - +
TestMergeBooksList - test_delete_book
- PASS + +
+ ERROR +
+ + + + @@ -3196,13 +3675,13 @@ - + TestUserList 18 + 0 + 0 18 0 - 0 - 0 Detail @@ -3210,164 +3689,1810 @@ - +
TestUserList - test_edit_user_email
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_list_visibility
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_admin_role
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_check_sort
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_denied_tags
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_download_role
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_button
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_email
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_kindle
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_language
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_locale
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_name
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_edit_visiblility
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_guest_edit
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_remove_admin
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_requests
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_search
- PASS + +
+ ERROR +
+ + + + - +
TestUserList - test_user_list_sort
- PASS + +
+ ERROR +
+ + + + + + + + + + + _ErrorHolder + 1 + 0 + 0 + 1 + 0 + + Detail + + + + + + + +
tearDownClass (test_user_list)
+ + +
+ ERROR +
+ + + + @@ -3381,13 +5506,13 @@ 0 0 - Detail + Detail - +
TestUserLoad - test_user_change_vis
@@ -3405,13 +5530,13 @@ 0 0 - Detail + Detail - +
TestUserTemplate - test_allow_column_restriction
@@ -3420,7 +5545,7 @@ - +
TestUserTemplate - test_allow_tag_restriction
@@ -3429,7 +5554,7 @@ - +
TestUserTemplate - test_archived_format_template
@@ -3438,7 +5563,7 @@ - +
TestUserTemplate - test_author_user_template
@@ -3447,7 +5572,7 @@ - +
TestUserTemplate - test_best_user_template
@@ -3456,7 +5581,7 @@ - +
TestUserTemplate - test_category_user_template
@@ -3465,7 +5590,7 @@ - +
TestUserTemplate - test_deny_column_restriction
@@ -3474,7 +5599,7 @@ - +
TestUserTemplate - test_deny_tag_restriction
@@ -3483,7 +5608,7 @@ - +
TestUserTemplate - test_detail_random_user_template
@@ -3492,7 +5617,7 @@ - +
TestUserTemplate - test_download_user_template
@@ -3501,7 +5626,7 @@ - +
TestUserTemplate - test_format_user_template
@@ -3510,7 +5635,7 @@ - +
TestUserTemplate - test_hot_user_template
@@ -3519,7 +5644,7 @@ - +
TestUserTemplate - test_language_user_template
@@ -3528,7 +5653,7 @@ - +
TestUserTemplate - test_limit_book_languages
@@ -3537,7 +5662,7 @@ - +
TestUserTemplate - test_list_user_template
@@ -3546,7 +5671,7 @@ - +
TestUserTemplate - test_publisher_user_template
@@ -3555,7 +5680,7 @@ - +
TestUserTemplate - test_random_user_template
@@ -3564,7 +5689,7 @@ - +
TestUserTemplate - test_read_user_template
@@ -3573,7 +5698,7 @@ - +
TestUserTemplate - test_recent_user_template
@@ -3582,7 +5707,7 @@ - +
TestUserTemplate - test_series_user_template
@@ -3591,7 +5716,7 @@ - +
TestUserTemplate - test_ui_language_settings
@@ -3609,13 +5734,13 @@ 0 0 - Detail + Detail - +
TestCalibreWebVisibilitys - test_about
@@ -3624,7 +5749,7 @@ - +
TestCalibreWebVisibilitys - test_admin_SMTP_Settings
@@ -3633,7 +5758,7 @@ - +
TestCalibreWebVisibilitys - test_admin_add_user
@@ -3642,7 +5767,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_password
@@ -3651,7 +5776,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_archived
@@ -3660,7 +5785,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_authors
@@ -3669,7 +5794,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_category
@@ -3678,7 +5803,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_file_formats
@@ -3687,7 +5812,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_hot
@@ -3696,7 +5821,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_language
@@ -3705,7 +5830,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_publisher
@@ -3714,7 +5839,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_random
@@ -3723,7 +5848,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_rated
@@ -3732,7 +5857,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_rating
@@ -3741,7 +5866,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_read
@@ -3750,7 +5875,7 @@ - +
TestCalibreWebVisibilitys - test_admin_change_visibility_series
@@ -3759,7 +5884,7 @@ - +
TestCalibreWebVisibilitys - test_allow_columns
@@ -3768,7 +5893,7 @@ - +
TestCalibreWebVisibilitys - test_allow_tags
@@ -3777,7 +5902,7 @@ - +
TestCalibreWebVisibilitys - test_archive_books
@@ -3786,7 +5911,7 @@ - +
TestCalibreWebVisibilitys - test_authors_max_settings
@@ -3795,7 +5920,7 @@ - +
TestCalibreWebVisibilitys - test_change_title
@@ -3804,7 +5929,7 @@ - +
TestCalibreWebVisibilitys - test_checked_logged_in
@@ -3813,7 +5938,7 @@ - +
TestCalibreWebVisibilitys - test_hide_custom_column
@@ -3822,7 +5947,7 @@ - +
TestCalibreWebVisibilitys - test_link_column_to_read_status
@@ -3831,7 +5956,7 @@ - +
TestCalibreWebVisibilitys - test_random_books_available
@@ -3840,7 +5965,7 @@ - +
TestCalibreWebVisibilitys - test_request_link_column_to_read_status
@@ -3849,7 +5974,7 @@ - +
TestCalibreWebVisibilitys - test_restrict_columns
@@ -3858,7 +5983,7 @@ - +
TestCalibreWebVisibilitys - test_restrict_tags
@@ -3867,7 +5992,7 @@ - +
TestCalibreWebVisibilitys - test_save_views_recent
@@ -3876,7 +6001,7 @@ - +
TestCalibreWebVisibilitys - test_search_functions
@@ -3885,7 +6010,7 @@ - +
TestCalibreWebVisibilitys - test_search_order
@@ -3894,7 +6019,7 @@ - +
TestCalibreWebVisibilitys - test_search_string
@@ -3903,7 +6028,7 @@ - +
TestCalibreWebVisibilitys - test_user_email_available
@@ -3912,7 +6037,7 @@ - +
TestCalibreWebVisibilitys - test_user_visibility_sidebar
@@ -3930,13 +6055,13 @@ 0 0 - Detail + Detail - +
TestCalibreHelper - test_author_sort
@@ -3945,7 +6070,7 @@ - +
TestCalibreHelper - test_author_sort_comma
@@ -3954,7 +6079,7 @@ - +
TestCalibreHelper - test_author_sort_junior
@@ -3963,7 +6088,7 @@ - +
TestCalibreHelper - test_author_sort_oneword
@@ -3972,7 +6097,7 @@ - +
TestCalibreHelper - test_author_sort_roman
@@ -3981,7 +6106,7 @@ - +
TestCalibreHelper - test_check_Limit_Length
@@ -3990,7 +6115,7 @@ - +
TestCalibreHelper - test_check_char_replacement
@@ -3999,7 +6124,7 @@ - +
TestCalibreHelper - test_check_chinese_Characters
@@ -4008,7 +6133,7 @@ - +
TestCalibreHelper - test_check_deg_eur_replacement
@@ -4017,7 +6142,7 @@ - +
TestCalibreHelper - test_check_doubleS
@@ -4026,7 +6151,7 @@ - +
TestCalibreHelper - test_check_finish_Dot
@@ -4035,7 +6160,7 @@ - +
TestCalibreHelper - test_check_high23
@@ -4044,7 +6169,7 @@ - +
TestCalibreHelper - test_check_umlauts
@@ -4053,7 +6178,7 @@ - +
TestCalibreHelper - test_random_password
@@ -4062,7 +6187,7 @@ - +
TestCalibreHelper - test_split_authors
@@ -4071,7 +6196,7 @@ - +
TestCalibreHelper - test_whitespaces
@@ -4082,11 +6207,11 @@ Total - 366 - 360 - 0 - 0 - 6 + 361 + 312 + 15 + 27 + 7   @@ -4114,7 +6239,7 @@ Platform - Linux 5.11.0-38-generic #42~20.04.1-Ubuntu SMP Tue Sep 28 20:41:07 UTC 2021 x86_64 x86_64 + Linux 5.13.0-21-generic #21~20.04.1-Ubuntu SMP Tue Oct 26 15:49:20 UTC 2021 x86_64 x86_64 Basic @@ -4240,7 +6365,7 @@ SQLAlchemy - 1.4.26 + 1.4.27 Basic @@ -4270,13 +6395,13 @@ google-api-python-client - 2.28.0 + 2.31.0 TestCliGdrivedb httplib2 - 0.20.1 + 0.20.2 TestCliGdrivedb @@ -4300,13 +6425,13 @@ google-api-python-client - 2.28.0 + 2.31.0 TestEbookConvertCalibreGDrive httplib2 - 0.20.1 + 0.20.2 TestEbookConvertCalibreGDrive @@ -4330,13 +6455,13 @@ google-api-python-client - 2.28.0 + 2.31.0 TestEbookConvertGDriveKepubify httplib2 - 0.20.1 + 0.20.2 TestEbookConvertGDriveKepubify @@ -4372,13 +6497,13 @@ google-api-python-client - 2.28.0 + 2.31.0 TestEditBooksOnGdrive httplib2 - 0.20.1 + 0.20.2 TestEditBooksOnGdrive @@ -4402,13 +6527,13 @@ google-api-python-client - 2.28.0 + 2.31.0 TestSetupGdrive httplib2 - 0.20.1 + 0.20.2 TestSetupGdrive @@ -4444,13 +6569,13 @@ jsonschema - 4.1.2 + 4.2.1 TestKoboSync jsonschema - 4.1.2 + 4.2.1 TestKoboSyncBig @@ -4462,7 +6587,7 @@ jsonschema - 4.1.2 + 4.2.1 TestLdapLogin @@ -4474,7 +6599,7 @@ Flask-Dance - 5.0.0 + 5.1.0 TestOAuthLogin @@ -4492,7 +6617,7 @@