Fix notification: --all dont relay on a repo (#159)
fix nil pointer exeption make notifications work outside a repo Co-authored-by: 6543 <6543@obermui.de> Reviewed-on: https://gitea.com/gitea/tea/pulls/159 Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com> Reviewed-by: mrsdizzie <info@mrsdizzie.com>
This commit is contained in:
parent
a979df1070
commit
edd180a8f5
|
@ -197,6 +197,9 @@ func curGitRepoPath(path string) (*Login, string, error) {
|
||||||
} else {
|
} else {
|
||||||
repo, err = git.RepoFromPath(path)
|
repo, err = git.RepoFromPath(path)
|
||||||
}
|
}
|
||||||
|
if err != nil {
|
||||||
|
return nil, "", err
|
||||||
|
}
|
||||||
gitConfig, err := repo.Config()
|
gitConfig, err := repo.Config()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, "", err
|
return nil, "", err
|
||||||
|
|
|
@ -46,9 +46,6 @@ var CmdNotifications = cli.Command{
|
||||||
}
|
}
|
||||||
|
|
||||||
func runNotifications(ctx *cli.Context) error {
|
func runNotifications(ctx *cli.Context) error {
|
||||||
login, owner, repo := initCommand()
|
|
||||||
|
|
||||||
client := login.Client()
|
|
||||||
var news []*gitea.NotificationThread
|
var news []*gitea.NotificationThread
|
||||||
var err error
|
var err error
|
||||||
|
|
||||||
|
@ -58,11 +55,13 @@ func runNotifications(ctx *cli.Context) error {
|
||||||
}
|
}
|
||||||
|
|
||||||
if ctx.Bool("all") {
|
if ctx.Bool("all") {
|
||||||
news, err = client.ListNotifications(gitea.ListNotificationOptions{
|
login := initCommandLoginOnly()
|
||||||
|
news, err = login.Client().ListNotifications(gitea.ListNotificationOptions{
|
||||||
ListOptions: listOpts,
|
ListOptions: listOpts,
|
||||||
})
|
})
|
||||||
} else {
|
} else {
|
||||||
news, err = client.ListRepoNotifications(owner, repo, gitea.ListNotificationOptions{
|
login, owner, repo := initCommand()
|
||||||
|
news, err = login.Client().ListRepoNotifications(owner, repo, gitea.ListNotificationOptions{
|
||||||
ListOptions: listOpts,
|
ListOptions: listOpts,
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue