Skip to content

When maxListCount limit is not reached, the wrong pop-up box prompts a maxListCount error #8846

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

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

java4candy
Copy link
Contributor

bugfix: When maxListCount limit is not reached, the wrong pop-up box prompts a maxListCount error

box.Ls(path) will count the same document repeatedly (x/x.sy).
Before checking maxListCount limit, assign len(ret) to the totals variable so that it only represents the number of .sy documents

Verified

This commit was signed with the committer’s verified signature.
gaborbernat Bernát Gábor
…prompts a maxListCount error

box.Ls(path) will count the same document repeatedly (x/x.sy).
Before checking maxListCount limit, assign len(ret) to the totals variable so that it only represents the number of .sy documents
@88250 88250 added the Bug label Jul 29, 2023
@88250 88250 added this to the 2.9.8 milestone Jul 29, 2023
@88250 88250 changed the title bugfix: When maxListCount limit is not reached, the wrong pop-up box prompts a maxListCount error When maxListCount limit is not reached, the wrong pop-up box prompts a maxListCount error Jul 29, 2023
@88250 88250 merged commit 0c621de into siyuan-note:dev Jul 29, 2023
@java4candy java4candy deleted the patch-1 branch July 29, 2023 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants