gmail-api-ruby 0.0.9 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 085675b3ee758627447a15b1e2c3fc0eb4b158a0
-  data.tar.gz: 7772bbd3cb2ae53a45b4e48613620c98fd2175c7
+  metadata.gz: 7f721a46ac6615f3e1fb812174992b8a10bce73b
+  data.tar.gz: 6d1ae38e0a720d3fa4e644f50b6b12ef9a5d4e21
 SHA512:
-  metadata.gz: 269893f1e9fb4b1b877a46d632d44f08f3e3dde7b5ca3bf3364615b7775f1bf6a90eee9ab201c2401bed5c634648357de58efa5448256ad4bc59f0f9bad3c3b4
-  data.tar.gz: 0013825ff0f97f195138c28e4209d954f03cca732a267d340bff9c3cb03e1eced8faf3543d66ce2e49eb53f82bc75fe7a839ed0fbcdc09529a6d4ca903a3550a
+  metadata.gz: 6325c873f3f7495a5be90837a3b787f1b34ad61e86817b184d01d624beee3701e2d756dd04ec29cdb13a5b78012e13b2bd67129c4173397121b937d2be64efe2
+  data.tar.gz: d471f984ec9a7878730efa8e5ba9e1b53e9fa6e4b84e732444e627c6681cf3664004364b396b23bf04fcf182b0a6d5aaf5dcb61a7470a5d7e7cdb7ef586f7b30

data/CHANGELOG.md

@@ -31,3 +31,7 @@
 
 * minor fix
 
+## 0.0.10 / 2015-02-17
+
+* Fixed email parsing in forward and reply method when contact names contain ,
+

data/gmail.gemspec

@@ -23,6 +23,7 @@ Gem::Specification.new do |s|
   s.add_dependency 'google-api-client'
   s.add_dependency "hooks", ">=0.4.0"
   s.add_dependency "hashie", ">=3.3.2"
+  s.add_dependency "stringex"
   
   # development dependencies
   s.add_development_dependency "rake"

data/lib/gmail/message.rb

@@ -7,6 +7,8 @@ module Gmail
     include Base::Modify
     include Base::Trash
 
+    require "stringex"
+
     after_initialize :set_basics
 
     def thread
@@ -136,10 +138,13 @@ module Gmail
     end
 
     def set_headers_for_reply msg
-      to_ar = []
-      split_regexp = Regexp.new("\s*,\s*")
+      #to_ar = []
+      #split_regexp = Regexp.new("\s*,\s*")
       own_email = delivered_to || Gmail.mailbox_email
-      to_ar = (to || "").split(split_regexp) + (cc || "").split(split_regexp)
+
+
+      to_ar = (Mail::AddressList.new("#{to}".to_ascii).addresses + Mail::AddressList.new("#{cc}".to_ascii).addresses).map(&:to_s)
+      #to_ar = (to || "").split(split_regexp) + (cc || "").split(split_regexp)
       result = to_ar.grep(Regexp.new(own_email, "i"))
       to_ar = to_ar - result
 

data/lib/gmail/version.rb

@@ -1,3 +1,3 @@
 module Gmail
-  VERSION = "0.0.9"
+  VERSION = "0.0.10"
 end # Gmail

data/test/gmail/message_test.rb

@@ -286,6 +286,18 @@ module Gmail
 
     end
 
+    should "Construct correctly set_headers_for_reply" do
+      m = Gmail::Message.new({body: ""})
+      m.from = "\"John, Malkovich\" john@malkovich.com"
+      m.to = "\"Julie, Desk\"julie@juliedesk.com, \"Judith, Desk\"judith@juliedesk.com"
+      m.delivered_to = "julie@juliedesk.com"
+      new_msg = m.send(:set_headers_for_reply, Gmail::Message.new({body: ""}))
+
+      #assert_equal new_msg.from, "\"Julie, Desk\"julie@juliedesk.com"
+      assert_equal new_msg.to, "\"John, Malkovich\" john@malkovich.com"
+      assert_equal new_msg.cc, "\"Judith, Desk\"judith@juliedesk.com"
+    end
+
     should "Reply to all construct should be easy and call getProfile if delivered_to is not set" do
       m = Gmail::Message.new test_to_reply_message2
       reply_message = Gmail::Message.new test_reply_message

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gmail-api-ruby
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
 platform: ruby
 authors:
 - Julien Hobeika
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-12 00:00:00.000000000 Z
+date: 2015-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mime
@@ -80,6 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.3.2
+- !ruby/object:Gem::Dependency
+  name: stringex
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement