2017-03-17 11:27:43 +05:30
|
|
|
<div class="four wide column">
|
|
|
|
<div class="ui segment metas">
|
2017-08-24 18:00:27 +05:30
|
|
|
{{template "repo/issue/branch_selector_field" .}}
|
|
|
|
|
2022-08-31 21:28:54 +05:30
|
|
|
{{if .Issue.IsPull}}
|
2020-04-06 22:03:34 +05:30
|
|
|
|
2021-02-28 19:35:55 +05:30
|
|
|
<input id="reviewer_id" name="reviewer_id" type="hidden" value="{{.reviewer_id}}">
|
|
|
|
<div class="ui {{if or (not .Reviewers) (not .CanChooseReviewer) .Repository.IsArchived}}disabled{{end}} floating jump select-reviewers-modify dropdown">
|
2023-02-13 23:29:59 +05:30
|
|
|
<a class="text gt-df gt-ac muted">
|
2022-06-28 02:28:46 +05:30
|
|
|
<strong>{{.locale.Tr "repo.issues.review.reviewers"}}</strong>
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if and .CanChooseReviewer (not .Repository.IsArchived)}}
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-gear" 16 "gt-ml-2"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{end}}
|
2021-05-17 01:48:18 +05:30
|
|
|
</a>
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="filter menu" data-action="update" data-issue-id="{{$.Issue.ID}}" data-update-url="{{$.RepoLink}}/issues/request_review">
|
2022-06-28 02:28:46 +05:30
|
|
|
<div class="header" style="text-transform: none;font-size:16px;">{{.locale.Tr "repo.issues.new.add_reviewer_title"}}</div>
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if .Reviewers}}
|
|
|
|
<div class="ui icon search input">
|
2023-02-13 23:29:59 +05:30
|
|
|
<i class="icon gt-df gt-ac gt-jc">{{svg "octicon-search" 16}}</i>
|
2022-06-28 02:28:46 +05:30
|
|
|
<input type="text" placeholder="{{.locale.Tr "repo.issues.filter_reviewers"}}">
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
{{if .Reviewers}}
|
|
|
|
{{range .Reviewers}}
|
|
|
|
{{if .User}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="{{if not .CanChange}}ui{{end}} item {{if .Checked}}checked{{end}} {{if not .CanChange}}ban-change{{end}}" href="#" data-id="{{.ItemID}}" data-id-selector="#review_request_{{.ItemID}}" {{if not .CanChange}} data-tooltip-content="{{$.locale.Tr "repo.issues.remove_request_review_block"}}"{{end}}>
|
2021-02-28 19:35:55 +05:30
|
|
|
<span class="octicon-check {{if not .Checked}}invisible{{end}}">{{svg "octicon-check"}}</span>
|
|
|
|
<span class="text">
|
2023-03-16 22:02:25 +05:30
|
|
|
{{avatar $.Context .User 28 "gt-mr-3"}}{{template "repo/search_name" .User}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</span>
|
|
|
|
</a>
|
|
|
|
{{end}}
|
2020-04-06 22:03:34 +05:30
|
|
|
{{end}}
|
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if .TeamReviewers}}
|
|
|
|
<div class="ui divider"></div>
|
|
|
|
{{range .TeamReviewers}}
|
|
|
|
{{if .Team}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="{{if not .CanChange}}ui{{end}} item {{if .Checked}}checked{{end}} {{if not .CanChange}}ban-change{{end}}" href="#" data-id="{{.ItemID}}" data-id-selector="#review_request_team_{{.Team.ID}}" {{if not .CanChange}} data-tooltip-content="{{$.locale.Tr "repo.issues.remove_request_review_block"}}"{{end}}>
|
2021-02-28 19:35:55 +05:30
|
|
|
<span class="octicon-check {{if not .Checked}}invisible{{end}}">{{svg "octicon-check" 16}}</span>
|
|
|
|
<span class="text">
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-people" 16 "gt-ml-4 gt-mr-2"}}{{$.Issue.Repo.OwnerName}}/{{.Team.Name}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</span>
|
|
|
|
</a>
|
|
|
|
{{end}}
|
2020-10-13 01:25:13 +05:30
|
|
|
{{end}}
|
2020-04-06 22:03:34 +05:30
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
2020-04-06 22:03:34 +05:30
|
|
|
</div>
|
|
|
|
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="ui assignees list">
|
2023-02-19 09:36:14 +05:30
|
|
|
<span class="no-select item {{if or .OriginalReviews .PullReviewers}}gt-hidden{{end}}">{{.locale.Tr "repo.issues.new.no_reviewers"}}</span>
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="selected">
|
|
|
|
{{range .PullReviewers}}
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="item gt-mb-2">
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if .User}}
|
|
|
|
<a class="muted sidebar-item-link" href="{{.User.HomeLink}}">
|
Add context cache as a request level cache (#22294)
To avoid duplicated load of the same data in an HTTP request, we can set
a context cache to do that. i.e. Some pages may load a user from a
database with the same id in different areas on the same page. But the
code is hidden in two different deep logic. How should we share the
user? As a result of this PR, now if both entry functions accept
`context.Context` as the first parameter and we just need to refactor
`GetUserByID` to reuse the user from the context cache. Then it will not
be loaded twice on an HTTP request.
But of course, sometimes we would like to reload an object from the
database, that's why `RemoveContextData` is also exposed.
The core context cache is here. It defines a new context
```go
type cacheContext struct {
ctx context.Context
data map[any]map[any]any
lock sync.RWMutex
}
var cacheContextKey = struct{}{}
func WithCacheContext(ctx context.Context) context.Context {
return context.WithValue(ctx, cacheContextKey, &cacheContext{
ctx: ctx,
data: make(map[any]map[any]any),
})
}
```
Then you can use the below 4 methods to read/write/del the data within
the same context.
```go
func GetContextData(ctx context.Context, tp, key any) any
func SetContextData(ctx context.Context, tp, key, value any)
func RemoveContextData(ctx context.Context, tp, key any)
func GetWithContextCache[T any](ctx context.Context, cacheGroupKey string, cacheTargetID any, f func() (T, error)) (T, error)
```
Then let's take a look at how `system.GetString` implement it.
```go
func GetSetting(ctx context.Context, key string) (string, error) {
return cache.GetWithContextCache(ctx, contextCacheKey, key, func() (string, error) {
return cache.GetString(genSettingCacheKey(key), func() (string, error) {
res, err := GetSettingNoCache(ctx, key)
if err != nil {
return "", err
}
return res.SettingValue, nil
})
})
}
```
First, it will check if context data include the setting object with the
key. If not, it will query from the global cache which may be memory or
a Redis cache. If not, it will get the object from the database. In the
end, if the object gets from the global cache or database, it will be
set into the context cache.
An object stored in the context cache will only be destroyed after the
context disappeared.
2023-02-15 19:07:34 +05:30
|
|
|
{{avatar $.Context .User 28 "gt-mr-3"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{.User.GetDisplayName}}
|
2020-04-06 22:03:34 +05:30
|
|
|
</a>
|
2021-02-28 19:35:55 +05:30
|
|
|
{{else if .Team}}
|
|
|
|
<span class="text">{{svg "octicon-people" 16 "teamavatar"}}{{$.Issue.Repo.OwnerName}}/{{.Team.Name}}</span>
|
2020-04-06 22:03:34 +05:30
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
<span class="ui right type-icon text {{if eq .Review.Type 1}}green
|
|
|
|
{{- else if eq .Review.Type 2}}grey
|
|
|
|
{{- else if eq .Review.Type 3}}red
|
|
|
|
{{- else if eq .Review.Type 4}}yellow
|
|
|
|
{{- else}}grey{{end}} right ">
|
|
|
|
|
|
|
|
{{if .CanChange}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a href="#" class="ui icon re-request-review {{if .Checked}}checked{{end}}" data-tooltip-content="{{if .Checked}} {{$.locale.Tr "repo.issues.remove_request_review"}} {{else}} {{$.locale.Tr "repo.issues.re_request_review"}} {{end}}" data-issue-id="{{$.Issue.ID}}" data-id="{{.ItemID}}" data-update-url="{{$.RepoLink}}/issues/request_review">
|
2021-03-22 09:34:19 +05:30
|
|
|
{{if .Checked}} {{svg "octicon-trash"}} {{else}} {{svg "octicon-sync"}} {{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</a>
|
|
|
|
{{end}}
|
|
|
|
{{svg (printf "octicon-%s" .Review.Type.Icon)}}
|
|
|
|
</span>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
{{range .OriginalReviews}}
|
|
|
|
<div class="item" style="margin-bottom: 10px;">
|
2023-03-24 16:05:38 +05:30
|
|
|
<a href="{{$.Repository.OriginalURL}}" data-tooltip-content="{{$.locale.Tr "repo.migrated_from_fake" ($.Repository.GetOriginalURLHostname|Escape) | Safe}}">
|
2021-09-18 21:52:51 +05:30
|
|
|
<span class="text black">
|
|
|
|
{{svg (MigrationIcon $.Repository.GetOriginalURLHostname)}}
|
2022-08-31 21:28:54 +05:30
|
|
|
{{.OriginalAuthor}}
|
2021-09-18 21:52:51 +05:30
|
|
|
</span>
|
|
|
|
</a>
|
2021-02-28 19:35:55 +05:30
|
|
|
<span class="ui right type-icon text {{if eq .Type 1}}green
|
|
|
|
{{- else if eq .Type 2}}grey
|
|
|
|
{{- else if eq .Type 3}}red
|
|
|
|
{{- else if eq .Type 4}}yellow
|
|
|
|
{{- else}}grey{{end}} right ">
|
|
|
|
{{svg (printf "octicon-%s" .Type.Icon)}}
|
|
|
|
</span>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
2020-04-06 22:03:34 +05:30
|
|
|
</div>
|
2021-05-28 01:32:04 +05:30
|
|
|
{{if and (or .HasIssuesOrPullsWritePermission .IsIssuePoster) (not .HasMerged) (not .Issue.IsClosed) (not .IsPullWorkInProgress)}}
|
2021-11-16 23:48:25 +05:30
|
|
|
<div class="toggle-wip" data-title="{{.Issue.Title}}" data-wip-prefix="{{(index .PullRequestWorkInProgressPrefixes 0| Escape)}}" data-update-url="{{.Issue.Link}}/title">
|
2021-05-28 01:32:04 +05:30
|
|
|
<a class="muted">
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.pulls.still_in_progress"}} {{.locale.Tr "repo.pulls.add_prefix" (index .PullRequestWorkInProgressPrefixes 0| Escape) | Safe}}
|
2021-05-28 01:32:04 +05:30
|
|
|
</a>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="ui divider"></div>
|
2020-06-17 19:42:06 +05:30
|
|
|
{{end}}
|
2020-04-06 22:03:34 +05:30
|
|
|
|
2023-03-13 02:09:56 +05:30
|
|
|
{{template "repo/issue/labels/labels_selector_field" .}}
|
2023-03-02 23:14:06 +05:30
|
|
|
{{template "repo/issue/labels/labels_sidebar" dict "root" $}}
|
2017-03-17 11:27:43 +05:30
|
|
|
|
|
|
|
<div class="ui divider"></div>
|
|
|
|
|
2020-04-04 11:09:48 +05:30
|
|
|
<div class="ui {{if or (not .HasIssuesOrPullsWritePermission) .Repository.IsArchived}}disabled{{end}} floating jump select-milestone dropdown">
|
2023-02-13 23:29:59 +05:30
|
|
|
<a class="text gt-df gt-ac muted">
|
2022-06-28 02:28:46 +05:30
|
|
|
<strong>{{.locale.Tr "repo.issues.new.milestone"}}</strong>
|
2020-04-04 11:09:48 +05:30
|
|
|
{{if and .HasIssuesOrPullsWritePermission (not .Repository.IsArchived)}}
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-gear" 16 "gt-ml-2"}}
|
2020-03-17 21:49:03 +05:30
|
|
|
{{end}}
|
2021-05-17 01:48:18 +05:30
|
|
|
</a>
|
2017-03-17 11:27:43 +05:30
|
|
|
<div class="menu" data-action="update" data-issue-id="{{$.Issue.ID}}" data-update-url="{{$.RepoLink}}/issues/milestone">
|
2023-04-30 18:42:49 +05:30
|
|
|
{{template "repo/issue/milestone/select_menu" .}}
|
2017-03-17 11:27:43 +05:30
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
<div class="ui select-milestone list">
|
2023-02-19 09:36:14 +05:30
|
|
|
<span class="no-select item {{if .Issue.Milestone}}gt-hidden{{end}}">{{.locale.Tr "repo.issues.new.no_milestone"}}</span>
|
2017-03-17 11:27:43 +05:30
|
|
|
<div class="selected">
|
|
|
|
{{if .Issue.Milestone}}
|
2020-11-29 11:52:04 +05:30
|
|
|
<a class="item muted sidebar-item-link" href="{{.RepoLink}}/milestone/{{.Issue.Milestone.ID}}">
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-milestone" 18 "gt-mr-3"}}
|
2020-11-29 11:52:04 +05:30
|
|
|
{{.Issue.Milestone.Name}}
|
|
|
|
</a>
|
2017-03-17 11:27:43 +05:30
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
|
2020-08-17 08:37:38 +05:30
|
|
|
{{if .IsProjectsEnabled}}
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="ui divider"></div>
|
2020-11-29 11:52:04 +05:30
|
|
|
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="ui {{if or (not .HasIssuesOrPullsWritePermission) .Repository.IsArchived}}disabled{{end}} floating jump select-project dropdown">
|
2023-02-13 23:29:59 +05:30
|
|
|
<a class="text gt-df gt-ac muted">
|
2022-06-28 02:28:46 +05:30
|
|
|
<strong>{{.locale.Tr "repo.issues.new.projects"}}</strong>
|
2021-03-12 20:44:02 +05:30
|
|
|
{{if and .HasIssuesOrPullsWritePermission (not .Repository.IsArchived)}}
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-gear" 16 "gt-ml-2"}}
|
2021-03-12 20:44:02 +05:30
|
|
|
{{end}}
|
2021-05-17 01:48:18 +05:30
|
|
|
</a>
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="menu" data-action="update" data-issue-id="{{$.Issue.ID}}" data-update-url="{{$.RepoLink}}/issues/projects">
|
2023-02-01 04:10:38 +05:30
|
|
|
<div class="header" style="text-transform: none;font-size:16px;">{{.locale.Tr "repo.issues.new.add_project_title"}}</div>
|
|
|
|
{{if or .OpenProjects .ClosedProjects}}
|
|
|
|
<div class="ui icon search input">
|
2023-02-13 23:29:59 +05:30
|
|
|
<i class="icon gt-df gt-ac gt-jc">{{svg "octicon-search" 16}}</i>
|
2023-02-01 04:10:38 +05:30
|
|
|
<input type="text" placeholder="{{.locale.Tr "repo.issues.filter_projects"}}">
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<div class="no-select item">{{.locale.Tr "repo.issues.new.clear_projects"}}</div>
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if .OpenProjects}}
|
|
|
|
<div class="divider"></div>
|
|
|
|
<div class="header">
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.new.open_projects"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
|
|
|
{{range .OpenProjects}}
|
2023-01-20 17:12:33 +05:30
|
|
|
<a class="item muted sidebar-item-link" data-id="{{.ID}}" data-href="{{.Link}}">
|
2023-03-19 18:14:48 +05:30
|
|
|
{{svg .IconName 18 "gt-mr-3"}}{{.Title}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</a>
|
|
|
|
{{end}}
|
2020-08-17 08:37:38 +05:30
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if .ClosedProjects}}
|
|
|
|
<div class="divider"></div>
|
|
|
|
<div class="header">
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.new.closed_projects"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
|
|
|
{{range .ClosedProjects}}
|
2023-01-20 17:12:33 +05:30
|
|
|
<a class="item muted sidebar-item-link" data-id="{{.ID}}" data-href="{{.Link}}">
|
2023-03-19 18:14:48 +05:30
|
|
|
{{svg .IconName 18 "gt-mr-3"}}{{.Title}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</a>
|
|
|
|
{{end}}
|
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
<div class="ui select-project list">
|
2023-02-19 09:36:14 +05:30
|
|
|
<span class="no-select item {{if .Issue.ProjectID}}gt-hidden{{end}}">{{.locale.Tr "repo.issues.new.no_projects"}}</span>
|
2021-02-28 19:35:55 +05:30
|
|
|
<div class="selected">
|
|
|
|
{{if .Issue.ProjectID}}
|
2023-01-20 17:12:33 +05:30
|
|
|
<a class="item muted sidebar-item-link" href="{{.Issue.Project.Link}}">
|
2023-03-19 18:14:48 +05:30
|
|
|
{{svg .Issue.Project.IconName 18 "gt-mr-3"}}{{.Issue.Project.Title}}
|
2020-11-29 11:52:04 +05:30
|
|
|
</a>
|
2020-08-17 08:37:38 +05:30
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
2020-08-17 08:37:38 +05:30
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
|
2017-03-17 11:27:43 +05:30
|
|
|
<div class="ui divider"></div>
|
|
|
|
|
|
|
|
<input id="assignee_id" name="assignee_id" type="hidden" value="{{.assignee_id}}">
|
2020-04-04 11:09:48 +05:30
|
|
|
<div class="ui {{if or (not .HasIssuesOrPullsWritePermission) .Repository.IsArchived}}disabled{{end}} floating jump select-assignees-modify dropdown">
|
2023-02-13 23:29:59 +05:30
|
|
|
<a class="text gt-df gt-ac muted">
|
2022-06-28 02:28:46 +05:30
|
|
|
<strong>{{.locale.Tr "repo.issues.new.assignees"}}</strong>
|
2020-04-04 11:09:48 +05:30
|
|
|
{{if and .HasIssuesOrPullsWritePermission (not .Repository.IsArchived)}}
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-gear" 16 "gt-ml-2"}}
|
2020-03-17 21:49:03 +05:30
|
|
|
{{end}}
|
2021-05-17 01:48:18 +05:30
|
|
|
</a>
|
2020-09-02 22:25:13 +05:30
|
|
|
<div class="filter menu" data-action="update" data-issue-id="{{$.Issue.ID}}" data-update-url="{{$.RepoLink}}/issues/assignee">
|
2022-06-28 02:28:46 +05:30
|
|
|
<div class="header" style="text-transform: none;font-size:16px;">{{.locale.Tr "repo.issues.new.add_assignees_title"}}</div>
|
2020-04-04 11:09:48 +05:30
|
|
|
<div class="ui icon search input">
|
2023-02-13 23:29:59 +05:30
|
|
|
<i class="icon gt-df gt-ac gt-jc">{{svg "octicon-search" 16}}</i>
|
2022-06-28 02:28:46 +05:30
|
|
|
<input type="text" placeholder="{{.locale.Tr "repo.issues.filter_assignees"}}">
|
2020-04-04 11:09:48 +05:30
|
|
|
</div>
|
2022-06-28 02:28:46 +05:30
|
|
|
<div class="no-select item">{{.locale.Tr "repo.issues.new.clear_assignees"}}</div>
|
2017-03-17 11:27:43 +05:30
|
|
|
{{range .Assignees}}
|
2018-05-09 21:59:04 +05:30
|
|
|
|
|
|
|
{{$AssigneeID := .ID}}
|
2020-11-12 11:25:34 +05:30
|
|
|
<a class="item{{range $.Issue.Assignees}}{{if eq .ID $AssigneeID}} checked{{end}}{{end}}" href="#" data-id="{{.ID}}" data-id-selector="#assignee_{{.ID}}">
|
2020-02-11 22:32:41 +05:30
|
|
|
{{$checked := false}}
|
|
|
|
{{range $.Issue.Assignees}}
|
|
|
|
{{if eq .ID $AssigneeID}}
|
|
|
|
{{$checked = true}}
|
|
|
|
{{end}}
|
2018-05-09 21:59:04 +05:30
|
|
|
{{end}}
|
2020-09-12 01:49:00 +05:30
|
|
|
<span class="octicon-check {{if not $checked}}invisible{{end}}">{{svg "octicon-check"}}</span>
|
2018-05-09 21:59:04 +05:30
|
|
|
<span class="text">
|
2023-04-07 05:41:02 +05:30
|
|
|
{{avatar $.Context . 20 "gt-mr-3"}}{{template "repo/search_name" .}}
|
2018-05-09 21:59:04 +05:30
|
|
|
</span>
|
|
|
|
</a>
|
2017-03-17 11:27:43 +05:30
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
2018-05-09 21:59:04 +05:30
|
|
|
<div class="ui assignees list">
|
2023-02-19 09:36:14 +05:30
|
|
|
<span class="no-select item {{if .Issue.Assignees}}gt-hidden{{end}}">{{.locale.Tr "repo.issues.new.no_assignees"}}</span>
|
2017-03-17 11:27:43 +05:30
|
|
|
<div class="selected">
|
2018-05-09 21:59:04 +05:30
|
|
|
{{range .Issue.Assignees}}
|
2020-11-29 11:52:04 +05:30
|
|
|
<div class="item">
|
|
|
|
<a class="muted sidebar-item-link" href="{{$.RepoLink}}/{{if $.Issue.IsPull}}pulls{{else}}issues{{end}}?assignee={{.ID}}">
|
Add context cache as a request level cache (#22294)
To avoid duplicated load of the same data in an HTTP request, we can set
a context cache to do that. i.e. Some pages may load a user from a
database with the same id in different areas on the same page. But the
code is hidden in two different deep logic. How should we share the
user? As a result of this PR, now if both entry functions accept
`context.Context` as the first parameter and we just need to refactor
`GetUserByID` to reuse the user from the context cache. Then it will not
be loaded twice on an HTTP request.
But of course, sometimes we would like to reload an object from the
database, that's why `RemoveContextData` is also exposed.
The core context cache is here. It defines a new context
```go
type cacheContext struct {
ctx context.Context
data map[any]map[any]any
lock sync.RWMutex
}
var cacheContextKey = struct{}{}
func WithCacheContext(ctx context.Context) context.Context {
return context.WithValue(ctx, cacheContextKey, &cacheContext{
ctx: ctx,
data: make(map[any]map[any]any),
})
}
```
Then you can use the below 4 methods to read/write/del the data within
the same context.
```go
func GetContextData(ctx context.Context, tp, key any) any
func SetContextData(ctx context.Context, tp, key, value any)
func RemoveContextData(ctx context.Context, tp, key any)
func GetWithContextCache[T any](ctx context.Context, cacheGroupKey string, cacheTargetID any, f func() (T, error)) (T, error)
```
Then let's take a look at how `system.GetString` implement it.
```go
func GetSetting(ctx context.Context, key string) (string, error) {
return cache.GetWithContextCache(ctx, contextCacheKey, key, func() (string, error) {
return cache.GetString(genSettingCacheKey(key), func() (string, error) {
res, err := GetSettingNoCache(ctx, key)
if err != nil {
return "", err
}
return res.SettingValue, nil
})
})
}
```
First, it will check if context data include the setting object with the
key. If not, it will query from the global cache which may be memory or
a Redis cache. If not, it will get the object from the database. In the
end, if the object gets from the global cache or database, it will be
set into the context cache.
An object stored in the context cache will only be destroyed after the
context disappeared.
2023-02-15 19:07:34 +05:30
|
|
|
{{avatar $.Context . 28 "gt-mr-3"}}
|
2020-11-12 11:25:34 +05:30
|
|
|
{{.GetDisplayName}}
|
|
|
|
</a>
|
2018-05-09 21:59:04 +05:30
|
|
|
</div>
|
2017-03-17 11:27:43 +05:30
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
|
|
|
|
<div class="ui divider"></div>
|
|
|
|
|
2020-12-08 08:50:12 +05:30
|
|
|
{{if .Participants}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text"><strong>{{.locale.Tr "repo.issues.num_participants" .NumParticipants}}</strong></span>
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="ui list gt-df gt-fw">
|
2017-03-17 11:27:43 +05:30
|
|
|
{{range .Participants}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a {{if gt .ID 0}}href="{{.HomeLink}}"{{end}} data-tooltip-content="{{.GetDisplayName}}">
|
Add context cache as a request level cache (#22294)
To avoid duplicated load of the same data in an HTTP request, we can set
a context cache to do that. i.e. Some pages may load a user from a
database with the same id in different areas on the same page. But the
code is hidden in two different deep logic. How should we share the
user? As a result of this PR, now if both entry functions accept
`context.Context` as the first parameter and we just need to refactor
`GetUserByID` to reuse the user from the context cache. Then it will not
be loaded twice on an HTTP request.
But of course, sometimes we would like to reload an object from the
database, that's why `RemoveContextData` is also exposed.
The core context cache is here. It defines a new context
```go
type cacheContext struct {
ctx context.Context
data map[any]map[any]any
lock sync.RWMutex
}
var cacheContextKey = struct{}{}
func WithCacheContext(ctx context.Context) context.Context {
return context.WithValue(ctx, cacheContextKey, &cacheContext{
ctx: ctx,
data: make(map[any]map[any]any),
})
}
```
Then you can use the below 4 methods to read/write/del the data within
the same context.
```go
func GetContextData(ctx context.Context, tp, key any) any
func SetContextData(ctx context.Context, tp, key, value any)
func RemoveContextData(ctx context.Context, tp, key any)
func GetWithContextCache[T any](ctx context.Context, cacheGroupKey string, cacheTargetID any, f func() (T, error)) (T, error)
```
Then let's take a look at how `system.GetString` implement it.
```go
func GetSetting(ctx context.Context, key string) (string, error) {
return cache.GetWithContextCache(ctx, contextCacheKey, key, func() (string, error) {
return cache.GetString(genSettingCacheKey(key), func() (string, error) {
res, err := GetSettingNoCache(ctx, key)
if err != nil {
return "", err
}
return res.SettingValue, nil
})
})
}
```
First, it will check if context data include the setting object with the
key. If not, it will query from the global cache which may be memory or
a Redis cache. If not, it will get the object from the database. In the
end, if the object gets from the global cache or database, it will be
set into the context cache.
An object stored in the context cache will only be destroyed after the
context disappeared.
2023-02-15 19:07:34 +05:30
|
|
|
{{avatar $.Context . 28 "gt-my-1 gt-mr-2"}}
|
2017-03-17 11:27:43 +05:30
|
|
|
</a>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
2020-12-08 08:50:12 +05:30
|
|
|
{{end}}
|
2017-03-30 05:01:47 +05:30
|
|
|
|
2019-01-24 00:28:38 +05:30
|
|
|
{{if and $.IssueWatch (not .Repository.IsArchived)}}
|
2017-09-12 12:18:13 +05:30
|
|
|
<div class="ui divider"></div>
|
2017-03-30 05:01:47 +05:30
|
|
|
|
2017-09-12 12:18:13 +05:30
|
|
|
<div class="ui watching">
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text"><strong>{{.locale.Tr "notification.notifications"}}</strong></span>
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="gt-mt-3">
|
2021-11-16 23:48:25 +05:30
|
|
|
<form method="POST" action="{{.Issue.Link}}/watch">
|
2023-03-27 21:35:51 +05:30
|
|
|
<input type="hidden" name="watch" value="{{if $.IssueWatch.IsWatching}}0{{else}}1{{end}}">
|
2017-09-12 12:18:13 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
2023-02-13 23:29:59 +05:30
|
|
|
<button class="fluid ui button gt-df gt-jc">
|
2017-09-12 12:18:13 +05:30
|
|
|
{{if $.IssueWatch.IsWatching}}
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-mute" 16 "gt-mr-3"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.unsubscribe"}}
|
2017-09-12 12:18:13 +05:30
|
|
|
{{else}}
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-unmute" 16 "gt-mr-3"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.subscribe"}}
|
2017-09-12 12:18:13 +05:30
|
|
|
{{end}}
|
|
|
|
</button>
|
|
|
|
</form>
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2022-12-10 08:16:31 +05:30
|
|
|
{{if .Repository.IsTimetrackerEnabled $.Context}}
|
2019-01-24 00:28:38 +05:30
|
|
|
{{if and .CanUseTimetracker (not .Repository.IsArchived)}}
|
2017-09-12 12:18:13 +05:30
|
|
|
<div class="ui divider"></div>
|
|
|
|
<div class="ui timetrack">
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text"><strong>{{.locale.Tr "repo.issues.tracker"}}</strong></span>
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="gt-mt-3">
|
2021-11-16 23:48:25 +05:30
|
|
|
<form method="POST" action="{{.Issue.Link}}/times/stopwatch/toggle" id="toggle_stopwatch_form">
|
2017-09-12 12:18:13 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
|
|
|
</form>
|
2021-11-16 23:48:25 +05:30
|
|
|
<form method="POST" action="{{.Issue.Link}}/times/stopwatch/cancel" id="cancel_stopwatch_form">
|
2017-09-12 12:18:13 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
|
|
|
</form>
|
|
|
|
{{if $.IsStopwatchRunning}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<button class="ui fluid button issue-stop-time">{{.locale.Tr "repo.issues.stop_tracking"}}</button>
|
2023-02-13 23:29:59 +05:30
|
|
|
<button class="ui fluid negative button issue-cancel-time gt-mt-3">{{.locale.Tr "repo.issues.cancel_tracking"}}</button>
|
2017-03-30 05:01:47 +05:30
|
|
|
{{else}}
|
2017-09-12 12:18:13 +05:30
|
|
|
{{if .HasUserStopwatch}}
|
|
|
|
<div class="ui warning message">
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.tracking_already_started" (.OtherStopwatchURL|Escape) | Safe}}
|
2017-09-12 12:18:13 +05:30
|
|
|
</div>
|
|
|
|
{{end}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<button class="ui fluid button issue-start-time" data-tooltip-content='{{.locale.Tr "repo.issues.start_tracking"}}'>{{.locale.Tr "repo.issues.start_tracking_short"}}</button>
|
2021-05-17 01:48:18 +05:30
|
|
|
<div class="ui mini modal issue-start-time-modal">
|
2022-06-28 02:28:46 +05:30
|
|
|
<div class="header">{{.locale.Tr "repo.issues.add_time"}}</div>
|
2021-05-17 01:48:18 +05:30
|
|
|
<div class="content">
|
2021-11-16 23:48:25 +05:30
|
|
|
<form method="POST" id="add_time_manual_form" action="{{.Issue.Link}}/times/add" class="ui action input fluid">
|
2021-05-17 01:48:18 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<input placeholder='{{.locale.Tr "repo.issues.add_time_hours"}}' type="number" name="hours">
|
|
|
|
<input placeholder='{{.locale.Tr "repo.issues.add_time_minutes"}}' type="number" name="minutes" class="ui compact">
|
2021-05-17 01:48:18 +05:30
|
|
|
</form>
|
|
|
|
</div>
|
|
|
|
<div class="actions">
|
2023-03-14 09:04:09 +05:30
|
|
|
<button class="ui green approve button">{{.locale.Tr "repo.issues.add_time_short"}}</button>
|
|
|
|
<button class="ui red cancel button">{{.locale.Tr "repo.issues.add_time_cancel"}}</button>
|
2017-09-12 12:18:13 +05:30
|
|
|
</div>
|
|
|
|
</div>
|
2023-03-24 16:05:38 +05:30
|
|
|
<button class="ui fluid button green issue-add-time gt-mt-3" data-tooltip-content='{{.locale.Tr "repo.issues.add_time"}}'>{{.locale.Tr "repo.issues.add_time_short"}}</button>
|
2017-03-30 05:01:47 +05:30
|
|
|
{{end}}
|
2017-09-12 12:18:13 +05:30
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
{{if gt (len .WorkingUsers) 0}}
|
|
|
|
<div class="ui divider"></div>
|
2020-12-08 08:50:12 +05:30
|
|
|
<div class="ui comments">
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text"><strong>{{.locale.Tr "repo.issues.time_spent_from_all_authors" ($.Issue.TotalTrackedTime | Sec2Time) | Safe}}</strong></span>
|
2017-09-12 12:18:13 +05:30
|
|
|
<div>
|
|
|
|
{{range $user, $trackedtime := .WorkingUsers}}
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="comment gt-mt-3">
|
2017-09-12 12:18:13 +05:30
|
|
|
<a class="avatar">
|
Add context cache as a request level cache (#22294)
To avoid duplicated load of the same data in an HTTP request, we can set
a context cache to do that. i.e. Some pages may load a user from a
database with the same id in different areas on the same page. But the
code is hidden in two different deep logic. How should we share the
user? As a result of this PR, now if both entry functions accept
`context.Context` as the first parameter and we just need to refactor
`GetUserByID` to reuse the user from the context cache. Then it will not
be loaded twice on an HTTP request.
But of course, sometimes we would like to reload an object from the
database, that's why `RemoveContextData` is also exposed.
The core context cache is here. It defines a new context
```go
type cacheContext struct {
ctx context.Context
data map[any]map[any]any
lock sync.RWMutex
}
var cacheContextKey = struct{}{}
func WithCacheContext(ctx context.Context) context.Context {
return context.WithValue(ctx, cacheContextKey, &cacheContext{
ctx: ctx,
data: make(map[any]map[any]any),
})
}
```
Then you can use the below 4 methods to read/write/del the data within
the same context.
```go
func GetContextData(ctx context.Context, tp, key any) any
func SetContextData(ctx context.Context, tp, key, value any)
func RemoveContextData(ctx context.Context, tp, key any)
func GetWithContextCache[T any](ctx context.Context, cacheGroupKey string, cacheTargetID any, f func() (T, error)) (T, error)
```
Then let's take a look at how `system.GetString` implement it.
```go
func GetSetting(ctx context.Context, key string) (string, error) {
return cache.GetWithContextCache(ctx, contextCacheKey, key, func() (string, error) {
return cache.GetString(genSettingCacheKey(key), func() (string, error) {
res, err := GetSettingNoCache(ctx, key)
if err != nil {
return "", err
}
return res.SettingValue, nil
})
})
}
```
First, it will check if context data include the setting object with the
key. If not, it will query from the global cache which may be memory or
a Redis cache. If not, it will get the object from the database. In the
end, if the object gets from the global cache or database, it will be
set into the context cache.
An object stored in the context cache will only be destroyed after the
context disappeared.
2023-02-15 19:07:34 +05:30
|
|
|
{{avatar $.Context $user}}
|
2017-09-12 12:18:13 +05:30
|
|
|
</a>
|
|
|
|
<div class="content">
|
2022-11-19 09:32:30 +05:30
|
|
|
{{template "shared/user/authorlink" $user}}
|
2017-09-12 12:18:13 +05:30
|
|
|
<div class="text">
|
|
|
|
{{$trackedtime}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2017-04-29 11:22:25 +05:30
|
|
|
{{end}}
|
2018-05-02 00:35:28 +05:30
|
|
|
|
|
|
|
<div class="ui divider"></div>
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text"><strong>{{.locale.Tr "repo.issues.due_date"}}</strong></span>
|
2018-07-16 18:13:00 +05:30
|
|
|
<div class="ui form" id="deadline-loader">
|
2023-02-19 09:36:14 +05:30
|
|
|
<div class="ui negative message gt-hidden" id="deadline-err-invalid-date">
|
2021-05-17 01:48:18 +05:30
|
|
|
{{svg "octicon-x" 16 "close icon"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.due_date_invalid"}}
|
2018-07-16 18:13:00 +05:30
|
|
|
</div>
|
|
|
|
{{if ne .Issue.DeadlineUnix 0}}
|
|
|
|
<p>
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="gt-df gt-sb gt-ac">
|
2023-03-24 16:05:38 +05:30
|
|
|
<div class="due-date {{if .Issue.IsOverdue}}text red{{end}}" {{if .Issue.IsOverdue}}data-tooltip-content="{{.locale.Tr "repo.issues.due_date_overdue"}}"{{end}}>
|
2023-02-13 23:29:59 +05:30
|
|
|
{{svg "octicon-calendar" 16 "gt-mr-3"}}
|
2023-04-26 01:18:30 +05:30
|
|
|
{{DateTime "long" .Issue.DeadlineUnix}}
|
2021-05-17 01:48:18 +05:30
|
|
|
</div>
|
|
|
|
<div>
|
|
|
|
{{if and .HasIssuesOrPullsWritePermission (not .Repository.IsArchived)}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="issue-due-edit muted" data-tooltip-content="{{$.locale.Tr "repo.issues.due_date_form_edit"}}">{{svg "octicon-pencil" 16 "gt-mr-2"}}</a>
|
|
|
|
<a class="issue-due-remove muted" data-tooltip-content="{{$.locale.Tr "repo.issues.due_date_form_remove"}}">{{svg "octicon-trash"}}</a>
|
2021-05-17 01:48:18 +05:30
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
2018-07-16 18:13:00 +05:30
|
|
|
</p>
|
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<p>{{.locale.Tr "repo.issues.due_date_not_set"}}</p>
|
2018-07-16 18:13:00 +05:30
|
|
|
{{end}}
|
2018-05-02 00:35:28 +05:30
|
|
|
|
2020-04-04 11:09:48 +05:30
|
|
|
{{if and .HasIssuesOrPullsWritePermission (not .Repository.IsArchived)}}
|
2023-02-19 09:36:14 +05:30
|
|
|
<div {{if ne .Issue.DeadlineUnix 0}} class="gt-hidden"{{end}} id="deadlineForm">
|
2022-04-08 00:29:56 +05:30
|
|
|
<form class="ui fluid action input issue-due-form" action="{{AppSubUrl}}/{{PathEscape .Repository.Owner.Name}}/{{PathEscape .Repository.Name}}/issues/{{.Issue.Index}}/deadline" method="post" id="update-issue-deadline-form">
|
2018-07-16 18:13:00 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<input required placeholder="{{.locale.Tr "repo.issues.due_date_form"}}" {{if gt .Issue.DeadlineUnix 0}}value="{{.Issue.DeadlineUnix.Format "2006-01-02"}}"{{end}} type="date" name="deadlineDate" id="deadlineDate">
|
2018-07-16 18:13:00 +05:30
|
|
|
<button class="ui green icon button">
|
|
|
|
{{if ne .Issue.DeadlineUnix 0}}
|
2021-05-17 01:48:18 +05:30
|
|
|
{{svg "octicon-pencil"}}
|
2018-07-16 18:13:00 +05:30
|
|
|
{{else}}
|
2021-05-17 01:48:18 +05:30
|
|
|
{{svg "octicon-plus"}}
|
2018-07-16 18:13:00 +05:30
|
|
|
{{end}}
|
|
|
|
</button>
|
|
|
|
</form>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
2018-07-18 02:53:58 +05:30
|
|
|
|
2022-12-10 08:16:31 +05:30
|
|
|
{{if .Repository.IsDependenciesEnabled $.Context}}
|
2018-07-18 02:53:58 +05:30
|
|
|
<div class="ui divider"></div>
|
|
|
|
|
|
|
|
<div class="ui depending">
|
2023-03-28 22:53:25 +05:30
|
|
|
{{if (and (not .BlockedByDependencies) (not .BlockedByDependenciesNotPermitted) (not .BlockingDependencies) (not .BlockingDependenciesNotPermitted))}}
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text"><strong>{{.locale.Tr "repo.issues.dependency.title"}}</strong></span>
|
2018-09-07 08:02:46 +05:30
|
|
|
<br>
|
2020-11-10 23:58:07 +05:30
|
|
|
<p>
|
|
|
|
{{if .Issue.IsPull}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.dependency.pr_no_dependencies"}}
|
2018-09-07 08:02:46 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.dependency.issue_no_dependencies"}}
|
2020-11-10 23:58:07 +05:30
|
|
|
{{end}}
|
|
|
|
</p>
|
2018-09-07 08:02:46 +05:30
|
|
|
{{end}}
|
|
|
|
|
2023-03-28 22:53:25 +05:30
|
|
|
{{if or .BlockingDependencies .BlockingDependenciesNotPermitted}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<span class="text" data-tooltip-content="{{if .Issue.IsPull}}{{.locale.Tr "repo.issues.dependency.pr_close_blocks"}}{{else}}{{.locale.Tr "repo.issues.dependency.issue_close_blocks"}}{{end}}">
|
2022-06-28 02:28:46 +05:30
|
|
|
<strong>{{.locale.Tr "repo.issues.dependency.blocks_short"}}</strong>
|
2018-07-18 02:53:58 +05:30
|
|
|
</span>
|
|
|
|
<div class="ui relaxed divided list">
|
2018-09-07 08:02:46 +05:30
|
|
|
{{range .BlockingDependencies}}
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="item dependency{{if .Issue.IsClosed}} is-closed{{end}} gt-df gt-ac gt-sb">
|
|
|
|
<div class="item-left gt-df gt-jc gt-fc gt-f1">
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="title" href="{{.Issue.Link}}" data-tooltip-content="#{{.Issue.Index}} {{.Issue.Title | RenderEmoji $.Context}}">
|
2023-03-06 03:29:05 +05:30
|
|
|
#{{.Issue.Index}} {{.Issue.Title | RenderEmoji $.Context}}
|
2020-11-10 23:58:07 +05:30
|
|
|
</a>
|
|
|
|
<div class="text small">
|
|
|
|
{{.Repository.OwnerName}}/{{.Repository.Name}}
|
|
|
|
</div>
|
|
|
|
</div>
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="item-right gt-df gt-ac">
|
2019-01-24 00:28:38 +05:30
|
|
|
{{if and $.CanCreateIssueDependencies (not $.Repository.IsArchived)}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="delete-dependency-button ci muted" data-id="{{.Issue.ID}}" data-type="blocking" data-tooltip-content="{{$.locale.Tr "repo.issues.dependency.remove_info"}}">
|
2021-03-22 09:34:19 +05:30
|
|
|
{{svg "octicon-trash" 16}}
|
2018-07-18 02:53:58 +05:30
|
|
|
</a>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2023-03-28 22:53:25 +05:30
|
|
|
{{if .BlockingDependenciesNotPermitted}}
|
|
|
|
<div class="item gt-df gt-ac gt-sb">
|
|
|
|
<span>{{$.locale.TrN (len .BlockingDependenciesNotPermitted) "repo.issues.dependency.no_permission_1" "repo.issues.dependency.no_permission_n" (len .BlockingDependenciesNotPermitted)}}</span>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2018-07-18 02:53:58 +05:30
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
|
2023-03-28 22:53:25 +05:30
|
|
|
{{if or .BlockedByDependencies .BlockedByDependenciesNotPermitted}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<span class="text" data-tooltip-content="{{if .Issue.IsPull}}{{.locale.Tr "repo.issues.dependency.pr_closing_blockedby"}}{{else}}{{.locale.Tr "repo.issues.dependency.issue_closing_blockedby"}}{{end}}">
|
2022-06-28 02:28:46 +05:30
|
|
|
<strong>{{.locale.Tr "repo.issues.dependency.blocked_by_short"}}</strong>
|
2018-07-18 02:53:58 +05:30
|
|
|
</span>
|
|
|
|
<div class="ui relaxed divided list">
|
2018-09-07 08:02:46 +05:30
|
|
|
{{range .BlockedByDependencies}}
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="item dependency{{if .Issue.IsClosed}} is-closed{{end}} gt-df gt-ac gt-sb">
|
|
|
|
<div class="item-left gt-df gt-jc gt-fc gt-f1">
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="title" href="{{.Issue.Link}}" data-tooltip-content="#{{.Issue.Index}} {{.Issue.Title | RenderEmoji $.Context}}">
|
2023-03-06 03:29:05 +05:30
|
|
|
#{{.Issue.Index}} {{.Issue.Title | RenderEmoji $.Context}}
|
2020-11-10 23:58:07 +05:30
|
|
|
</a>
|
|
|
|
<div class="text small">
|
|
|
|
{{.Repository.OwnerName}}/{{.Repository.Name}}
|
|
|
|
</div>
|
|
|
|
</div>
|
2023-02-13 23:29:59 +05:30
|
|
|
<div class="item-right gt-df gt-ac">
|
Allow cross-repository dependencies on issues (#7901)
* in progress changes for #7405, added ability to add cross-repo dependencies
* removed unused repolink var
* fixed query that was breaking ci tests; fixed check in issue dependency add so that the id of the issue and dependency is checked rather than the indexes
* reverted removal of string in local files becasue these are done via crowdin, not updated manually
* removed 'Select("issue.*")' from getBlockedByDependencies and getBlockingDependencies based on comments in PR review
* changed getBlockedByDependencies and getBlockingDependencies to use a more xorm-like query, also updated the sidebar as a result
* simplified the getBlockingDependencies and getBlockedByDependencies methods; changed the sidebar to show the dependencies in a different format where you can see the name of the repository
* made some changes to the issue view in the dependencies (issue name on top, repo full name on separate line). Change view of issue in the dependency search results (also showing the full repo name on separate line)
* replace call to FindUserAccessibleRepoIDs with SearchRepositoryByName. The former was hardcoded to use isPrivate = false on the repo search, but this code needed it to be true. The SearchRepositoryByName method is used more in the code including on the user's dashboard
* some more tweaks to the layout of the issues when showing dependencies and in the search box when you add new dependencies
* added Name to the RepositoryMeta struct
* updated swagger doc
* fixed total count for link header on SearchIssues
* fixed indentation
* fixed aligment of remove icon on dependencies in issue sidebar
* removed unnecessary nil check (unnecessary because issue.loadRepo is called prior to this block)
* reverting .css change, somehow missed or forgot that less is used
* updated less file and generated css; updated sidebar template with styles to line up delete and issue index
* added ordering to the blocked by/depends on queries
* fixed sorting in issue dependency search and the depends on/blocks views to show issues from the current repo first, then by created date descending; added a "all cross repository dependencies" setting to allow this feature to be turned off, if turned off, the issue dependency search will work the way it did before (restricted to the current repository)
* re-applied my swagger changes after merge
* fixed split string condition in issue search
* changed ALLOW_CROSS_REPOSITORY_DEPENDENCIES description to sound more global than just the issue dependency search; returning 400 in the cross repo issue search api method if not enabled; fixed bug where the issue count did not respect the state parameter
* when adding a dependency to an issue, added a check to make sure the issue and dependency are in the same repo if cross repo dependencies is not enabled
* updated sortIssuesSession call in PullRequests, another commit moved this method from pull.go to pull_list.go so I had to re-apply my change here
* fixed incorrect setting of user id parameter in search repos call
2019-10-31 10:36:10 +05:30
|
|
|
{{if and $.CanCreateIssueDependencies (not $.Repository.IsArchived)}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<a class="delete-dependency-button ci muted" data-id="{{.Issue.ID}}" data-type="blockedBy" data-tooltip-content="{{$.locale.Tr "repo.issues.dependency.remove_info"}}">
|
2021-03-22 09:34:19 +05:30
|
|
|
{{svg "octicon-trash" 16}}
|
2018-07-18 02:53:58 +05:30
|
|
|
</a>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2023-03-28 22:53:25 +05:30
|
|
|
{{if $.CanCreateIssueDependencies}}
|
|
|
|
{{range .BlockedByDependenciesNotPermitted}}
|
|
|
|
<div class="item dependency{{if .Issue.IsClosed}} is-closed{{end}} gt-df gt-ac gt-sb">
|
|
|
|
<div class="item-left gt-df gt-jc gt-fc gt-f1">
|
|
|
|
<div>
|
|
|
|
<span data-tooltip-content="{{$.locale.Tr "repo.issues.dependency.no_permission.can_remove"}}">{{svg "octicon-lock" 16}}</span>
|
|
|
|
<span class="title" data-tooltip-content="#{{.Issue.Index}} {{.Issue.Title | RenderEmoji $.Context}}">
|
|
|
|
#{{.Issue.Index}} {{.Issue.Title | RenderEmoji $.Context}}
|
|
|
|
</span>
|
|
|
|
</div>
|
|
|
|
<div class="text small">
|
|
|
|
{{.Repository.OwnerName}}/{{.Repository.Name}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
<div class="item-right gt-df gt-ac">
|
|
|
|
{{if and $.CanCreateIssueDependencies (not $.Repository.IsArchived)}}
|
|
|
|
<a class="delete-dependency-button ci muted" data-id="{{.Issue.ID}}" data-type="blocking" data-tooltip-content="{{$.locale.Tr "repo.issues.dependency.remove_info"}}">
|
|
|
|
{{svg "octicon-trash" 16}}
|
|
|
|
</a>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
{{else if .BlockedByDependenciesNotPermitted}}
|
|
|
|
<div class="item gt-df gt-ac gt-sb">
|
|
|
|
<span>{{$.locale.TrN (len .BlockedByDependenciesNotPermitted) "repo.issues.dependency.no_permission_1" "repo.issues.dependency.no_permission_n" (len .BlockedByDependenciesNotPermitted)}}</span>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
2018-07-18 02:53:58 +05:30
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
|
2019-01-24 00:28:38 +05:30
|
|
|
{{if and .CanCreateIssueDependencies (not .Repository.IsArchived)}}
|
2018-07-18 02:53:58 +05:30
|
|
|
<div>
|
2021-11-16 23:48:25 +05:30
|
|
|
<form method="POST" action="{{.Issue.Link}}/dependency/add" id="addDependencyForm">
|
2018-07-18 02:53:58 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
|
|
|
<div class="ui fluid action input">
|
2019-05-12 16:40:12 +05:30
|
|
|
<div class="ui search selection dropdown" id="new-dependency-drop-list" data-issue-id="{{.Issue.ID}}">
|
2018-07-18 02:53:58 +05:30
|
|
|
<input name="newDependency" type="hidden">
|
2020-11-01 03:45:11 +05:30
|
|
|
{{svg "octicon-triangle-down" 14 "dropdown icon"}}
|
2018-07-18 02:53:58 +05:30
|
|
|
<input type="text" class="search">
|
2022-06-28 02:28:46 +05:30
|
|
|
<div class="default text">{{.locale.Tr "repo.issues.dependency.add"}}</div>
|
2018-07-18 02:53:58 +05:30
|
|
|
</div>
|
|
|
|
<button class="ui green icon button">
|
2021-05-17 01:48:18 +05:30
|
|
|
{{svg "octicon-plus"}}
|
2018-07-18 02:53:58 +05:30
|
|
|
</button>
|
|
|
|
</div>
|
|
|
|
</form>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if and .CanCreateIssueDependencies (not .Repository.IsArchived)}}
|
|
|
|
<input type="hidden" id="crossRepoSearch" value="{{.AllowCrossRepositoryDependencies}}">
|
|
|
|
|
2023-04-24 16:38:59 +05:30
|
|
|
<div class="ui g-modal-confirm modal remove-dependency">
|
2023-04-23 14:54:19 +05:30
|
|
|
<div class="header">
|
2021-03-22 09:34:19 +05:30
|
|
|
{{svg "octicon-trash"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.dependency.remove_header"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
|
|
|
<div class="content">
|
2021-11-16 23:48:25 +05:30
|
|
|
<form method="POST" action="{{.Issue.Link}}/dependency/delete" id="removeDependencyForm">
|
2021-02-28 19:35:55 +05:30
|
|
|
{{$.CsrfTokenHtml}}
|
2023-03-27 21:35:51 +05:30
|
|
|
<input type="hidden" value="" name="removeDependencyID" id="removeDependencyID">
|
|
|
|
<input type="hidden" value="" name="dependencyType" id="dependencyType">
|
2021-02-28 19:35:55 +05:30
|
|
|
</form>
|
|
|
|
<p>{{if .Issue.IsPull}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.dependency.pr_remove_text"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.dependency.issue_remove_text"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{end}}</p>
|
|
|
|
</div>
|
2023-04-23 14:54:19 +05:30
|
|
|
{{$ModalButtonCancelText := .locale.Tr "repo.issues.dependency.cancel"}}
|
|
|
|
{{$ModalButtonOkText := .locale.Tr "repo.issues.dependency.remove"}}
|
|
|
|
{{template "base/modal_actions_confirm" (dict "." . "ModalButtonCancelText" $ModalButtonCancelText "ModalButtonOkText" $ModalButtonOkText)}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
{{end}}
|
|
|
|
|
2022-05-06 00:28:37 +05:30
|
|
|
<div class="ui divider"></div>
|
|
|
|
<div class="ui equal width compact grid">
|
2023-02-15 17:04:10 +05:30
|
|
|
{{$issueReferenceLink := printf "%s#%d" .Issue.Repo.FullName .Issue.Index}}
|
2023-03-24 16:05:38 +05:30
|
|
|
<div class="row gt-ac" data-tooltip-content="{{$issueReferenceLink}}">
|
2022-06-28 02:28:46 +05:30
|
|
|
<span class="text column truncate">{{.locale.Tr "repo.issues.reference_link" $issueReferenceLink}}</span>
|
2023-02-13 23:29:59 +05:30
|
|
|
<button class="ui two wide button column gt-p-3" data-clipboard-text="{{$issueReferenceLink}}">{{svg "octicon-copy" 14}}</button>
|
2022-05-06 00:28:37 +05:30
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
|
2021-02-28 19:35:55 +05:30
|
|
|
{{if and .IsRepoAdmin (not .Repository.IsArchived)}}
|
2019-02-19 02:25:04 +05:30
|
|
|
<div class="ui divider"></div>
|
|
|
|
<div class="ui watching">
|
2022-08-31 21:28:54 +05:30
|
|
|
<button class="fluid ui show-modal button {{if .Issue.IsLocked}} negative {{end}}" data-modal="#lock">
|
2021-01-19 08:32:47 +05:30
|
|
|
{{if .Issue.IsLocked}}
|
|
|
|
{{svg "octicon-key"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.unlock"}}
|
2021-01-19 08:32:47 +05:30
|
|
|
{{else}}
|
|
|
|
{{svg "octicon-lock"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.lock"}}
|
2021-01-19 08:32:47 +05:30
|
|
|
{{end}}
|
|
|
|
</button>
|
2019-02-19 02:25:04 +05:30
|
|
|
</div>
|
|
|
|
<div class="ui tiny modal" id="lock">
|
|
|
|
<div class="header">
|
2022-08-31 21:28:54 +05:30
|
|
|
{{if .Issue.IsLocked}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.unlock.title"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.lock.title"}}
|
2021-02-28 19:35:55 +05:30
|
|
|
{{end}}
|
2019-02-19 02:25:04 +05:30
|
|
|
</div>
|
2021-03-22 00:28:28 +05:30
|
|
|
<div class="content">
|
2023-05-04 00:02:10 +05:30
|
|
|
<div class="ui warning message">
|
2022-08-31 21:28:54 +05:30
|
|
|
{{if .Issue.IsLocked}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.unlock.notice_1"}}<br>
|
|
|
|
{{.locale.Tr "repo.issues.unlock.notice_2"}}<br>
|
2021-03-22 00:28:28 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.lock.notice_1"}}<br>
|
|
|
|
{{.locale.Tr "repo.issues.lock.notice_2"}}<br>
|
|
|
|
{{.locale.Tr "repo.issues.lock.notice_3"}}<br>
|
2021-03-22 00:28:28 +05:30
|
|
|
{{end}}
|
|
|
|
</div>
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2022-08-31 21:28:54 +05:30
|
|
|
<form class="ui form" action="{{.Issue.Link}}{{if .Issue.IsLocked}}/unlock{{else}}/lock{{end}}"
|
2021-03-22 00:28:28 +05:30
|
|
|
method="post">
|
|
|
|
{{.CsrfTokenHtml}}
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2022-08-31 21:28:54 +05:30
|
|
|
{{if not .Issue.IsLocked}}
|
2021-03-22 00:28:28 +05:30
|
|
|
<div class="field">
|
2022-08-31 21:28:54 +05:30
|
|
|
<strong> {{.locale.Tr "repo.issues.lock.reason"}} </strong>
|
2021-03-22 00:28:28 +05:30
|
|
|
</div>
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2021-03-22 00:28:28 +05:30
|
|
|
<div class="field">
|
|
|
|
<div class="ui fluid dropdown selection" tabindex="0">
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2021-03-22 00:28:28 +05:30
|
|
|
<select name="reason">
|
|
|
|
<option value=""> </option>
|
|
|
|
{{range .LockReasons}}
|
|
|
|
<option value="{{.}}">{{.}}</option>
|
|
|
|
{{end}}
|
|
|
|
</select>
|
|
|
|
{{svg "octicon-triangle-down" 14 "dropdown icon"}}
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2021-03-22 00:28:28 +05:30
|
|
|
<div class="default text"> </div>
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2023-02-14 17:23:54 +05:30
|
|
|
<div class="menu">
|
2021-03-22 00:28:28 +05:30
|
|
|
{{range .LockReasons}}
|
|
|
|
<div class="item" data-value="{{.}}">{{.}}</div>
|
|
|
|
{{end}}
|
|
|
|
</div>
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
2019-02-19 02:25:04 +05:30
|
|
|
</div>
|
2021-03-22 00:28:28 +05:30
|
|
|
{{end}}
|
2019-02-19 02:25:04 +05:30
|
|
|
|
2021-03-22 00:28:28 +05:30
|
|
|
<div class="text right actions">
|
2023-03-14 09:04:09 +05:30
|
|
|
<button class="ui cancel button">{{.locale.Tr "settings.cancel"}}</button>
|
2021-03-22 00:28:28 +05:30
|
|
|
<button class="ui red button">
|
2022-08-31 21:28:54 +05:30
|
|
|
{{if .Issue.IsLocked}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.unlock_confirm"}}
|
2021-03-22 00:28:28 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.lock_confirm"}}
|
2021-03-22 00:28:28 +05:30
|
|
|
{{end}}
|
|
|
|
</button>
|
|
|
|
</div>
|
|
|
|
</form>
|
|
|
|
</div>
|
2018-07-18 02:53:58 +05:30
|
|
|
</div>
|
2023-02-13 23:29:59 +05:30
|
|
|
<button class="fluid ui show-modal button negative gt-mt-3" data-modal="#delete">
|
2022-03-09 06:08:11 +05:30
|
|
|
{{svg "octicon-trash"}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.delete"}}
|
2022-03-09 06:08:11 +05:30
|
|
|
</button>
|
2023-04-24 16:38:59 +05:30
|
|
|
<div class="ui g-modal-confirm modal" id="delete">
|
2023-04-23 14:54:19 +05:30
|
|
|
<div class="header">
|
2022-06-19 15:35:15 +05:30
|
|
|
{{if .Issue.IsPull}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.pulls.delete.title"}}
|
2022-06-19 15:35:15 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.delete.title"}}
|
2022-06-19 15:35:15 +05:30
|
|
|
{{end}}
|
2022-03-09 06:08:11 +05:30
|
|
|
</div>
|
2023-04-23 14:54:19 +05:30
|
|
|
<div class="content">
|
2022-03-09 06:08:11 +05:30
|
|
|
<p>
|
2022-06-19 15:35:15 +05:30
|
|
|
{{if .Issue.IsPull}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.pulls.delete.text"}}
|
2022-06-19 15:35:15 +05:30
|
|
|
{{else}}
|
2022-06-28 02:28:46 +05:30
|
|
|
{{.locale.Tr "repo.issues.delete.text"}}
|
2022-06-19 15:35:15 +05:30
|
|
|
{{end}}
|
2022-03-09 06:08:11 +05:30
|
|
|
</p>
|
|
|
|
</div>
|
|
|
|
<form action="{{.Issue.Link}}/delete" method="post">
|
|
|
|
{{.CsrfTokenHtml}}
|
2023-04-23 14:54:19 +05:30
|
|
|
{{template "base/modal_actions_confirm" .}}
|
2022-03-09 06:08:11 +05:30
|
|
|
</form>
|
|
|
|
</div>
|
2018-07-18 02:53:58 +05:30
|
|
|
{{end}}
|
2022-04-28 21:15:33 +05:30
|
|
|
|
2023-04-19 23:51:48 +05:30
|
|
|
{{if and .Issue.IsPull .IsIssuePoster (not .Issue.IsClosed) .Issue.PullRequest.HeadRepo}}
|
2022-04-28 21:15:33 +05:30
|
|
|
{{if and (not (eq .Issue.PullRequest.HeadRepo.FullName .Issue.PullRequest.BaseRepo.FullName)) .CanWriteToHeadRepo}}
|
|
|
|
<div class="ui divider"></div>
|
|
|
|
<div class="inline field">
|
2023-03-24 16:05:38 +05:30
|
|
|
<div class="ui checkbox" id="allow-edits-from-maintainers"
|
2022-04-28 21:15:33 +05:30
|
|
|
data-url="{{.Issue.Link}}"
|
Use a general approach to show tooltip, fix temporary tooltip bug (#23574)
## TLDR
* Improve performance: lazy creating the tippy instances.
* Transparently support all "tooltip" elements, no need to call
`initTooltip` again and again.
* Fix a temporary tooltip re-entrance bug, which causes showing temp
content forever.
* Upgrade vue3-calendar-heatmap to 2.0.2 with lazy tippy init
(initHeatmap time decreases from 100ms to 50ms)
## Details
### The performance
Creating a lot of tippy tooltip instances is expensive. This PR doesn't
create all tippy tooltip instances, instead, it only adds "mouseover"
event listener to necessary elements, and then switches to the tippy
tooltip
### The general approach for all tooltips
Before, dynamically generated tooltips need to be called with
`initTooltip`.
After, use MutationObserver to:
* Attach the event listeners to newly created tooltip elements, work for
Vue (easier than before)
* Catch changed attributes and update the tooltip content (better than
before)
It does help a lot, eg:
https://github.com/go-gitea/gitea/blob/1a4efa0ee9a49d48549be7479a46be133b9bc260/web_src/js/components/PullRequestMergeForm.vue#L33-L36
### Temporary tooltip re-entrance bug
To reproduce, on try.gitea.io, click the "copy clone url" quickly, then
the tooltip will be "Copied!" forever.
After this PR, with the help of `attachTippyTooltip`, the tooltip
content could be reset to the default correctly.
### Other changes
* `data-tooltip-content` is preferred from now on, the old
`data-content` may cause conflicts with other modules.
* `data-placement` was only used for tooltip, so it's renamed to
`data-tooltip-placement`, and removed from `createTippy`.
2023-03-23 15:26:15 +05:30
|
|
|
data-tooltip-content="{{.locale.Tr "repo.pulls.allow_edits_from_maintainers_desc"}}"
|
2022-06-28 02:28:46 +05:30
|
|
|
data-prompt-error="{{.locale.Tr "repo.pulls.allow_edits_from_maintainers_err"}}"
|
2022-04-28 21:15:33 +05:30
|
|
|
>
|
2022-06-28 02:28:46 +05:30
|
|
|
<label><strong>{{.locale.Tr "repo.pulls.allow_edits_from_maintainers"}}</strong></label>
|
2022-04-28 21:15:33 +05:30
|
|
|
<input type="checkbox" {{if .Issue.PullRequest.AllowMaintainerEdit}}checked{{end}}>
|
|
|
|
</div>
|
|
|
|
</div>
|
|
|
|
{{end}}
|
|
|
|
{{end}}
|
2021-02-28 19:35:55 +05:30
|
|
|
</div>
|
|
|
|
</div>
|