feat/incorrect priority parsing (#383)

Author: jgollenz

lua/orgmode/objects/priority_state.lua

@@ -29,13 +29,18 @@ function PriorityState:prompt_user()
   local choice = vim.fn.input(prompt)
 
   if choice == '' then
+    utils.echo_warning(string.format("Priority must be between '%s' and '%s'", self.high_priority, self.low_priority))
     return nil
   end
 
   choice = string.upper(choice)
+  if #choice > 1 and tonumber(choice) == nil then
+    utils.echo_warning(string.format('Only numeric priorities can be multiple characters long'))
+    return nil
+  end
   local choicenum = string.byte(choice)
   if choice ~= ' ' and (choicenum < string.byte(self.high_priority) or choicenum > string.byte(self.low_priority)) then
-    utils.echo_warning(string.format("Priority must be between '%s' and '%s'", self.low_priority, self.high_priority))
+    utils.echo_warning(string.format("Priority must be between '%s' and '%s'", self.high_priority, self.low_priority))
     return nil
   end
 

lua/orgmode/org/mappings.lua

@@ -327,6 +327,9 @@ function OrgMappings:set_priority(direction)
     new_priority = priority_state:decrease()
   elseif direction == nil then
     new_priority = priority_state:prompt_user()
+    if new_priority == nil then
+      return
+    end
   end
 
   headline:set_priority(new_priority)

lua/orgmode/treesitter/headline.lua

@@ -91,6 +91,12 @@ function Headline:set_priority(priority)
     return
   end
 
+  -- TODO: input validation is split between here and priority_state.lua:prompt_user().
+  --       should be unified
+  if vim.trim(priority) == '' then
+    return
+  end
+
   local todo = self:todo()
   if todo then
     local text = query.get_node_text(todo, 0)