Log locations when unable to load config.

When we can't find a config file, log all the locations we looked and
don't create a new file in $HOME/.ptkrc.

There is currently an intermittent bug in the config loading logic that
is leading to the default config file in $HOME/.ptkrc not being found
and then being overwritten with the default config. This is step one in
fixing it.

private/api.nim

@@ -1,7 +1,7 @@
 ## Personal Time Keeping API Interface
 ## ===================================
 
-import asyncdispatch, base64, bcrypt, cliutils, docopt, jester, json, logging,
+import asyncdispatch, base64, bcrypt, cliutils, docopt, httpcore, jester, json, logging,
   sequtils, strutils, os, tables, times, uuids
 
 import nre except toSeq
@@ -61,10 +61,10 @@ template checkAuth(cfg: PtkApiCfg) =
   var user {.inject.}: PtkUser = PtkUser()
 
   try:
-    if not request.headers.hasKey("Authorization"):
+    if not headers(request).hasKey("Authorization"):
       raiseEx "No auth token."
 
-    let headerVal = request.headers["Authorization"]
+    let headerVal = headers(request)["Authorization"]
     if not headerVal.startsWith("Basic "):
       raiseEx "Invalid Authorization type (only 'Basic' is supported)."
 

private/models.nim

@@ -22,10 +22,18 @@ const ISO_TIME_FORMAT* = "yyyy-MM-dd'T'HH:mm:ss"
   ## The canonical time format used by PTK.
 
 const TIME_FORMATS* = @[
+    (fmtStr: "yyyy-MM-dd'T'HH:mm:sszzz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd HH:mm:sszzz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd'T'HH:mm:sszz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd HH:mm:sszz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd'T'HH:mm:ssz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd HH:mm:ssz", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd'T'HH:mm:ss", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd HH:mm:ss", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd'T'HH:mm", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd HH:mm", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM", offsetFrom: OffsetFrom.None),
     (fmtStr: "MM-dd'T'HH:mm:ss", offsetFrom: OffsetFrom.Year),
     (fmtStr: "MM-dd HH:mm:ss", offsetFrom: OffsetFrom.Year),
     (fmtStr: "MM-dd'T'HH:mm", offsetFrom: OffsetFrom.Year),
@@ -48,15 +56,16 @@ proc getIfExists*(n: JsonNode, key: string): JsonNode =
 
 proc parseTime*(timeStr: string): DateTime =
   ## Helper to parse time strings trying multiple known formats.
+  let now = now()
+
   for fmt in TIME_FORMATS:
     try:
-      let now = now()
       let parsed = parse(timeStr, fmt.fmtStr)
       case fmt.offsetFrom:
         of OffsetFrom.None:
           return parsed
         of OffsetFrom.Year:
-          return initDateTime(parsed.monthday, parsed.month, now.year,
+          return dateTime(now.year, parsed.month, parsed.monthday,
             parsed.hour, parsed.minute, parsed.second, parsed.nanosecond,
             now.timezone)
         of OffsetFrom.Month:

private/version.nim

@@ -1 +1 @@
-const PTK_VERSION* = "1.0.8"
+const PTK_VERSION* = "1.0.13"

ptk.nim

@@ -4,7 +4,7 @@
 ## Simple time keeping CLI
 
 import algorithm, docopt, json, langutils, logging, os, nre, std/wordwrap,
-  sequtils, sets, strutils, tempfile, terminal, times, uuids
+  sequtils, sets, strutils, sugar, tempfile, terminal, times, uuids
 
 import timeutils except `-`
 
@@ -43,8 +43,7 @@ proc writeMarks(timeline: Timeline, indices: seq[int], includeNotes = false): vo
     writeLine(stdout, "No marks match the given criteria.")
     return
 
-  var idxs = indices.sorted(
+  var idxs = indices.sorted((a, b) => cmp(marks[a].time, marks[b].time))
-    proc(a, b: int): int = cmp(marks[a].time, marks[b].time))
 
   let largestInterval = now - marks[idxs.first].time
   let timeFormat =
@@ -279,8 +278,9 @@ Options:
   -e --edit               Open the mark in an editor.
   -f --file <file>        Use the given timeline file.
   -g --tags <tags>        Add the given tags (comma-separated) to the selected marks.
-  -G --remove-tags <tagx> Remove the given tag from the selected marks.
+  -G --remove-tags <tags> Remove the given tag from the selected marks.
   -h --help               Print this usage information.
+
   -m --matching <pattern> Restric the selection to marks matching <pattern>.
   -n --notes <notes>      For add and amend, set the notes for a time mark.
   -t --time <time>        For add and amend, use this time instead of the current time.
@@ -318,12 +318,13 @@ Options:
   var cfgFile: File
   if not fileExists(ptkrcFilename):
     warn "ptk: could not find .ptkrc file."
-    ptkrcFilename = $getEnv("HOME") & "/.ptkrc"
+    debug "ptk: considered the following locations:\n\t" & ptkrcLocations.join("\n\t")
-    try:
+    #ptkrcFilename = $getEnv("HOME") & "/.ptkrc"
-      cfgFile = open(ptkrcFilename, fmWrite)
+    #try:
-      cfgFile.write("{\"timelineLogFile\": \"timeline.log.json\"}")
+    #  cfgFile = open(ptkrcFilename, fmWrite)
-    except: warn "ptk: could not write default .ptkrc to " & ptkrcFilename
+    #  cfgFile.write("{\"timelineLogFile\": \"timeline.log.json\"}")
-    finally: close(cfgFile)
+    #except: warn "ptk: could not write default .ptkrc to " & ptkrcFilename
+    #finally: close(cfgFile)
 
   try: cfg = parseFile(ptkrcFilename)
   except: raise newException(IOError,
@@ -491,8 +492,7 @@ Options:
         mark.tags = mark.tags.deduplicate
       if args["--remove-tags"]:
         let tagsToRemove = (args["--remove-tags"] ?: "").split({',', ';'})
-        mark.tags = mark.tags.filter(proc (t: string): bool =
+        mark.tags = mark.tags.filter((t) => not anyIt(tagsToRemove, it == t))
-          anyIt(tagsToRemove, it == t))
       if args["--time"]:
         try: mark.time = parseTime($args["--time"])
         except: raise newException(ValueError,

ptk.nimble

@@ -1,6 +1,6 @@
 # Package
 
-version       = "1.0.8"
+version       = "1.0.13"
 author        = "Jonathan Bernard"
 description   = "Personal Time Keeper"
 license       = "MIT"
@@ -16,10 +16,10 @@ requires @[
   "isaac >= 0.1.3",
   "bcrypt",
   "jester 0.5.0",
-  "https://git.jdb-labs.com/jdb/nim-lang-utils.git",
+  "https://git.jdb-software.com/jdb/nim-lang-utils.git",
-  "https://git.jdb-labs.com/jdb/nim-cli-utils.git >= 0.6.5",
+  "https://git.jdb-software.com/jdb/nim-cli-utils.git >= 0.6.5",
-  "https://git.jdb-labs.com/jdb/nim-time-utils.git >= 0.5.2",
+  "https://git.jdb-software.com/jdb/nim-time-utils.git >= 0.5.2",
-  "https://git.jdb-labs.com/jdb/update-nim-package-version"
+  "https://git.jdb-software.com/jdb/update-nim-package-version"
 ]
 
 task updateVersion, "Update the version of this package.":