Fix add team on collaborator page when same name as organization

[davidsvantesson]

Fix #9662

I think the intention of this code was to ignore if user tries to add "Owner" team to repository. The owner team is added to the repo anyhow and will be detected as a duplicate, so this check is not needed.

[6543]

should we backport this to 1.10.3 ?

[lunny]

@6543 I think we should.

[codecov-io]

Codecov Report

Merging #9767 into master will decrease coverage by 0.04%.
The diff coverage is 32.36%.

@@            Coverage Diff             @@
##           master    #9767      +/-   ##
==========================================
- Coverage    42.2%   42.16%   -0.05%     
==========================================
  Files         602      603       +1     
  Lines       78669    78834     +165     
==========================================
+ Hits        33206    33241      +35     
- Misses      41399    41522     +123     
- Partials     4064     4071       +7

Impacted Files	Coverage Δ
models/migrations/migrations.go	1.3% <ø> (ø)	⬆️
modules/migrations/base/pullrequest.go	0% <ø> (ø)	⬆️
models/user.go	50.46% <0%> (-0.26%)	⬇️
models/issue.go	54.81% <0%> (-0.62%)	⬇️
models/migrations/v123.go	0% <0%> (ø)
modules/migrations/gitea.go	6.89% <0%> (-1.03%)	⬇️
routers/repo/issue.go	38.01% <0%> (ø)	⬆️
models/migrate.go	0% <0%> (ø)	⬆️
models/external_login_user.go	21.29% <0%> (-0.61%)	⬇️
routers/api/v1/repo/issue_reaction.go	75.27% <100%> (ø)	⬆️
... and 7 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 009d375...7b7adad. Read the comment docs.

[6543]

@davidsvantesson can you send backport to 1.10.3?