[PR #2611] [MERGED] Fix doubled issue tab introduced in migration v16 #16505

Closed
opened 2025-11-02 12:12:12 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/2611
Author: @daviian
Created: 9/26/2017
Status: Merged
Merged: 9/28/2017
Merged by: @lafriks

Base: masterHead: bugfix/doubled-issues-unit-type


📝 Commits (5)

  • 160f766 fix duplicate issue tab as UnitTypeIssue and UnitTypeExternalTracker are existing at the same time
  • 9f6c9b8 remove duplcate wiki unit types
  • 1329ebe fix lint error
  • 99d07b1 remove unnecessary repository table join
  • 4c0da6c Merge branch 'master' into bugfix/doubled-issues-unit-type

📊 Changes

2 files changed (+71 additions, -0 deletions)

View changed files

📝 models/migrations/migrations.go (+2 -0)
models/migrations/v44.go (+69 -0)

📄 Description

Migration v16 creates UnitTypeIssue and UnitTypeExternalTracker for each repository with enabled external issue tracker as repo.EnableIssues is true even for external tracker.

And because both Unit Types exist the issue tab is shown twice.

The same behaviour is happening to UnitTypeWiki and UnitTypeExternalWiki, but because the tab is not changed in view it has no visual impact.

PR fixes both mistakes and removes UnitTypeIssue for repositories with UniTypeExternalTracker and removes UnitTypeWiki for repositories with UnitTypeExternalWiki.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/2611 **Author:** [@daviian](https://github.com/daviian) **Created:** 9/26/2017 **Status:** ✅ Merged **Merged:** 9/28/2017 **Merged by:** [@lafriks](https://github.com/lafriks) **Base:** `master` ← **Head:** `bugfix/doubled-issues-unit-type` --- ### 📝 Commits (5) - [`160f766`](https://github.com/go-gitea/gitea/commit/160f766bce688f5b34d1fc44cbb80415423976d0) fix duplicate issue tab as UnitTypeIssue and UnitTypeExternalTracker are existing at the same time - [`9f6c9b8`](https://github.com/go-gitea/gitea/commit/9f6c9b8a4322d3897bcac8f6f34c0c2a284b41b8) remove duplcate wiki unit types - [`1329ebe`](https://github.com/go-gitea/gitea/commit/1329ebeb3866807ad504952fc9725e77122bde09) fix lint error - [`99d07b1`](https://github.com/go-gitea/gitea/commit/99d07b167793365a097e072fa928067a2419491a) remove unnecessary repository table join - [`4c0da6c`](https://github.com/go-gitea/gitea/commit/4c0da6cb373f04174071b7873eaa4f3b0e433060) Merge branch 'master' into bugfix/doubled-issues-unit-type ### 📊 Changes **2 files changed** (+71 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `models/migrations/migrations.go` (+2 -0) ➕ `models/migrations/v44.go` (+69 -0) </details> ### 📄 Description Migration v16 creates UnitTypeIssue and UnitTypeExternalTracker for each repository with enabled external issue tracker as repo.EnableIssues is true even for external tracker. And because both Unit Types exist the issue tab is shown twice. The same behaviour is happening to UnitTypeWiki and UnitTypeExternalWiki, but because the tab is not changed in view it has no visual impact. PR fixes both mistakes and removes UnitTypeIssue for repositories with UniTypeExternalTracker and removes UnitTypeWiki for repositories with UnitTypeExternalWiki. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 12:12:12 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#16505