Fix webhook bug (#10427) (#10432)

lunny · techknowlogick · web-flow · commit b51d7c459e65 · 2020-02-24T01:48:30.000-03:00
Co-authored-by: techknowlogick &lt;matti@mdranta.net&gt;
diff --git a/modules/notification/webhook/webhook.go b/modules/notification/webhook/webhook.go
@@ -689,12 +689,12 @@ func (m *webhookNotifier) NotifyDeleteRef(pusher *models.User, repo *models.Repo
 
 	if err := webhook_module.PrepareWebhooks(repo, models.HookEventDelete, &api.DeletePayload{
 		Ref:        refName,
-		RefType:    "branch",
+		RefType:    refType,
 		PusherType: api.PusherTypeUser,
 		Repo:       apiRepo,
 		Sender:     apiPusher,
 	}); err != nil {
-		log.Error("PrepareWebhooks.(delete branch): %v", err)
+		log.Error("PrepareWebhooks.(delete %s): %v", refType, err)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -689,12 +689,12 @@ func (m webhookNotifier) NotifyDeleteRef(pusher models.User, repo *models.Repo`
`689`	`689`
`690`	`690`	`if err := webhook_module.PrepareWebhooks(repo, models.HookEventDelete, &api.DeletePayload{`
`691`	`691`	`Ref: refName,`
`692`		`- RefType: "branch",`
	`692`	`+ RefType: refType,`
`693`	`693`	`PusherType: api.PusherTypeUser,`
`694`	`694`	`Repo: apiRepo,`
`695`	`695`	`Sender: apiPusher,`
`696`	`696`	`}); err != nil {`
`697`		`- log.Error("PrepareWebhooks.(delete branch): %v", err)`
	`697`	`+ log.Error("PrepareWebhooks.(delete %s): %v", refType, err)`
`698`	`698`	`}`
`699`	`699`	`}`
`700`	`700`