Merge branch 'master' into 13682-review-requested-filter

Author: jpraet

models/issue.go

@@ -1717,7 +1717,7 @@ func SearchIssueIDsByKeyword(kw string, repoIDs []int64, limit, start int) (int6
 	)
 
 	var ids = make([]int64, 0, limit)
-	err := x.Distinct("id").Table("issue").Where(cond).Limit(limit, start).Find(&ids)
+	err := x.Distinct("id").Table("issue").Where(cond).OrderBy("`updated_unix` DESC").Limit(limit, start).Find(&ids)
 	if err != nil {
 		return 0, nil, err
 	}

models/issue_test.go

@@ -299,7 +299,7 @@ func TestIssue_SearchIssueIDsByKeyword(t *testing.T) {
 	total, ids, err = SearchIssueIDsByKeyword("for", []int64{1}, 10, 0)
 	assert.NoError(t, err)
 	assert.EqualValues(t, 5, total)
-	assert.EqualValues(t, []int64{1, 2, 3, 5, 11}, ids)
+	assert.ElementsMatch(t, []int64{1, 2, 3, 5, 11}, ids)
 
 	// issue1's comment id 2
 	total, ids, err = SearchIssueIDsByKeyword("good", []int64{1}, 10, 0)

models/project_board.go

@@ -8,6 +8,7 @@ import (
 	"code.gitea.io/gitea/modules/setting"
 	"code.gitea.io/gitea/modules/timeutil"
 
+	"xorm.io/builder"
 	"xorm.io/xorm"
 )
 
@@ -164,45 +165,99 @@ func UpdateProjectBoard(board *ProjectBoard) error {
 func updateProjectBoard(e Engine, board *ProjectBoard) error {
 	_, err := e.ID(board.ID).Cols(
 		"title",
-		"default",
 	).Update(board)
 	return err
 }
 
 // GetProjectBoards fetches all boards related to a project
-func GetProjectBoards(projectID int64) ([]*ProjectBoard, error) {
+// if no default board set, first board is a temporary "Uncategorized" board
+func GetProjectBoards(projectID int64) (ProjectBoardList, error) {
+	return getProjectBoards(x, projectID)
+}
 
+func getProjectBoards(e Engine, projectID int64) ([]*ProjectBoard, error) {
 	var boards = make([]*ProjectBoard, 0, 5)
 
-	sess := x.Where("project_id=?", projectID)
-	return boards, sess.Find(&boards)
+	if err := e.Where("project_id=? AND `default`=?", projectID, false).Find(&boards); err != nil {
+		return nil, err
+	}
+
+	defaultB, err := getDefaultBoard(e, projectID)
+	if err != nil {
+		return nil, err
+	}
+
+	return append([]*ProjectBoard{defaultB}, boards...), nil
 }
 
-// GetUncategorizedBoard represents a board for issues not assigned to one
-func GetUncategorizedBoard(projectID int64) (*ProjectBoard, error) {
+// getDefaultBoard return default board and create a dummy if none exist
+func getDefaultBoard(e Engine, projectID int64) (*ProjectBoard, error) {
+	var board ProjectBoard
+	exist, err := e.Where("project_id=? AND `default`=?", projectID, true).Get(&board)
+	if err != nil {
+		return nil, err
+	}
+	if exist {
+		return &board, nil
+	}
+
+	// represents a board for issues not assigned to one
 	return &ProjectBoard{
 		ProjectID: projectID,
 		Title:     "Uncategorized",
 		Default:   true,
 	}, nil
 }
 
+// SetDefaultBoard represents a board for issues not assigned to one
+// if boardID is 0 unset default
+func SetDefaultBoard(projectID, boardID int64) error {
+	sess := x
+
+	_, err := sess.Where(builder.Eq{
+		"project_id": projectID,
+		"`default`":  true,
+	}).Cols("`default`").Update(&ProjectBoard{Default: false})
+	if err != nil {
+		return err
+	}
+
+	if boardID > 0 {
+		_, err = sess.ID(boardID).Where(builder.Eq{"project_id": projectID}).
+			Cols("`default`").Update(&ProjectBoard{Default: true})
+	}
+
+	return err
+}
+
 // LoadIssues load issues assigned to this board
 func (b *ProjectBoard) LoadIssues() (IssueList, error) {
-	var boardID int64
-	if !b.Default {
-		boardID = b.ID
-
-	} else {
-		// Issues without ProjectBoardID
-		boardID = -1
-	}
-	issues, err := Issues(&IssuesOptions{
-		ProjectBoardID: boardID,
-		ProjectID:      b.ProjectID,
-	})
-	b.Issues = issues
-	return issues, err
+	issueList := make([]*Issue, 0, 10)
+
+	if b.ID != 0 {
+		issues, err := Issues(&IssuesOptions{
+			ProjectBoardID: b.ID,
+			ProjectID:      b.ProjectID,
+		})
+		if err != nil {
+			return nil, err
+		}
+		issueList = issues
+	}
+
+	if b.Default {
+		issues, err := Issues(&IssuesOptions{
+			ProjectBoardID: -1, // Issues without ProjectBoardID
+			ProjectID:      b.ProjectID,
+		})
+		if err != nil {
+			return nil, err
+		}
+		issueList = append(issueList, issues...)
+	}
+
+	b.Issues = issueList
+	return issueList, nil
 }
 
 // LoadIssues load issues assigned to the boards

modules/indexer/issues/bleve.go

@@ -247,6 +247,7 @@ func (b *BleveIndexer) Search(keyword string, repoIDs []int64, limit, start int)
 			newMatchPhraseQuery(keyword, "Comments", issueIndexerAnalyzer),
 		))
 	search := bleve.NewSearchRequestOptions(indexerQuery, limit, start, false)
+	search.SortBy([]string{"-_score"})
 
 	result, err := b.indexer.Search(search)
 	if err != nil {

modules/indexer/issues/elastic_search.go

@@ -205,7 +205,7 @@ func (b *ElasticSearchIndexer) Search(keyword string, repoIDs []int64, limit, st
 	searchResult, err := b.client.Search().
 		Index(b.indexerName).
 		Query(query).
-		Sort("id", true).
+		Sort("_score", false).
 		From(start).Size(limit).
 		Do(context.Background())
 	if err != nil {

modules/indexer/issues/indexer_test.go

@@ -65,7 +65,7 @@ func TestBleveSearchIssues(t *testing.T) {
 
 	ids, err = SearchIssuesByKeyword([]int64{1}, "for")
 	assert.NoError(t, err)
-	assert.EqualValues(t, []int64{1, 2, 3, 5, 11}, ids)
+	assert.ElementsMatch(t, []int64{1, 2, 3, 5, 11}, ids)
 
 	ids, err = SearchIssuesByKeyword([]int64{1}, "good")
 	assert.NoError(t, err)
@@ -89,7 +89,7 @@ func TestDBSearchIssues(t *testing.T) {
 
 	ids, err = SearchIssuesByKeyword([]int64{1}, "for")
 	assert.NoError(t, err)
-	assert.EqualValues(t, []int64{1, 2, 3, 5, 11}, ids)
+	assert.ElementsMatch(t, []int64{1, 2, 3, 5, 11}, ids)
 
 	ids, err = SearchIssuesByKeyword([]int64{1}, "good")
 	assert.NoError(t, err)

options/locale/locale_en-US.ini

@@ -945,6 +945,8 @@ projects.board.edit_title = "New Board Name"
 projects.board.new_title = "New Board Name"
 projects.board.new_submit = "Submit"
 projects.board.new = "New Board"
+projects.board.set_default = "Set Default"
+projects.board.set_default_desc = "Set this board as default for uncategorized issues and pulls"
 projects.board.delete = "Delete Board"
 projects.board.deletion_desc = "Deleting a project board moves all related issues to 'Uncategorized'. Continue?"
 projects.open = Open

options/locale/locale_ja-JP.ini

@@ -358,6 +358,7 @@ password_not_match=パスワードが一致しません。
 lang_select_error=言語をリストから選択してください。
 
 username_been_taken=ユーザー名が既に使用されています。
+username_change_not_local_user=非ローカルユーザーのユーザー名は変更できません。
 repo_name_been_taken=リポジトリ名が既に使用されています。
 repository_files_already_exist=このリポジトリのファイルはすでに存在します。システム管理者に問い合わせてください。
 repository_files_already_exist.adopt=このリポジトリのファイルはすでに存在しており、それらを登録することしかできません。
@@ -821,6 +822,8 @@ tag=タグ
 released_this=がこれをリリース
 file_raw=Raw
 file_history=履歴
+file_view_source=ソースを表示
+file_view_rendered=レンダリング表示
 file_view_raw=Rawデータを見る
 file_permalink=パーマリンク
 file_too_large=このファイルは大きすぎるため、表示できません。
@@ -1997,6 +2000,7 @@ dashboard=ダッシュボード
 users=ユーザーアカウント
 organizations=組織
 repositories=リポジトリ
+hooks=Webhook
 authentication=認証ソース
 emails=ユーザーメールアドレス
 config=設定
@@ -2146,8 +2150,13 @@ repos.forks=フォーク
 repos.issues=課題
 repos.size=サイズ
 
+defaulthooks=デフォルトWebhook
+defaulthooks.desc=Webhookは、特定のGiteaイベントトリガーが発生した際に、自動的にHTTP POSTリクエストをサーバーへ送信するものです。 ここで定義されたWebhookはデフォルトとなり、全ての新規リポジトリにコピーされます。 詳しくは<a target="_blank" rel="noopener" href="https://docs.gitea.io/en-us/webhooks/">webhooks guide</a>をご覧下さい。
+defaulthooks.add_webhook=デフォルトWebhookの追加
+defaulthooks.update_webhook=デフォルトWebhookの更新
 
 systemhooks=システムWebhook
+systemhooks.desc=Webhookは、特定のGiteaイベントトリガーが発生した際に、自動的にHTTP POSTリクエストをサーバーへ送信するものです。 ここで定義したWebhookはシステム内のすべてのリポジトリで呼び出されます。 そのため、パフォーマンスに及ぼす影響を考慮したうえで設定してください。 詳しくは<a target="_blank" rel="noopener" href="https://docs.gitea.io/en-us/webhooks/">webhooks guide</a>をご覧下さい。
 systemhooks.add_webhook=システムWebhookを追加
 systemhooks.update_webhook=システムWebhookを更新
 

options/locale/locale_tr-TR.ini

@@ -358,6 +358,7 @@ password_not_match=Parolalar uyuşmuyor.
 lang_select_error=Listeden bir dil seçin.
 
 username_been_taken=Bu kullanıcı adı daha önce alınmış.
+username_change_not_local_user=Yerel olmayan kullanıcılar kendi kullanıcı adlarını değiştiremezler.
 repo_name_been_taken=Depo adı zaten kullanılıyor.
 repository_files_already_exist=Bu depo için dosyalar zaten var. Sistem yöneticisine başvurun.
 repository_files_already_exist.adopt=Bu depo için dosyalar zaten var ve yalnızca Kabul Edilebilir.
@@ -821,6 +822,8 @@ tag=Etiket
 released_this=bu sürümü yayınladı
 file_raw=Ham
 file_history=Geçmiş
+file_view_source=Kaynağı Görüntüle
+file_view_rendered=Oluşturulanları Görüntüle
 file_view_raw=Ham Görünüm
 file_permalink=Kalıcı Bağlantı
 file_too_large=Bu dosya görüntülemek için çok büyük.
@@ -941,6 +944,8 @@ projects.board.edit_title=Yeni Pano Adı
 projects.board.new_title=Yeni Pano Adı
 projects.board.new_submit=Gönder
 projects.board.new=Yeni Pano
+projects.board.set_default=Varsayılana Ayarla
+projects.board.set_default_desc=Kategorize edilmemiş konular ve çekme istekleri için bu panoyu varsayılan olarak ayarlayın
 projects.board.delete=Panoyu Sil
 projects.board.deletion_desc=Bir proje panosunun silinmesi, ilgili tüm konuları 'Kategorize edilmemiş'e taşır. Devam edilsin mi?
 projects.open=Aç
@@ -1482,7 +1487,7 @@ settings.use_external_issue_tracker=Harici Konu İzleyici Kullan
 settings.external_tracker_url=Harici Konu İzleyici URLsi
 settings.external_tracker_url_error=Harici konu izleyici URL'si geçerli bir URL değil.
 settings.external_tracker_url_desc=Ziyaretçiler, konular sekmesine tıkladığında harici konu izleyici URL'sine yönlendirilir.
-settings.tracker_url_format=Harici Sorun Takipçisi Bağlantı Formatı
+settings.tracker_url_format=Harici Konu İzleyici URL Biçimi
 settings.tracker_url_format_error=Harici konu izleyici URL biçimi geçerli bir URL değil.
 settings.tracker_issue_style=Harici Konu İzleyici Numara Biçimi
 settings.tracker_issue_style.numeric=Sayısal
@@ -1997,6 +2002,7 @@ dashboard=Pano
 users=Kullanıcı Hesapları
 organizations=Organizasyonlar
 repositories=Depolar
+hooks=Web İstemcileri
 authentication=Yetkilendirme Kaynakları
 emails=Kullanıcı E-postaları
 config=Yapılandırma
@@ -2084,7 +2090,7 @@ users.full_name=Tam İsim
 users.activated=Aktifleştirilmiş
 users.admin=Yönetici
 users.restricted=Kısıtlanmış
-users.2fa=2ED
+users.2fa=2FD
 users.repos=Depolar
 users.created=Oluşturuldu
 users.last_login=Son Oturum Açma
@@ -2113,6 +2119,7 @@ users.delete_account=Kullanıcı Hesabını Sil
 users.still_own_repo=Bu kullanıcı hala bir veya daha fazla depoya sahip. Önce bu depoları silin veya transfer edin.
 users.still_has_org=Bu kullanıcı bir organizasyonun üyesidir. Önce kullanıcıyı tüm organizasyonlardan çıkarın.
 users.deletion_success=Kullanıcı hesabı silindi.
+users.reset_2fa=2FD'yi sıfırla
 
 emails.email_manage_panel=Kullanıcı E-posta Yönetimi
 emails.primary=Birincil
@@ -2145,8 +2152,13 @@ repos.forks=Çatallar
 repos.issues=Konular
 repos.size=Boyut
 
+defaulthooks=Varsayılan Web İstemcileri
+defaulthooks.desc=Web İstemcileri, belirli Gitea olayları tetiklendiğinde otomatik olarak HTTP POST isteklerini sunucuya yapar. Burada tanımlanan Web İstemcileri varsayılandır ve tüm yeni depolara kopyalanır. <a target="_blank" rel="noopener" href="https://docs.gitea.io/en-us/webhooks/">web istemcileri kılavuzunda</a> daha fazla bilgi edinin.
+defaulthooks.add_webhook=Varsayılan Web İstemcisi Ekle
+defaulthooks.update_webhook=Varsayılan Web İstemcisini Güncelle
 
 systemhooks=Sistem Web İstemcileri
+systemhooks.desc=Belirli Gitea olayları tetiklendiğinde Web istemcileri otomatik olarak bir sunucuya HTTP POST istekleri yapar. Burada tanımlanan web istemcileri sistemdeki tüm depolar üzerinde çalışır, bu yüzden lütfen bunun olabilecek tüm performans sonuçlarını göz önünde bulundurun. <a target="_blank" rel="noopener" href="https://docs.gitea.io/en-us/webhooks/">web istemcileri kılavuzunda</a> daha fazla bilgi edinin.
 systemhooks.add_webhook=Sistem Web İstemcisi Ekle
 systemhooks.update_webhook=Sistem Web İstemcisi Güncelle
 
@@ -2489,6 +2501,7 @@ mirror_sync_delete=<a href="%[1]s">%[3]s</a> adresindeki <code>%[2]s</code> refe
 approve_pull_request=`<a href="%s/pulls/%s">%s#%[2]s</a> onaylandı`
 reject_pull_request=`<a href="%s/pulls/%s">%s#%[2]s</a> için değişiklik önerdi `
 publish_release=` <a href="%[1]s">%[3]s</a> deposunda <a href="%s/releases/tag/%s"> "%[4]s" </a> sürümü yayınlandı`
+create_branch=<a href="%[1]s/src/branch/%[2]s">%[3]s</a> dalını <a href="%[1]s">%[4]s</a> içinde oluşturdu
 
 [tool]
 ago=%s önce

options/locale/locale_zh-TW.ini

@@ -943,6 +943,8 @@ projects.board.edit_title=新看板名稱
 projects.board.new_title=新看板名稱
 projects.board.new_submit=送出
 projects.board.new=新增看板
+projects.board.set_default=設為預設
+projects.board.set_default_desc=將此看板設定為未分類問題及合併請求的預設看板
 projects.board.delete=刪除看板
 projects.board.deletion_desc=刪除專案看板會將相關的問題移動到 '未分類'。是否繼續？
 projects.open=開啟
@@ -1999,6 +2001,7 @@ dashboard=資訊主頁
 users=使用者帳戶
 organizations=組織
 repositories=儲存庫
+hooks=Webhook
 authentication=認證來源
 emails=使用者電子信箱
 config=組態
@@ -2148,8 +2151,13 @@ repos.forks=Fork 數
 repos.issues=問題數
 repos.size=大小
 
+defaulthooks=預設 Webhook
+defaulthooks.desc=當觸發某些 Gitea 事件時，Webhook 會自動發出 HTTP POST 請求到指定的伺服器。這裡所定義的 Webhook 是預設的，並且會複製到所有新儲存庫。在 <a target="_blank" rel="noopener" href="https://docs.gitea.io/en-us/webhooks/">Webhook 指南</a>閱讀更多內容。
+defaulthooks.add_webhook=新增預設 Webhook
+defaulthooks.update_webhook=更新預設 Webhook
 
 systemhooks=系統 Webhook
+systemhooks.desc=當觸發某些 Gitea 事件時，Webhook 會自動發出 HTTP POST 請求到指定的伺服器。由於這裡所定義的 Webhook 會影響此系統上的所有儲存庫，因此請評估這會對效能造成多少影響。在 <a target="_blank" rel="noopener" href="https://docs.gitea.io/en-us/webhooks/">Webhook 指南</a>閱讀更多內容。
 systemhooks.add_webhook=新增系統 Webhook
 systemhooks.update_webhook=更新系統 Webhook
 

routers/api/v1/api.go

@@ -640,7 +640,7 @@ func RegisterRoutes(m *macaron.Macaron) {
 			m.Group("/:username/:reponame", func() {
 				m.Combo("").Get(reqAnyRepoReader(), repo.Get).
 					Delete(reqToken(), reqOwner(), repo.Delete).
-					Patch(reqToken(), reqAdmin(), bind(api.EditRepoOption{}), context.RepoRefForAPI(), repo.Edit)
+					Patch(reqToken(), reqAdmin(), context.RepoRefForAPI(), bind(api.EditRepoOption{}), repo.Edit)
 				m.Post("/transfer", reqOwner(), bind(api.TransferRepoOption{}), repo.Transfer)
 				m.Combo("/notifications").
 					Get(reqToken(), notify.ListRepoNotifications).
@@ -713,8 +713,8 @@ func RegisterRoutes(m *macaron.Macaron) {
 								Delete(reqToken(), repo.DeleteIssueComment)
 							m.Combo("/reactions").
 								Get(repo.GetIssueCommentReactions).
-								Post(bind(api.EditReactionOption{}), reqToken(), repo.PostIssueCommentReaction).
-								Delete(bind(api.EditReactionOption{}), reqToken(), repo.DeleteIssueCommentReaction)
+								Post(reqToken(), bind(api.EditReactionOption{}), repo.PostIssueCommentReaction).
+								Delete(reqToken(), bind(api.EditReactionOption{}), repo.DeleteIssueCommentReaction)
 						})
 					})
 					m.Group("/:index", func() {
@@ -754,8 +754,8 @@ func RegisterRoutes(m *macaron.Macaron) {
 						})
 						m.Combo("/reactions").
 							Get(repo.GetIssueReactions).
-							Post(bind(api.EditReactionOption{}), reqToken(), repo.PostIssueReaction).
-							Delete(bind(api.EditReactionOption{}), reqToken(), repo.DeleteIssueReaction)
+							Post(reqToken(), bind(api.EditReactionOption{}), repo.PostIssueReaction).
+							Delete(reqToken(), bind(api.EditReactionOption{}), repo.DeleteIssueReaction)
 					})
 				}, mustEnableIssuesOrPulls)
 				m.Group("/labels", func() {