troldspejlet/forgejo
troldspejlet/forgejo
1
0
Fork 0
mirror of https://codeberg.org/forgejo/forgejo.git synced 2024-11-10 20:59:31 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

Fix JS error when changing PR's target branch (#23862)

Browse source 
Caught by @justusbunsi 

An old bug from #6488

In `pullrequest_targetbranch_change`, the `data` might be empty, because
`UpdatePullRequestTarget` may respond `http.StatusNoContent`.

And the old code's `$branchTarget.text(data.base_branch);` doesn't make
sense, because in the end, the page will be always reloaded.

So, just remove the `$branchTarget.text(data.base_branch);`, everything
should be fine.
This commit is contained in:
wxiaoguang 2023-04-01 22:14:38 +08:00 committed by GitHub
parent aa9c920980
commit eadda68ded
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
web_src/js/features/repo-issue.js
View file

@ -645,8 +645,6 @@ export function initRepoIssueTitleEdit() {
$.post(update_url, { $.post(update_url, {
_csrf: csrfToken, _csrf: csrfToken,
target_branch: targetBranch target_branch: targetBranch
}).done((data) => {
$branchTarget.text(data.base_branch);
}).always(() => { }).always(() => {
window.location.reload(); window.location.reload();
}); });