From 5eaf9e83e7cd56b8bb4915193d13adfb60575b21 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Michael=20Mur=C3=A9?= <batolettre@gmail.com>
Date: Mon, 17 Sep 2018 14:36:51 +0200
Subject: [PATCH] commands: add a "label rm" command to remove labels from a
 bug

---
 commands/label add.go        |  7 ++---
 commands/label rm.go         | 51 ++++++++++++++++++++++++++++++++++++
 doc/man/git-bug-label-rm.1   | 29 ++++++++++++++++++++
 doc/man/git-bug-label.1      |  2 +-
 doc/md/git-bug_label.md      |  1 +
 doc/md/git-bug_label_rm.md   | 22 ++++++++++++++++
 misc/bash_completion/git-bug | 21 +++++++++++++++
 misc/zsh_completion/git-bug  |  2 +-
 8 files changed, 130 insertions(+), 5 deletions(-)
 create mode 100644 commands/label rm.go
 create mode 100644 doc/man/git-bug-label-rm.1
 create mode 100644 doc/md/git-bug_label_rm.md

diff --git a/commands/label add.go b/commands/label add.go
index c7a37be803bbe49af46573d20853299ef60c6128..fccbfaa3726ec3a53e3c2499ff89ba421d44c637 100644
--- a/commands/label add.go	
+++ b/commands/label add.go	
@@ -28,14 +28,15 @@ func runLabelAdd(cmd *cobra.Command, args []string) error {
 	}
 
 	changes, err := b.ChangeLabels(add, nil)
-	if err != nil {
-		return err
-	}
 
 	for _, change := range changes {
 		fmt.Println(change)
 	}
 
+	if err != nil {
+		return err
+	}
+
 	return b.Commit()
 }
 
diff --git a/commands/label rm.go b/commands/label rm.go
new file mode 100644
index 0000000000000000000000000000000000000000..9908094c2b6467cb72bac119c6ff051ed8242ea3
--- /dev/null
+++ b/commands/label rm.go	
@@ -0,0 +1,51 @@
+package commands
+
+import (
+	"errors"
+	"fmt"
+
+	"github.com/MichaelMure/git-bug/cache"
+	"github.com/spf13/cobra"
+)
+
+func runLabelRm(cmd *cobra.Command, args []string) error {
+	if len(args) == 0 {
+		return errors.New("You must provide a bug id")
+	}
+
+	backend, err := cache.NewRepoCache(repo)
+	if err != nil {
+		return err
+	}
+	defer backend.Close()
+
+	prefix := args[0]
+	remove := args[1:]
+
+	b, err := backend.ResolveBugPrefix(prefix)
+	if err != nil {
+		return err
+	}
+
+	changes, err := b.ChangeLabels(nil, remove)
+
+	for _, change := range changes {
+		fmt.Println(change)
+	}
+
+	if err != nil {
+		return err
+	}
+
+	return b.Commit()
+}
+
+var labelRmCmd = &cobra.Command{
+	Use:   "rm <id> [<label>...]",
+	Short: "Remove a label from a bug",
+	RunE:  runLabelRm,
+}
+
+func init() {
+	labelCmd.AddCommand(labelRmCmd)
+}
diff --git a/doc/man/git-bug-label-rm.1 b/doc/man/git-bug-label-rm.1
new file mode 100644
index 0000000000000000000000000000000000000000..c7f308edd562d505250d0d7cc5ff287211caf2ae
--- /dev/null
+++ b/doc/man/git-bug-label-rm.1
@@ -0,0 +1,29 @@
+.TH "GIT-BUG" "1" "Sep 2018" "Generated from git-bug's source code" "" 
+.nh
+.ad l
+
+
+.SH NAME
+.PP
+git\-bug\-label\-rm \- Remove a label from a bug
+
+
+.SH SYNOPSIS
+.PP
+\fBgit\-bug label rm <id> [<label>\&...] [flags]\fP
+
+
+.SH DESCRIPTION
+.PP
+Remove a label from a bug
+
+
+.SH OPTIONS
+.PP
+\fB\-h\fP, \fB\-\-help\fP[=false]
+    help for rm
+
+
+.SH SEE ALSO
+.PP
+\fBgit\-bug\-label(1)\fP
diff --git a/doc/man/git-bug-label.1 b/doc/man/git-bug-label.1
index 235d704342a4892be26df4cda984150b14f745e5..6621b7a7ff9ce2b7f5414bfe2a4ad607a162da1d 100644
--- a/doc/man/git-bug-label.1
+++ b/doc/man/git-bug-label.1
@@ -26,4 +26,4 @@ Display a bug labels
 
 .SH SEE ALSO
 .PP
