cleaning up

src/GPM/Issue.hs

@@ -97,9 +97,9 @@ instance ToMustache NewIssue where
     , "title"       ~> title
     , "user"        ~> user
     , "branch"      ~> branch
-    , "tags"        ~> tags
+    , "tags"        ~> if null tags then Nothing else Just (T.intercalate ":" tags)
     , "assignee"    ~> assignee
-    , "reviewers"   ~> reviewers
+    , "reviewers"   ~> if null reviewers then Nothing else Just (T.intercalate "," reviewers)
     , "description" ~> description
     ]
 
@@ -119,44 +119,48 @@ handleNewIssue opts br = do
               then interactiveNewIssue newIssueTmp
               else return newIssueTmp
   createTmpNewIssue newIssue
-  validTmpNewIssue
+  validTmpNewIssue newIssue
 
 interactiveNewIssue :: NewIssue -> IO NewIssue
 interactiveNewIssue ni =
   NewIssue
-    <$> (fromMaybe (priority ni)
-          <$> ask "priority" (ptot (priority ni)) toPriority)
-    <*> (fromMaybe (status ni)
-         <$> ask "status" (status ni) identity)
-    <*> (fromMaybe (title ni)
-         <$> ask "title" (title ni) identity)
-    <*> (maybe (user ni) Just
-         <$>
-         ask "user" (fromMaybe "your name" (user ni)) identity)
-    <*> (maybe (branch ni) Just
-         <$> ask "branch" (fromMaybe "related branch" (branch ni)) identity)
-    <*> (fromMaybe (tags ni)
-         <$> ask "tags" "comma separated tags" (T.splitOn ","))
-    <*> (maybe (assignee ni) Just
-         <$> ask "assignee" "a single nick" identity)
-    <*> (fromMaybe (tags ni)
-         <$> ask "reviewers" "comma separated nicks" (T.splitOn ","))
-    <*> (maybe (description ni) Just
-         <$> ask "description" "the long description" identity)
+    <$> ask "priority" (ptot (priority ni)) toPriority
+    <*> ask "status" (status ni) identity
+    <*> ask "title" (title ni) identity
+    <*> ask "user" (fromMaybe "" (user ni)) notEmpty
+    <*> ask "branch" (fromMaybe "" (branch ni)) notEmpty
+    <*> ask "tags" (T.intercalate "," (tags ni)) (T.splitOn ",")
+    <*> ask "assignee" (fromMaybe "" (assignee ni)) notEmpty
+    <*> ask "reviewers (comma separated)" (T.intercalate "," (reviewers ni)) (T.splitOn ",")
+    <*> ask "description" (fromMaybe "" (description ni)) notEmpty
   where
     ptot :: Priority -> Text
     ptot PriorityA = "A"
     ptot PriorityB = "B"
     ptot PriorityC = "C"
-    ask :: Text -> Text -> (Text -> a) -> IO (Maybe a)
-    ask field ex tr = do
-      putText $ "Please enter " <> field <> "("<> ex <>"): "
-      fmap (tr . lineToText) <$> readline
 
-validTmpNewIssue :: IO ()
-validTmpNewIssue = do
+    notEmpty :: Text -> Maybe Text
+    notEmpty "" = Nothing
+    notEmpty str = Just str
+
+    ask :: Text -> Text -> (Text -> a) -> IO a
+    ask field defaultValue tr = do
+      putText $ "Please enter " <> field
+        <> (if defaultValue /= "" then " ("<> defaultValue <>"): " else "")
+      mline <- readline
+      case mline of
+        Nothing -> return (tr defaultValue)
+        Just line -> if line == ""
+                        then return (tr defaultValue)
+                        else return . tr . lineToText $ line
+
+validTmpNewIssue :: NewIssue -> IO ()
+validTmpNewIssue ni = do
   tmpIssue <- readFile ".issues.org.tmp"
   appendFile "issues.org" ("\n\n" <> tmpIssue)
