Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

After update to v2.27.3 folders missing pagination #1070

Closed
jas8522 opened this issue Apr 4, 2023 · 8 comments
Closed

After update to v2.27.3 folders missing pagination #1070

jas8522 opened this issue Apr 4, 2023 · 8 comments

Comments

@jas8522
Copy link

jas8522 commented Apr 4, 2023

Describe the bug
The page numbers at the bottom of mail folder lists are missing when there should be pagination available.

To Reproduce
Steps to reproduce the behavior:

  1. Login to webmail and go to any folder where there are more messages than should fit on one page
  2. No option found at bottom of folder column providing option to change pages

Expected behavior
When the folder has enough messages for pagination to occur, the option to select page 2, 3, 4, etc. appears.

Screenshots
missing-pagination

Please complete the following information:

  • Browser: Firefox 111.0.1
  • IMAP daemon: Dovecot
  • PHP version: 8.1
  • SnappyMail Version: 2.27.3
@Zero3K
Copy link

Zero3K commented Apr 4, 2023

Are you using a dark theme? I am using a light theme and can still see the page numbers.

@the-djmaze
Copy link
Owner

The problem must be the STATUS command.
When using SELECT or EXAMINE it should return the amount of MESSAGES.

Enable logging/debug and check the log for the results.

@jas8522
Copy link
Author

jas8522 commented Apr 4, 2023

Here's the SELECT from the debug logs when tested on Archive folder.

[2023-04-04 21:20:19.356][f0f42f81] IMAP[INFO]: < * STATUS INBOX.Archive (MESSAGES 48226 UIDNEXT 121568 UIDVALIDITY 1279666258 UNSEEN 0 HIGHESTMODSEQ 89106)\r\n

And a little further down showing it has the count and page limit:

[2023-04-04 21:20:19.397][f0f42f81] [INFO]: List optimization (count: 48226, limit:50)
[2023-04-04 21:20:19.399][f0f42f81] [INFO]: Get Serialized IDS from cache ("INBOX.Archive" /  / 48226,48225,48224,48223,48222,48221,48220,48219,48218,48217,48216,48215,48214,48213,48212,48211,48210,48209,48208,48207,48206,48205,48204,48203,48202,48201,48200,48199,48198,48197,48196,48195,48194,48193,48192,48191,48190,48189,48188,48187,48186,48185,48184,48183,48182,48181,48180,48179,48178,48177,48176,48175,48174,48173,48172,48171,48170,48169,48168,48167 UNDELETED) [count:60]
[2023-04-04 21:20:19.400][f0f42f81] IMAP[INFO]: > TAG6 FETCH 48226,48225,48224,48223,48222,48221,48218,48217,48216,48215,48214,48213,48212,48211,48210,48209,48208,48207,48219,48204 (UID RFC822.SIZE INTERNALDATE FLAGS BODYSTRUCTURE BODY.PEEK[HEADER.FIELDS (RETURN-PATH RECEIVED MIME-VERSION MESSAGE-ID CONTENT-TYPE FROM TO CC BCC SENDER REPLY-TO DELIVERED-TO IN-REPLY-TO REFERENCES DATE SUBJECT X-MSMAIL-PRIORITY IMPORTANCE X-PRIORITY X-DRAFT-INFO RETURN-RECEIPT-TO DISPOSITION-NOTIFICATION-TO X-CONFIRM-READING-TO AUTHENTICATION-RESULTS X-DKIM-AUTHENTICATION-RESULTS LIST-UNSUBSCRIBE AUTOCRYPT X-SPAM-STATUS X-SPAMD-RESULT X-BOGOSITY X-VIRUS X-VIRUS-SCANNED X-VIRUS-STATUS)])\r\n

Snappy certainly seems to know how many messages there are and be grabbing the first page successfully.

When I downgrade to 2.24.4, it works as expected with page numbers appearing as normal. Back to 2.27.3 and no more page numbers.

@jas8522
Copy link
Author

jas8522 commented Apr 4, 2023

Just in case the issue was with some of my customizations, I checked this by reverting to the default theme, cleared server-side and browser cache: No difference.

@the-djmaze
Copy link
Owner

Go to Admin -> Domains
Click on the domain you use.
What value does the messages limit says?
Is it solved when set to 0?

@jas8522
Copy link
Author

jas8522 commented Apr 4, 2023

It was set to 50, and yep setting it to 0 fixes it :)

I was under the impression that was a per page limit, but apparently not. Thanks for the tip!

@jas8522 jas8522 closed this as completed Apr 4, 2023
@jas8522
Copy link
Author

jas8522 commented Apr 4, 2023

Although; if our per page limit is 20, shouldn't it still at least have 3 pages total?

Edit: I set the limit to 500 and no pagination, so I guess that's not the solution but rather a key to where the issue lies?

@jas8522 jas8522 reopened this Apr 4, 2023
@the-djmaze
Copy link
Owner

Although; if our per page limit is 20, shouldn't it still at least have 3 pages total?

Edit: I set the limit to 500 and no pagination, so I guess that's not the solution but rather a key to where the issue lies?

Could be, i will look at it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants