From 35c148d2551bbb65a6d469b44876ba78578397c1 Mon Sep 17 00:00:00 2001 From: Carlos Tadeu Panato Junior Date: Mon, 17 Sep 2018 09:29:04 +0200 Subject: fixing log messages (#9341) * fixing log messages * update per review * Update notification.go * Update server.go * Update notification.go --- app/import_functions.go | 4 ++-- app/notification.go | 4 ++-- app/notification_email.go | 2 +- cmd/mattermost/commands/server.go | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/app/import_functions.go b/app/import_functions.go index dce84feb5..ff22f23bb 100644 --- a/app/import_functions.go +++ b/app/import_functions.go @@ -491,10 +491,10 @@ func (a *App) ImportUser(data *UserImportData, dryRun bool) *model.AppError { if data.ProfileImage != nil { file, err := os.Open(*data.ProfileImage) if err != nil { - mlog.Error(fmt.Sprint("api.import.import_user.profile_image.error FIXME: NOT FOUND IN TRANSLATIONS FILE", err)) + mlog.Error("Unable to open the profile image.", mlog.Any("err", err)) } if err := a.SetProfileImageFromFile(savedUser.Id, file); err != nil { - mlog.Error(fmt.Sprint("api.import.import_user.profile_image.error FIXME: NOT FOUND IN TRANSLATIONS FILE", err)) + mlog.Error("Unable to set the profile image from a file.", mlog.Any("err", err)) } } diff --git a/app/notification.go b/app/notification.go index 910748cd3..92b69039b 100644 --- a/app/notification.go +++ b/app/notification.go @@ -284,7 +284,7 @@ func (a *App) SendNotifications(post *model.Post, team *model.Team, channel *mod if *a.Config().EmailSettings.SendPushNotifications { pushServer := *a.Config().EmailSettings.PushNotificationServer if license := a.License(); pushServer == model.MHPNS && (license == nil || !*license.Features.MHPNS) { - mlog.Warn("api.post.send_notifications_and_forget.push_notification.mhpnsWarn FIXME: NOT FOUND IN TRANSLATIONS FILE") + mlog.Warn("Push notifications are disabled. Go to System Console > Notifications > Mobile Push to enable them.") sendPushNotifications = false } else { sendPushNotifications = true @@ -365,7 +365,7 @@ func (a *App) SendNotifications(post *model.Post, team *model.Team, channel *mod var infos []*model.FileInfo if result := <-fchan; result.Err != nil { - mlog.Warn(fmt.Sprint("api.post.send_notifications.files.error FIXME: NOT FOUND IN TRANSLATIONS FILE", post.Id, result.Err), mlog.String("post_id", post.Id)) + mlog.Warn(fmt.Sprint("Unable to get fileInfo for push notifications.", post.Id, result.Err), mlog.String("post_id", post.Id)) } else { infos = result.Data.([]*model.FileInfo) } diff --git a/app/notification_email.go b/app/notification_email.go index f5b55f9a8..b3835a4ee 100644 --- a/app/notification_email.go +++ b/app/notification_email.go @@ -89,7 +89,7 @@ func (a *App) sendNotificationEmail(post *model.Post, user *model.User, channel a.Go(func() { if err := a.SendMail(user.Email, html.UnescapeString(subjectText), bodyText); err != nil { - mlog.Error(fmt.Sprint("api.post.send_notifications_and_forget.send.error FIXME: NOT FOUND IN TRANSLATIONS FILE", user.Email, err)) + mlog.Error(fmt.Sprint("Error to send the email", user.Email, err)) } }) diff --git a/cmd/mattermost/commands/server.go b/cmd/mattermost/commands/server.go index 0b7ef009c..95dc2f1c7 100644 --- a/cmd/mattermost/commands/server.go +++ b/cmd/mattermost/commands/server.go @@ -243,7 +243,7 @@ func runSessionCleanupJob(a *app.App) { func resetStatuses(a *app.App) { if result := <-a.Srv.Store.Status().ResetAll(); result.Err != nil { - mlog.Error(fmt.Sprint("mattermost.reset_status.error FIXME: NOT FOUND IN TRANSLATIONS FILE", result.Err.Error())) + mlog.Error(fmt.Sprint("Error to reset the server status.", result.Err.Error())) } } -- cgit v1.2.3-1-g7c22