MM 15187: Migrate "WebHook.DeleteOutgoing" to Sync by default (#10708)

* cherry commit for DeleteOutgoing and generate store mocks

* go fmt the code
Этот коммит содержится в:
Puneeth Reddy
2019-04-25 22:44:00 -07:00
коммит произвёл Jesús Espino
родитель 9fc05b5865
Коммит caf0c0d375
6 изменённых файлов: 16 добавлений и 20 удалений

Просмотреть файл

@@ -833,7 +833,7 @@ func (a *App) DeleteChannel(channel *model.Channel, userId string) *model.AppErr
}
for _, hook := range outgoingHooks {
if result := <-a.Srv.Store.Webhook().DeleteOutgoing(hook.Id, now); result.Err != nil {
if err := a.Srv.Store.Webhook().DeleteOutgoing(hook.Id, now); err != nil {
mlog.Error(fmt.Sprintf("Encountered error deleting outgoing webhook, id=%v", hook.Id))
}
}

Просмотреть файл

@@ -536,11 +536,7 @@ func (a *App) DeleteOutgoingWebhook(hookId string) *model.AppError {
return model.NewAppError("DeleteOutgoingWebhook", "api.outgoing_webhook.disabled.app_error", nil, "", http.StatusNotImplemented)
}
if result := <-a.Srv.Store.Webhook().DeleteOutgoing(hookId, model.GetMillis()); result.Err != nil {
return result.Err
}
return nil
return a.Srv.Store.Webhook().DeleteOutgoing(hookId, model.GetMillis())
}
func (a *App) RegenOutgoingWebhookToken(hook *model.OutgoingWebhook) (*model.OutgoingWebhook, *model.AppError) {