chore: fix check setting value

pull/3083/head
steven 1 year ago
parent 77bafba682
commit ed1954c58c

@ -145,21 +145,21 @@ func (t *TelegramHandler) CallbackQueryHandle(ctx context.Context, bot *telegram
return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Memo %d not found, possibly deleted elsewhere", memoID)) return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Memo %d not found, possibly deleted elsewhere", memoID))
} }
var disablePublicMemo bool
setting, err := t.store.GetWorkspaceSetting(ctx, &store.FindWorkspaceSetting{ setting, err := t.store.GetWorkspaceSetting(ctx, &store.FindWorkspaceSetting{
Name: apiv1.SystemSettingDisablePublicMemosName.String(), Name: apiv1.SystemSettingDisablePublicMemosName.String(),
}) })
if err != nil { if err != nil {
return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Failed to get workspace setting %s", err)) return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Failed to get workspace setting %s", err))
} }
if setting != nil && setting.Value != "" {
err = json.Unmarshal([]byte(setting.Value), &disablePublicMemo) disablePublicMemo := false
if err != nil { err = json.Unmarshal([]byte(setting.Value), &disablePublicMemo)
return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Failed to get workspace setting %s", err)) if err != nil {
} return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Failed to get workspace setting %s", err))
}
if disablePublicMemo && visibility == store.Public { if disablePublicMemo && visibility == store.Public {
return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Failed to changing Memo %d to %s\n(workspace disallowed public memo)", memoID, visibility)) return bot.AnswerCallbackQuery(ctx, callbackQuery.ID, fmt.Sprintf("Failed to changing Memo %d to %s\n(workspace disallowed public memo)", memoID, visibility))
}
} }
update := store.UpdateMemo{ update := store.UpdateMemo{

Loading…
Cancel
Save