Merge all deleteBranch as one function and also fix bug when delete branch don't close related PRs

[lunny]

As title.

[jpraet]

CloseBranchPulls is already called in PushUpdate here:

gitea/services/repository/push.go

Line 216 in ab77a24

if err = pull_service.CloseBranchPulls(pusher, repo.ID, branch); err != nil {

[lunny]

Yes, but it's safe to run it twice and there is slight performance reduce. The invoke in push.go will be invoked from both git command and DeleteBranch and others.

[codecov-commenter]

Codecov Report

Merging #16067 (4b948c7) into main (3607f79) will decrease coverage by 0.02%.
The diff coverage is 43.07%.

@@            Coverage Diff             @@
##             main   #16067      +/-   ##
==========================================
- Coverage   44.19%   44.17%   -0.03%     
==========================================
  Files         683      684       +1     
  Lines       82375    82335      -40     
==========================================
- Hits        36409    36373      -36     
- Misses      40071    40074       +3     
+ Partials     5895     5888       -7     

Impacted Files	Coverage Δ
routers/repo/pull.go	30.37% <0.00%> (-0.79%)	⬇️
routers/repo/branch.go	56.97% <35.71%> (+1.45%)	⬆️
routers/api/v1/repo/branch.go	38.14% <44.44%> (-0.34%)	⬇️
services/repository/branch.go	61.29% <61.29%> (ø)
modules/queue/queue_channel.go	91.66% <0.00%> (-5.00%)	⬇️
modules/charset/charset.go	71.71% <0.00%> (-4.05%)	⬇️
services/pull/temp_repo.go	26.59% <0.00%> (-3.20%)	⬇️
services/pull/check.go	26.02% <0.00%> (-2.74%)	⬇️
modules/process/manager.go	72.83% <0.00%> (-2.47%)	⬇️
services/pull/patch.go	54.23% <0.00%> (-1.70%)	⬇️
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3607f79...4b948c7. Read the comment docs.