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

Pull request search shows merged PRs in Open tab #30684

Closed
otbutz opened this issue Apr 24, 2024 · 4 comments · Fixed by #30715
Closed

Pull request search shows merged PRs in Open tab #30684

otbutz opened this issue Apr 24, 2024 · 4 comments · Fixed by #30715
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Milestone

Comments

@otbutz
Copy link

otbutz commented Apr 24, 2024

Description

Searching for a PR lists merged pull requests in the tab Open. The count is correct though.

Reproducer:

https://try.gitea.io/otbutz99/test/pulls?state=open&q=Add

Gitea Version

1.21.11

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

2024-04-24_13-31

Git Version

No response

Operating System

No response

How are you running Gitea?

Github binary started via systemd behind a caddy reverse proxy.

Database

PostgreSQL

@Zettat123
Copy link
Contributor

Zettat123 commented Apr 25, 2024

The results are correct when searching with no keywords. So I think the bug might be related to the issue indexer.
Can you try to rebuild issue indexer then check the search results?

(It is in Site Administration > Monitoring > Cron Tasks)

@otbutz
Copy link
Author

otbutz commented Apr 25, 2024

Can you try to rebuild issue indexer then check the search results?

Same result.

So I think the bug might be related to the issue indexer.

I can reproduce it locally and on try.gitea.io.

@wolfogre
Copy link
Member

wolfogre commented Apr 26, 2024

I have "Rebuild issue indexer" on try.gitea.io, the index has been fixed now.

This is not to say that bugs do not exist, but to help maintainers determine the cause of the problem.

This is a screenshot before I rebuilt the index.

xnip_240426_140938

@lunny lunny added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Apr 26, 2024
@otbutz
Copy link
Author

otbutz commented Apr 26, 2024

@wolfogre thanks! I tried to reproduce it again locally and things are now working as expected. Perhaps I was a bit impatient and refreshed before the task was actually finished. Either way, I can confirm that the cause is the outdated index.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants