vcard 0.2.2 → 0.2.3

data/lib/vcard.rb

@@ -30,9 +30,9 @@ module Vcard
         line.chomp!
         # If it's a continuation line, add it to the last.
         # If it's an empty line, drop it from the input.
-        if( line =~ /^[ \t]/ )
+        if line =~ /^[ \t]/
           unfolded[-1] << line[1, line.size-1]
-        elsif( line =~ /^$/ )
+        elsif line =~ /^$/
         else
           unfolded << line
         end
@@ -49,14 +49,13 @@ module Vcard
       item.chomp!(sep)
       list << yield(item)
     end
+
     list
   end
 
   # Convert a RFC 2425 date into an array of [year, month, day].
   def self.decode_date(v) # :nodoc:
-    if !(v =~ Bnf::DATE)
-      raise ::Vcard::InvalidEncodingError, "date not valid (#{v})"
-    end
+    raise ::Vcard::InvalidEncodingError, "date not valid (#{v})" unless v =~ Bnf::DATE
     [$1.to_i, $2.to_i, $3.to_i]
   end
 
@@ -70,31 +69,29 @@ module Vcard
 
   # Encode a Date object as "yyyymmdd".
   def self.encode_date(d) # :nodoc:
-     "%0.4d%0.2d%0.2d" % [ d.year, d.mon, d.day ]
+     "%0.4d%0.2d%0.2d" % [d.year, d.mon, d.day]
   end
 
   # Encode a Date object as "yyyymmdd".
   def self.encode_time(d) # :nodoc:
-     "%0.4d%0.2d%0.2d" % [ d.year, d.mon, d.day ]
+     "%0.4d%0.2d%0.2d" % [d.year, d.mon, d.day]
   end
 
   # Encode a Time or DateTime object as "yyyymmddThhmmss"
   def self.encode_date_time(d) # :nodoc:
-     "%0.4d%0.2d%0.2dT%0.2d%0.2d%0.2d" % [ d.year, d.mon, d.day, d.hour, d.min, d.sec ]
+     "%0.4d%0.2d%0.2dT%0.2d%0.2d%0.2d" % [d.year, d.mon, d.day, d.hour, d.min, d.sec]
   end
 
   # Convert a RFC 2425 time into an array of [hour,min,sec,secfrac,timezone]
   def self.decode_time(v) # :nodoc:
-    if !(match = Bnf::TIME.match(v))
-      raise ::Vcard::InvalidEncodingError, "time '#{v}' not valid"
-    end
+    raise ::Vcard::InvalidEncodingError, "time '#{v}' not valid" unless match = Bnf::TIME.match(v)
     hour, min, sec, secfrac, tz = match.to_a[1..5]
 
     [hour.to_i, min.to_i, sec.to_i, secfrac ? secfrac.to_f : 0, tz]
   end
 
   def self.array_datetime_to_time(dtarray) #:nodoc:
-  # We get [ year, month, day, hour, min, sec, usec, tz ]
+  # We get [year, month, day, hour, min, sec, usec, tz]
     tz = (dtarray.pop == "Z") ? :gm : :local
     Time.send(tz, *dtarray)
   rescue ArgumentError => e
@@ -108,9 +105,7 @@ module Vcard
 
   # Convert a RFC 2425 date-time into an array of [year,mon,day,hour,min,sec,secfrac,timezone]
   def self.decode_date_time(v) # :nodoc:
