Add support for silent mode.
Fixes #13 partially. Only missing STDIN reading as a special file.
This commit is contained in:
parent
edc5120921
commit
7f8947b970
9 changed files with 188 additions and 125 deletions
|
|
@ -4,19 +4,20 @@ import (
|
|||
"bytes"
|
||||
"errors"
|
||||
"flag"
|
||||
"fmt"
|
||||
"io"
|
||||
"os"
|
||||
"fmt"
|
||||
"text/template"
|
||||
|
||||
"otremblay.com/jkl"
|
||||
)
|
||||
|
||||
type CreateCmd struct {
|
||||
args []string
|
||||
project string
|
||||
file string
|
||||
args []string
|
||||
project string
|
||||
file string
|
||||
issuetype string
|
||||
silent bool
|
||||
}
|
||||
|
||||
func NewCreateCmd(args []string) (*CreateCmd, error) {
|
||||
|
|
|
|||
|
|
@ -3,6 +3,7 @@ package main
|
|||
import (
|
||||
"bytes"
|
||||
"flag"
|
||||
"fmt"
|
||||
"os"
|
||||
|
||||
"text/template"
|
||||
|
|
@ -21,9 +22,9 @@ func NewEditCmd(args []string) (*EditCmd, error) {
|
|||
ccmd := &EditCmd{project: os.Getenv("JIRA_PROJECT")}
|
||||
f := flag.NewFlagSet("x", flag.ExitOnError)
|
||||
f.StringVar(&ccmd.project, "p", "", "Jira project key")
|
||||
f.StringVar(&ccmd.file, "f", "filename", "File to get issue description from")
|
||||
f.StringVar(&ccmd.file, "f", "", "File to get issue description from")
|
||||
f.Parse(args)
|
||||
ccmd.taskKey = flag.Arg(0)
|
||||
ccmd.taskKey = f.Arg(0)
|
||||
return ccmd, nil
|
||||
}
|
||||
|
||||
|
|
@ -31,7 +32,7 @@ func (ecmd *EditCmd) Edit() error {
|
|||
b := bytes.NewBuffer(nil)
|
||||
iss, err := jkl.GetIssue(ecmd.taskKey)
|
||||
if err != nil {
|
||||
return err
|
||||
return fmt.Errorf("Edit failed: %v", err)
|
||||
}
|
||||
err = editTmpl.Execute(b, iss)
|
||||
if err != nil {
|
||||
|
|
@ -42,12 +43,12 @@ func (ecmd *EditCmd) Edit() error {
|
|||
iss, err = GetIssueFromFile(ecmd.file, b, iss.EditMeta)
|
||||
|
||||
if err != nil {
|
||||
return err
|
||||
return fmt.Errorf("Error getting issue from file: %v", err)
|
||||
}
|
||||
} else {
|
||||
iss, err = GetIssueFromTmpFile(b, iss.EditMeta)
|
||||
if err != nil {
|
||||
return err
|
||||
return fmt.Errorf("Error getting issue from temp file: %v", err)
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
|||
|
|
@ -10,6 +10,7 @@ import (
|
|||
"os/exec"
|
||||
"regexp"
|
||||
"strings"
|
||||
"unicode"
|
||||
|
||||
"reflect"
|
||||
|
||||
|
|
@ -17,6 +18,7 @@ import (
|
|||
|
||||
"otremblay.com/jkl"
|
||||
)
|
||||
|
||||
// def get_editor do
|
||||
// [System.get_env("EDITOR"), "nano", "vim", "vi"]
|
||||
// |> Enum.find(nil, fn (ed) -> System.find_executable(ed) != nil end)
|
||||
|
|
@ -42,12 +44,12 @@ func copyInitial(dst io.WriteSeeker, initial io.Reader) {
|
|||
func GetIssueFromTmpFile(initial io.Reader, editMeta *jkl.EditMeta) (*jkl.JiraIssue, error) {
|
||||
f, err := ioutil.TempFile(os.TempDir(), "jkl")
|
||||
if err != nil {
|
||||
return nil, err
|
||||
return nil, fmt.Errorf("Error opening tempfile: %v", err)
|
||||
}
|
||||
copyInitial(f, initial)
|
||||
f2, err := GetTextFromFile(f)
|
||||
if err != nil {
|
||||
return nil, err
|
||||
return nil, fmt.Errorf("Error reading tempfile: %v", err)
|
||||
}
|
||||
return IssueFromReader(f2, editMeta), nil
|
||||
}
|
||||
|
|
@ -73,16 +75,19 @@ func GetTextFromSpecifiedFile(filename string, initial io.Reader) (io.Reader, er
|
|||
}
|
||||
|
||||
func GetTextFromFile(file *os.File) (io.Reader, error) {
|
||||
cmd := exec.Command(GetEditor(), file.Name())
|
||||
cmd.Stdin = os.Stdin
|
||||
cmd.Stdout = os.Stdout
|
||||
cmd.Stderr = os.Stderr
|
||||
err := cmd.Run()
|
||||
if err != nil {
|
||||
fmt.Println(err)
|
||||
return nil, err
|
||||
var err error
|
||||
if !*SilentMode {
|
||||
cmd := exec.Command(GetEditor(), file.Name())
|
||||
cmd.Stdin = os.Stdin
|
||||
cmd.Stdout = os.Stdout
|
||||
cmd.Stderr = os.Stderr
|
||||
err := cmd.Run()
|
||||
if err != nil {
|
||||
fmt.Println(err)
|
||||
return nil, err
|
||||
}
|
||||
_, err = file.Seek(0, 0)
|
||||
}
|
||||
_, err = file.Seek(0, 0)
|
||||
return file, err
|
||||
}
|
||||
|
||||
|
|
@ -108,6 +113,7 @@ func IssueFromReader(f io.Reader, editMeta *jkl.EditMeta) *jkl.JiraIssue {
|
|||
riss := reflect.ValueOf(iss).Elem()
|
||||
fieldsField := riss.FieldByName("Fields").Elem()
|
||||
currentField := reflect.Value{}
|
||||
currFieldName := ""
|
||||
brd := bufio.NewReader(f)
|
||||
for {
|
||||
b, _, err := brd.ReadLine()
|
||||
|
|
@ -131,6 +137,7 @@ func IssueFromReader(f io.Reader, editMeta *jkl.EditMeta) *jkl.JiraIssue {
|
|||
if len(parts) > 0 {
|
||||
iss.Fields.IssueType = &jkl.IssueType{}
|
||||
currentField = reflect.Value{}
|
||||
currFieldName = potentialField
|
||||
f2 := newfield.Elem()
|
||||
f3 := f2.FieldByName("Name")
|
||||
f3.SetString(strings.TrimSpace(strings.Join(parts, ":")))
|
||||
|
|
@ -139,6 +146,7 @@ func IssueFromReader(f io.Reader, editMeta *jkl.EditMeta) *jkl.JiraIssue {
|
|||
if len(parts) > 0 {
|
||||
iss.Fields.Project = &jkl.Project{}
|
||||
currentField = reflect.Value{}
|
||||
currFieldName = potentialField
|
||||
f2 := newfield.Elem()
|
||||
f3 := f2.FieldByName("Key")
|
||||
f3.SetString(strings.TrimSpace(strings.Join(parts, ":")))
|
||||
|
|
@ -147,19 +155,27 @@ func IssueFromReader(f io.Reader, editMeta *jkl.EditMeta) *jkl.JiraIssue {
|
|||
if len(parts) > 0 {
|
||||
iss.Fields.Parent = &jkl.JiraIssue{}
|
||||
currentField = reflect.Value{}
|
||||
currFieldName = potentialField
|
||||
f2 := newfield.Elem()
|
||||
f3 := f2.FieldByName("Key")
|
||||
f3.SetString(strings.TrimSpace(strings.Join(parts, ":")))
|
||||
}
|
||||
} else {
|
||||
currFieldName = potentialField
|
||||
currentField = newfield
|
||||
}
|
||||
} else if editMeta != nil {
|
||||
} else if editMeta != nil {
|
||||
// If it's not valid, throw it at the createmeta. It will probably end up in ExtraFields.
|
||||
|
||||
|
||||
}
|
||||
if currentField.IsValid() {
|
||||
currentField.SetString(strings.TrimSpace(currentField.String() + "\n" + strings.Join(parts, ":")))
|
||||
newString := currentField.String() + "\n" + strings.Join(parts, ":")
|
||||
if currFieldName != "Description" {
|
||||
newString = strings.TrimSpace(newString)
|
||||
} else if currentField.String() == "" {
|
||||
newString = strings.TrimLeftFunc(newString, unicode.IsSpace)
|
||||
}
|
||||
currentField.SetString(newString)
|
||||
}
|
||||
}
|
||||
return iss
|
||||
|
|
|
|||
|
|
@ -12,7 +12,8 @@ hell
|
|||
Issue Type: Sometype
|
||||
this is ignored
|
||||
Summary: Dookienator
|
||||
also ignored`, "\n"))
|
||||
also ignored`, "\n"), nil)
|
||||
|
||||
AssertEqual(t, `Cowboys
|
||||
from
|
||||
hell`, iss.Fields.Description)
|
||||
|
|
@ -34,3 +35,13 @@ Actual: %v`, expected, actual)
|
|||
func Assert(t *testing.T, fn func() bool) {
|
||||
|
||||
}
|
||||
|
||||
func TestIssueFromFile(t *testing.T) {
|
||||
*SilentMode = true
|
||||
issue, err := GetIssueFromFile("editor_test_file.issue", nil, nil)
|
||||
if err != nil {
|
||||
t.Errorf("Apparently we effed up some! --> %v", err)
|
||||
}
|
||||
AssertEqual(t, "Dookie", issue.Fields.Summary)
|
||||
AssertEqual(t, "McDoogal", issue.Fields.Description)
|
||||
}
|
||||
|
|
|
|||
2
cmd/jkl/editor_test_file.issue
Normal file
2
cmd/jkl/editor_test_file.issue
Normal file
|
|
@ -0,0 +1,2 @@
|
|||
Summary: Dookie
|
||||
Description: McDoogal
|
||||
|
|
@ -14,6 +14,7 @@ import (
|
|||
|
||||
var verbose = flag.Bool("v", false, "Output debug information about jkl")
|
||||
var help = flag.Bool("h", false, "Outputs usage information message")
|
||||
var SilentMode = flag.Bool("s", false, "Silent execution.")
|
||||
|
||||
func main() {
|
||||
jkl.FindRCFile()
|
||||
|
|
|
|||
|
|
@ -39,6 +39,9 @@ func (l *listissue) Color() string {
|
|||
}
|
||||
return ""
|
||||
}
|
||||
func (l *listissue) EFByName(name string) string {
|
||||
return (*jkl.JiraIssue)(l).EFByName(name)
|
||||
}
|
||||
|
||||
type ListCmd struct {
|
||||
args []string
|
||||
|
|
@ -76,7 +79,7 @@ func (l *ListCmd) List() error {
|
|||
if issues, err := jkl.List(strings.Join(l.args, " ")); err != nil {
|
||||
return err
|
||||
} else {
|
||||
for _, issue := range issues {
|
||||
for issue := range issues {
|
||||
var li listissue
|
||||
li = listissue(*issue)
|
||||
err := l.tmpl.Execute(os.Stdout, &li)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue