Can't reply to review comment of deleted file #2584

Closed
opened 2025-11-02 04:41:11 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @iyzana on GitHub (Nov 27, 2018).

  • Gitea version (or commit ref): 0da8bc9 / tag v1.6.0
  • Git version: 2.15.3
  • Operating system: gitea docker container
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

Steps to reproduce:

  • create a new pr, which adds a new file
  • add a review to that file
  • delete the file again
  • reply to the review comment

Repro url: https://try.gitea.io/succcubbus/review-deletion-outdated-repro/pulls/1

Expected:

  • reply is added to the review

Actual:

  • It is not possible to add a reply to the review comment
  • It causes a 500 server error
  • Relevant log (new lines on executing request):
2018/11/27 16:19:45 [D] Session ID: <redacted>
2018/11/27 16:19:45 [D] CSRF Token: <redacted>
2018/11/27 16:19:45 [.../repo/pull_review.go:80 CreateCodeComment()] [E] CreateCodeComment: LineBlame[refs/pull/115/head, /data/git/repositories/<repo-name-redacted>.git, <deleted-file-path-redacted>, 5]: exit status 128
- fatal: no such path <deleted-file-path-redacted> in refs/pull/115/head
Originally created by @iyzana on GitHub (Nov 27, 2018). <!-- 1. Please speak English, this is the language all of us can speak and write. 2. Please ask questions or configuration/deploy problems on our Discord server (https://discord.gg/NsatcWJ) or forum (https://discourse.gitea.io). 3. Please take a moment to check that your issue doesn't already exist. 4. Please give all relevant information below for bug reports, because incomplete details will be handled as an invalid report. --> - Gitea version (or commit ref): 0da8bc9 / tag v1.6.0 - Git version: 2.15.3 - Operating system: gitea docker container - Database (use `[x]`): - [ ] PostgreSQL - [ ] MySQL - [ ] MSSQL - [x] SQLite - Can you reproduce the bug at https://try.gitea.io: - [x] Yes (provide example URL) - [ ] No - [ ] Not relevant - Log gist: ## Description Steps to reproduce: - create a new pr, which adds a new file - add a review to that file - delete the file again - reply to the review comment Repro url: https://try.gitea.io/succcubbus/review-deletion-outdated-repro/pulls/1 Expected: - reply is added to the review Actual: - It is not possible to add a reply to the review comment - It causes a 500 server error - Relevant log (new lines on executing request): ``` 2018/11/27 16:19:45 [D] Session ID: <redacted> 2018/11/27 16:19:45 [D] CSRF Token: <redacted> 2018/11/27 16:19:45 [.../repo/pull_review.go:80 CreateCodeComment()] [E] CreateCodeComment: LineBlame[refs/pull/115/head, /data/git/repositories/<repo-name-redacted>.git, <deleted-file-path-redacted>, 5]: exit status 128 - fatal: no such path <deleted-file-path-redacted> in refs/pull/115/head ```
GiteaMirror added the issue/confirmedtype/bug labels 2025-11-02 04:41:11 -06:00
Author
Owner

@stale[bot] commented on GitHub (Jan 26, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Jan 26, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
Author
Owner

@adelowo commented on GitHub (Jan 28, 2019):

This is a valid bug.

@adelowo commented on GitHub (Jan 28, 2019): This is a valid bug.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#2584