1
0
Fork 0
forgejo/routers
Zettat123 9a83aa28a3
Get rules by id when editing branch protection rule (#22932)
When users rename an existing branch protection rule, a new rule with
the new name will be created and the old rule will still exist.

![image](https://user-images.githubusercontent.com/15528715/219276442-d3c001ad-e693-44ec-9ad2-b33f2666b49b.png)

---

![image](https://user-images.githubusercontent.com/15528715/219276478-547c3b93-b3f1-4292-a1ef-c1b7747fe1bb.png)

The reason is that the `SettingsProtectedBranchPost` function only get
branch protection rule by name before updating or creating a rule. When
the rule name changes, the function cannot find the existing rule so it
will create a new rule rather than update the existing rule. To fix the
bug, the function should get rule by id first.

---------

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
2023-02-20 19:30:41 +08:00
..
api Refactor the setting to make unit test easier (#22405) 2023-02-20 00:12:01 +08:00
common Refactor the setting to make unit test easier (#22405) 2023-02-20 00:12:01 +08:00
install Refactor the setting to make unit test easier (#22405) 2023-02-20 00:12:01 +08:00
private Refactor the setting to make unit test easier (#22405) 2023-02-20 00:12:01 +08:00
utils Implement FSFE REUSE for golang files (#21840) 2022-11-27 18:20:29 +00:00
web Get rules by id when editing branch protection rule (#22932) 2023-02-20 19:30:41 +08:00
init.go Refactor the setting to make unit test easier (#22405) 2023-02-20 00:12:01 +08:00