Merge pull request #84 from Steap/allow-empty-comment-message

Michael Muré created 6 years ago

Allow empty messages in comments.

Change summary

bug/op_add_comment.go | 4 ----
bug/operation_test.go | 1 -
commands/show.go      | 9 ++++++++-
3 files changed, 8 insertions(+), 6 deletions(-)

Detailed changes

bug/op_add_comment.go

@@ -58,10 +58,6 @@ func (op *AddCommentOperation) Validate() error {
 		return err
 	}
 
-	if text.Empty(op.Message) {
-		return fmt.Errorf("message is empty")
-	}
-
 	if !text.Safe(op.Message) {
 		return fmt.Errorf("message is not fully printable")
 	}

bug/operation_test.go

@@ -47,7 +47,6 @@ func TestValidate(t *testing.T) {
 		NewSetTitleOp(rene, unix, "title", "multi\nline"),
 		NewSetTitleOp(rene, unix, "title\u001b", "title2"),
 		NewSetTitleOp(rene, unix, "title", "title2\u001b"),
-		NewAddCommentOp(rene, unix, "", nil),
 		NewAddCommentOp(rene, unix, "message\u001b", nil),
 		NewAddCommentOp(rene, unix, "message", []git.Hash{git.Hash("invalid")}),
 		NewSetStatusOp(rene, unix, 1000),

commands/show.go

@@ -58,6 +58,7 @@ func runShowBug(cmd *cobra.Command, args []string) error {
 	indent := "  "
 
 	for i, comment := range snapshot.Comments {
+		var message string
 		fmt.Printf("%s#%d %s <%s>\n\n",
 			indent,
 			i,
@@ -65,9 +66,15 @@ func runShowBug(cmd *cobra.Command, args []string) error {
 			comment.Author.Email,
 		)
 
+		if comment.Message == "" {
+			message = colors.GreyBold("No description provided.")
+		} else {
+			message = comment.Message
+		}
+
 		fmt.Printf("%s%s\n\n\n",
 			indent,
-			comment.Message,
+			message,
 		)
 	}