vmail 2.3.7 → 2.3.8

data/lib/vmail.vim

@@ -632,7 +632,7 @@ endfunc
 function! s:send_message()
   let mail = join(getline(1,'$'), "\n")
   echo "Sending message"
-  let res = s:system_with_error_handling(s:deliver_command, mail)
+  let res = system(s:deliver_command, mail)
   if match(res, '^Failed') == -1
     write!
     call s:close_and_focus_list_window()

data/lib/vmail/imap_client.rb

@@ -194,8 +194,7 @@ module Vmail
       self.max_seqno -= num
     end
 
-    # TODO why not just reload the current page?
-    def update
+    def check_for_new_messages
       if search_query?
         log "Update aborted because query is search query: #{@query.inspect}"
         return ""
@@ -215,10 +214,16 @@ module Vmail
       log "- got seqnos: #{ids.inspect}"
       log "- getting seqnos > #{self.max_seqno}"
       new_ids = ids.select {|seqno| seqno > self.max_seqno}
-      @ids = @ids + new_ids
-      log "- update: new uids: #{new_ids.inspect}"
+      log "- new uids found: #{new_ids.inspect}"
+      new_ids
+    end
+
+    # TODO why not just reload the current page?
+    def update
+      new_ids = check_for_new_messages 
       if !new_ids.empty?
         self.max_seqno = new_ids[-1]
+        @ids = @ids + new_ids
         message_ids = fetch_and_cache_headers(new_ids)
         res = get_message_headers(message_ids)
         res
@@ -226,8 +231,7 @@ module Vmail
         ''
       end
     rescue
-      log $!
-      log $!.backtrace.join("\n")
+      puts "VMAIL_ERROR: #{[$!.message, $!.backtrace].join("\n")}"
     end
 
     # gets 100 messages prior to id

data/lib/vmail/version.rb

@@ -1,3 +1,3 @@
 module Vmail
-  VERSION = '2.3.7'
+  VERSION = '2.3.8'
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 2
   - 3
-  - 7
-  version: 2.3.7
+  - 8
+  version: 2.3.8
 platform: ruby
 authors: 
 - Daniel Choi