-    if !(match = Bnf::DATE_TIME.match(v))
-      raise ::Vcard::InvalidEncodingError, "date-time '#{v}' not valid"
-    end
+    raise ::Vcard::InvalidEncodingError, "date-time '#{v}' not valid" unless match = Bnf::DATE_TIME.match(v)
     year, month, day, hour, min, sec, secfrac, tz = match.to_a[1..8]
 
     [
@@ -135,9 +130,7 @@ module Vcard
 
   # Convert an RFC2425 INTEGER value into an Integer
   def self.decode_integer(v) # :nodoc:
-    if !(v =~ Bnf::INTEGER)
-      raise ::Vcard::InvalidEncodingError, "integer not valid (#{v})"
-    end
+    raise ::Vcard::InvalidEncodingError, "integer not valid (#{v})" unless v =~ Bnf::INTEGER
     v.to_i
   end
 
@@ -148,9 +141,7 @@ module Vcard
   def self.decode_date_list(v) # :nodoc:
     decode_list(v) do |date|
       date.strip!
-      if date.length > 0
-        decode_date(date)
-      end
+      decode_date(date) if date.length > 0
     end.compact
   end
 
@@ -158,9 +149,7 @@ module Vcard
   def self.decode_time_list(v) # :nodoc:
     decode_list(v) do |time|
       time.strip!
-      if time.length > 0
-        decode_time(time)
-      end
+      decode_time(time) if time.length > 0
     end.compact
   end
 
@@ -168,9 +157,7 @@ module Vcard
   def self.decode_date_time_list(v) # :nodoc:
     decode_list(v) do |datetime|
       datetime.strip!
-      if datetime.length > 0
-        decode_date_time(datetime)
-      end
+      decode_date_time(datetime) if datetime.length > 0
     end.compact
   end
 
@@ -197,27 +184,21 @@ module Vcard
   end
 
   def self.encode_text(v) #:nodoc:
-    v.to_str.gsub(/([\\,;\n])/) { $1 == "\n" ? "\\n" : "\\"+$1 }
+    v.to_str.gsub(/([\\,;\n])/) { $1 == "\n" ? "\\n" : "\\#{$1}" }
   end
 
   # v is an Array of String, or just a single String
   def self.encode_text_list(v, sep = ",") #:nodoc:
-    begin
-      v.to_ary.map{ |t| encode_text(t) }.join(sep)
-    rescue
-      encode_text(v)
-    end
+    v.to_ary.map { |t| encode_text(t) }.join(sep)
+  rescue
+    encode_text(v)
   end
 
   # Convert a +sep+-seperated list of TEXT values into an array of values.
   def self.decode_text_list(value, sep = ",") # :nodoc:
     # Need to do in two stages, as best I can find.
-    list = value.scan(/([^#{sep}\\]*(?:\\.[^#{sep}\\]*)*)#{sep}/).map do |v|
-      decode_text(v.first)
-    end
-    if value.match(/([^#{sep}\\]*(?:\\.[^#{sep}\\]*)*)$/)
-      list << $1
-    end
+    list = value.scan(/([^#{sep}\\]*(?:\\.[^#{sep}\\]*)*)#{sep}/).map { |v| decode_text(v.first) }
+    list << $1 if value.match(/([^#{sep}\\]*(?:\\.[^#{sep}\\]*)*)$/)
     list
   end
 
@@ -236,7 +217,7 @@ module Vcard
     if value =~ Bnf::ALL_SAFECHARS
       value
     elsif value =~ Bnf::ALL_QSAFECHARS
-      '"' + value + '"'
+      %Q{"#{value}"}
     else
       raise ::Vcard::Unencodable, "param-value #{value.inspect}"
     end
@@ -245,7 +226,7 @@ module Vcard
   # Unfold the lines in +card+, then return an array of one Field object per
   # line.
   def self.decode(card) #:nodoc:
-    unfold(card).collect { |line| DirectoryInfo::Field.decode(line) }
+    unfold(card).map { |line| DirectoryInfo::Field.decode(line) }
   end
 
 
@@ -259,15 +240,14 @@ module Vcard
     dst = []
     # stack used to track our nesting level, as we see begin/end we start a
     # new/finish the current entity, and push/pop that entity from the stack
-    current = [ dst ]
+    current = [dst]
 
     for f in src
       if f.name? "BEGIN"
-        e = [ f ]
+        e = [f]
 
         current.last.push(e)
         current.push(e)
-
       elsif f.name? "END"
         current.last.push(f)
 
@@ -276,7 +256,6 @@ module Vcard
         end
 
         current.pop
-
       else
         current.last.push(f)
       end
@@ -296,8 +275,8 @@ module Vcard
     inner = []
     fields.each do |line|
       case line
-      when Array; inner << line
-      else;       outer << line
+      when Array then inner << line
+      else outer << line
       end
     end
     return outer, inner

data/lib/vcard/bnf.rb

@@ -1,3 +1,4 @@
+# encoding: ascii
 # Copyright (C) 2008 Sam Roberts
 
 # This library is free software; you can redistribute it and/or modify
@@ -59,12 +60,14 @@ module Vcard
 
     # QSAFE-CHAR = WSP / %x21 / %x23-7E / NON-US-ASCII
     #  ; Any character except CTLs and DQUOTE
-    QSAFECHAR = /[ \t\x21\x23-\x7e\x80-\xff]/
+    # set ascii encoding so that multibyte chars can be properly escaped
+    QSAFECHAR = Regexp.new("[ \t\x21\x23-\x7e\x80-\xff]")
     ALL_QSAFECHARS = /\A#{QSAFECHAR}*\z/
 
     # SAFE-CHAR  = WSP / %x21 / %x23-2B / %x2D-39 / %x3C-7E / NON-US-ASCII
     #   ; Any character except CTLs, DQUOTE, ";", ":", ","
-    SAFECHAR = /[ \t\x21\x23-\x2b\x2d-\x39\x3c-\x7e\x80-\xff]/
+    # escape character classes then create new Regexp
+    SAFECHAR = Regexp.new(Regexp.escape("[ \t\x21\x23-\x2b\x2d-\x39\x3c-\x7e\x80-\xff]"))
     ALL_SAFECHARS = /\A#{SAFECHAR}*\z/
   end
 end

data/lib/vcard/version.rb

@@ -1,3 +1,3 @@
 module Vcard
-  VERSION = "0.2.2"
+  VERSION = "0.2.3"
 end

data/test/test_helper.rb

@@ -7,25 +7,25 @@ class Test::Unit::TestCase
   # Test equivalence where whitespace is compressed.
 
   def assert_equal_nospace(expected, got)
-    expected = expected.gsub(/\s+/,'')
-    got = expected.gsub(/\s+/,'')
+    expected = expected.gsub(/\s+/, "")
+    got = expected.gsub(/\s+/, "")
     assert_equal(expected, got)
   end
 
   def utf_name_test(c)
     card = Vcard::Vcard.decode(c).first
     assert_equal("name", card.name.family)
-  rescue
-    $!.message << " #{c.inspect}"
+  rescue => exception
+    exception.message << " #{c.inspect}"
     raise
   end
 
   def be(s)
-    s.unpack('U*').pack('n*')
+    s.unpack("U*").pack("n*")
   end
 
   def le(s)
-    s.unpack('U*').pack('v*')
+    s.unpack("U*").pack("v*")
   end
 
   def vcard(name)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vcard
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-17 00:00:00.000000000 Z
+date: 2013-02-14 00:00:00.000000000 Z
 dependencies: []
 description: Vcard extracted from Vpim
 email:
@@ -76,18 +76,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 376442794917458825
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 376442794917458825
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24