diff --git a/internal/cmd/auth/login.go b/internal/cmd/auth/login.go index b228775..f8eb16d 100644 --- a/internal/cmd/auth/login.go +++ b/internal/cmd/auth/login.go @@ -24,7 +24,7 @@ func loginApiKey(r *config.Red_t, cmd *cobra.Command, name, server, apikey strin print.Debug(r, "%d %s", status, string(res)) if err = api.StatusCode(status); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return false } @@ -37,7 +37,7 @@ func loginApiKey(r *config.Red_t, cmd *cobra.Command, name, server, apikey strin r.AddServer(name, server, user.User.ApiKey, "", 0, user.User.ID) if err = r.Save(); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return false } @@ -67,7 +67,7 @@ func loginPassword(r *config.Red_t, cmd *cobra.Command, name, server, username s } if password, err = r.Term.PromptPassword("Password", ""); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) continue } @@ -81,7 +81,7 @@ func loginPassword(r *config.Red_t, cmd *cobra.Command, name, server, username s if err = api.StatusCode(status); err != nil { print.Info("If Two-factor authentication is enabled, this login method will not work as it is not supported, please use Apikey instead, you will find the Apikey (Api access key) at /my/account\n") - print.Error(err.Error()) + print.Error("%s", err.Error()) continue } @@ -96,7 +96,7 @@ func loginPassword(r *config.Red_t, cmd *cobra.Command, name, server, username s r.AddServer(name, server, user.User.ApiKey, "", 0, user.User.ID) if err = r.Save(); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return } @@ -110,7 +110,7 @@ func displayLogin(r *config.Red_t, cmd *cobra.Command) { var apikey string var err error - print.Info(text.FgGreen.Sprint("Welcome to Red an Redmine CLI\n") + + print.Info("%s", text.FgGreen.Sprint("Welcome to Red an Redmine CLI\n") + "Before login make sure you have enabled `Enable REST web service`\nfind it in Administration -> Settings -> API or use url /settings?tab=api\nYou find ApiKey (API access key) from /my/account\n\n") if alias, err = r.Term.PromptStringRequire("Redmine server alias (Used for multi redmine server support)", ""); err != nil { diff --git a/internal/cmd/issue/create.go b/internal/cmd/issue/create.go index aca97e8..f5ae269 100644 --- a/internal/cmd/issue/create.go +++ b/internal/cmd/issue/create.go @@ -82,7 +82,7 @@ func displayCreateIssue(r *config.Red_t, cmd *cobra.Command, path string) { issue.Issue.ProjectID = projectID if idNames, err = project.GetTrackers(r, projectID); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return } @@ -111,7 +111,7 @@ func displayCreateIssue(r *config.Red_t, cmd *cobra.Command, path string) { } case FIELD_PRIORITY: if idNames, err = global.GetPriorities(r); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } id, _ := r.Term.Choose("Priority", idNames) @@ -121,7 +121,7 @@ func displayCreateIssue(r *config.Red_t, cmd *cobra.Command, path string) { } case FIELD_TARGET_VERSION: if idNames, err = project.GetVersions(r, projectID); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } id, _ := r.Term.Choose("Version", idNames) @@ -136,7 +136,7 @@ func displayCreateIssue(r *config.Red_t, cmd *cobra.Command, path string) { } case FIELD_ASSIGN: if idNames, err = project.GetAssigns(r, projectID); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } id, _ := r.Term.Choose("Assign", idNames) diff --git a/internal/cmd/issue/edit.go b/internal/cmd/issue/edit.go index 52626b4..54f6d0b 100644 --- a/internal/cmd/issue/edit.go +++ b/internal/cmd/issue/edit.go @@ -238,7 +238,7 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { print.Debug(r, "%d %s", status, string(body)) if err := api.StatusCode(status); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return } @@ -274,7 +274,7 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { subject, err := r.Term.PromptString("Subject", viewIssue.Issue.Subject) if err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } else { issue.Issue.Subject = subject viewIssue.Issue.Subject = subject @@ -283,7 +283,7 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { idName, err := cmdIssueEditIssueStatus(r, viewIssue.Issue.AllowedStatuses) if err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } else { issue.Issue.StatusID = idName.ID viewIssue.Issue.Status = idName @@ -292,7 +292,7 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { idName, err := cmdIssueEditIssuePriority(r) if err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } else { issue.Issue.PriorityID = idName.ID viewIssue.Issue.Priority = idName @@ -301,7 +301,7 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { idName, err := cmdIssueEditIssueTracker(r, viewIssue.Issue.Project.ID) if err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } else { issue.Issue.TrackerID = idName.ID viewIssue.Issue.Tracker = idName @@ -311,13 +311,13 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { viewIssue.Issue.Description = issue.Issue.Description case FIELD_NOTE: if err = cmdIssueEditIssueNote(r, &issue); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } case FIELD_ASSIGN: idName, err := cmdIssueEditIssueAssign(r, viewIssue.Issue.Project.ID) if err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } else { issue.Issue.AssignedToID = idName.ID viewIssue.Issue.AssignedTo = idName @@ -326,7 +326,7 @@ func cmdIssueEditIssue(r *config.Red_t, cmd *cobra.Command, id, path string) { idName, err := cmdIssueEditIssueTargetVersion(r, viewIssue.Issue.Project.ID) if err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } else { issue.Issue.FixedVersionID = idName.ID viewIssue.Issue.FixedVersion = idName diff --git a/internal/cmd/issue/note.go b/internal/cmd/issue/note.go index bf908b4..6307bce 100644 --- a/internal/cmd/issue/note.go +++ b/internal/cmd/issue/note.go @@ -59,7 +59,7 @@ func cmdIssueNoteRun(r *config.Red_t, cmd *cobra.Command, args []string) { print.Debug(r, "%d %s", status, string(res)) if err := api.StatusCode(status); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return } diff --git a/internal/editor/editor.go b/internal/editor/editor.go index 8f43579..920437b 100644 --- a/internal/editor/editor.go +++ b/internal/editor/editor.go @@ -51,19 +51,19 @@ func StartPage(exec, body string) { } if path, err = config.CreateTmpFile(body); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return } args = append(args, path) if err = createFile(view, args, body); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return } if err = os.Remove(path); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } } @@ -76,12 +76,12 @@ func createFile(editor string, arg []string, body string) error { cmd.Stderr = os.Stderr if err = cmd.Start(); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return err } if err = cmd.Wait(); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return err } return nil @@ -93,22 +93,22 @@ func editor(editor, body string) string { var err error if path, err = config.CreateTmpFile(body); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return "" } if err = createFile(editor, []string{path}, body); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return "" } if data, err = os.ReadFile(path); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) return "" } if err = os.Remove(path); err != nil { - print.Error(err.Error()) + print.Error("%s", err.Error()) } return string(data) } diff --git a/internal/print/print.go b/internal/print/print.go index 5857568..09a0c77 100644 --- a/internal/print/print.go +++ b/internal/print/print.go @@ -12,10 +12,10 @@ func Debug(r *config.Red_t, format string, a ...any) { return } if len(a) > 0 { - fmt.Println(text.FgHiBlack.Sprintf("DEBUG: "+format, a)) + fmt.Println(text.FgHiBlack.Sprintf("DEBUG: "+format, a...)) return } - fmt.Println(text.FgHiBlack.Sprintf("DEBUG: " + format)) + fmt.Println(text.FgHiBlack.Sprint("DEBUG: " + format)) } func PrintRowHead(r *config.Red_t, format string, a ...any) {