+  debug_ "git add issues.org"
+  debug_ $ "git commit -m \"New Issue: " <> T.replace "\"" "'" (title ni) <> "\""
+  rm ".issues.org.tmp"
 
 gatherNewIssueInfos :: NewIssue -> Text -> IO NewIssue
 gatherNewIssueInfos iss br = do

src/GPM/Review.hs

@@ -257,24 +257,22 @@ createTmpNewReview nr br = do
 interactiveNewReview :: NewReview -> IO NewReview
 interactiveNewReview nr =
   NewReview
-    <$> (fromMaybe (status nr)
-         <$> ask "status" (status nr) identity)
-    <*> (fromMaybe (title nr)
-         <$> ask "title" (title nr) identity)
-    <*> (maybe (user nr) Just
-         <$>
-         ask "user" (fromMaybe "your name" (user nr)) identity)
-    <*> (maybe (branch nr) Just
-         <$> ask "branch" (fromMaybe "related branch" (branch nr)) identity)
-    <*> (maybe (description nr) Just
-         <$> ask "description" "the long description" identity)
+    <$> ask "status"                     (status nr) identity
+    <*> ask "title"                       (title nr) identity
+    <*> ask "user"          (fromMaybe "" (user nr)) notEmpty
+    <*> ask "branch"      (fromMaybe "" (branch nr)) notEmpty
+    <*> ask "description"                        ""  notEmpty
   where
-    ask :: Text -> Text -> (Text -> a) -> IO (Maybe a)
+    notEmpty :: Text -> Maybe Text
+    notEmpty "" = Nothing
+    notEmpty str = Just str
+    ask :: Text -> Text -> (Text -> a) -> IO a
     ask field ex tr = do
-      putText $ "Please enter " <> field <> " ("<> ex <>"): "
+      putText $ "Please enter " <> field
+        <> (if ex /= "" then " ("<> ex <>"): " else "")
       mline <- readline
       case mline of
-        Nothing -> return Nothing
+        Nothing -> return (tr "")
         Just line -> if line == ""
-                        then return Nothing
-                        else return . Just . tr . lineToText $ line
+                        then return (tr "")
+                        else return . tr . lineToText $ line

src/GPM/Serve.hs

@@ -53,6 +53,8 @@ init = do
     output descriptionFile "Main repositories"
   repoRoot <- getProjectRoot
   publicProjectDir <- getPublicPrjDir
+  putText (format ("rmtree " % fp) publicProjectDir)
+  rmtree publicProjectDir
   debug_ (format ("git clone --bare "%fp%" "%fp)
                  repoRoot
                  publicProjectDir)

templates/new-issue.org

@@ -1,8 +1,8 @@
-**** {{priority}} {{status}} {{title}}                                      :{{#tags}}{{tag}}:{{/tags}}
+**** {{priority}} {{status}} {{title}}{{#tags}}                             :{{tags}}:{{/tags}}
    :PROPERTIES:
-   :CREATOR:  {{user}}
-   {{#branch}}:BRANCH:   {{branch}}{{/branch}}
-   {{#assignee}}:ASSIGNEE: {{assignee}}{{/assignee}}
-   {{#reviewers}}:REQUESTED_REVIEWERS:{{#reviewers}} {{reviewer}}{{/reviewers}}{{/reviewers}}
+   :CREATOR:  {{user}}{{#branch}}
+   :BRANCH:   {{branch}}{{/branch}}{{#assignee}}
+   :ASSIGNEE: {{assignee}}{{/assignee}}{{#reviewers}}
+   :REQUESTED_REVIEWERS: {{reviewers}}{{/reviewers}}
    :END:
 {{description}}

templates/new-review.org

@@ -1,6 +1,6 @@
 **** {{status}} {{title}}
    :PROPERTIES:
-   :REVIEWER:  {{user}}
-   {{#branch}}:BRANCH:   {{branch}}{{/branch}}
+   :REVIEWER:  {{user}}{{#branch}}
+   :BRANCH: {{branch}}{{/branch}}
    :END:
 {{description}}