AlistGo/alist

Incorrect search result count (total) returned

Open

#4,973 opened on Aug 9, 2023

View on GitHub
 (11 comments) (0 reactions) (0 assignees)Go (49,469 stars) (7,947 forks)batch import
bughelp wanted

Description

Please make sure of the following things

  • I have read the documentation. 我已经阅读了文档。

  • I'm sure there are no duplicate issues or discussions. 我确定没有重复的issue或讨论。

  • I'm sure it's due to AList and not something else(such as Network ,Dependencies or Operational). 我确定是AList的问题，而不是其他原因（例如网络，依赖或操作）。

  • I'm sure this issue is not fixed in the latest version. 我确定这个问题在最新版本中没有被修复。

AList Version / AList 版本

v3.25.1

Driver used / 使用的存储驱动

Any

Describe the bug / 问题描述

reason: https://github.com/alist-org/alist/blob/b91ed7a78ae255f427a7480e2a713ac8a7289032/server/handles/search.go#L46 the total is assigned before going through the meta to see what is hided

effect: image image

Reproduction / 复现链接

http://digitalocean1.itshenryz.com:5244/

Config / 配置

Unnecessary

Logs / 日志

Unnecessary

Contributor guide

Tech stack
go
Domain
backend
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
basic Go knowledge
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
The bug is in server/handles/search.go at line 46, where the total count is assigned before filtering hidden items via metadata. The fix should move the total assignment to after the meta filtering logic. Review the search handler to ensure all hidden items are excluded from the count.