-\fBgit\-bug(1)\fP, \fBgit\-bug\-label\-add(1)\fP
+\fBgit\-bug(1)\fP, \fBgit\-bug\-label\-add(1)\fP, \fBgit\-bug\-label\-rm(1)\fP
diff --git a/doc/md/git-bug_label.md b/doc/md/git-bug_label.md
index 77960e153e9dc51bc778b21d024105dd7a00bf39..75f4b8435419f5a5f2d09493118fc6b66eab04bd 100644
--- a/doc/md/git-bug_label.md
+++ b/doc/md/git-bug_label.md
@@ -20,4 +20,5 @@ git-bug label <id> [flags]
 
 * [git-bug](git-bug.md)	 - A bugtracker embedded in Git
 * [git-bug label add](git-bug_label_add.md)	 - Add a label to a bug
+* [git-bug label rm](git-bug_label_rm.md)	 - Remove a label from a bug
 
diff --git a/doc/md/git-bug_label_rm.md b/doc/md/git-bug_label_rm.md
new file mode 100644
index 0000000000000000000000000000000000000000..3c740a9f47aca2ab82bf97a877ce60e16ecd2023
--- /dev/null
+++ b/doc/md/git-bug_label_rm.md
@@ -0,0 +1,22 @@
+## git-bug label rm
+
+Remove a label from a bug
+
+### Synopsis
+
+Remove a label from a bug
+
+```
+git-bug label rm <id> [<label>...] [flags]
+```
+
+### Options
+
+```
+  -h, --help   help for rm
+```
+
+### SEE ALSO
+
+* [git-bug label](git-bug_label.md)	 - Display a bug labels
+
diff --git a/misc/bash_completion/git-bug b/misc/bash_completion/git-bug
index 7283ad2a764a53d31d7727e78fe923932b716d01..d0613dc75ab8f8232a9d8eb391170be6015091ee 100644
--- a/misc/bash_completion/git-bug
+++ b/misc/bash_completion/git-bug
@@ -367,6 +367,26 @@ _git-bug_label_add()
     noun_aliases=()
 }
 
+_git-bug_label_rm()
+{
+    last_command="git-bug_label_rm"
+
+    command_aliases=()
+
+    commands=()
+
+    flags=()
+    two_word_flags=()
+    local_nonpersistent_flags=()
+    flags_with_completion=()
+    flags_completion=()
+
+
+    must_have_one_flag=()
+    must_have_one_noun=()
+    noun_aliases=()
+}
+
 _git-bug_label()
 {
     last_command="git-bug_label"
@@ -375,6 +395,7 @@ _git-bug_label()
 
     commands=()
     commands+=("add")
+    commands+=("rm")
 
     flags=()
     two_word_flags=()
diff --git a/misc/zsh_completion/git-bug b/misc/zsh_completion/git-bug
index b564942f7db4a7c0090748b71ab642bac1a0230c..35506b769980aa33526ca7d3e26d2cc313736868 100644
--- a/misc/zsh_completion/git-bug
+++ b/misc/zsh_completion/git-bug
@@ -21,7 +21,7 @@ case $state in
         _arguments '2: :(add)'
       ;;
       label)
-        _arguments '2: :(add)'
+        _arguments '2: :(add rm)'
       ;;
       status)
         _arguments '2: :(close open)'