simple_currency 1.2.2 → 1.3.0

data/README.md

@@ -37,6 +37,15 @@ in the past? Just do this:
     42.eur.at(Time.parse('2009-09-01')).to_usd
     # => 60.12
 
+You can also add an subtract money expressions, which will return a result
+converted to the former currency of the expression:
+
+    42.eur + 30.usd
+    # => The same as adding 42 and 30.usd.to_eur
+
+    10.gbp + 1.eur
+    # => The same as adding 10 and 1.eur.to_gbp
+
 ##Installation
 
 ###Rails 3

data/VERSION

@@ -1 +1 @@
-1.2.2
+1.3.0

data/lib/core_ext/{numeric.rb → fixnum.rb}

@@ -1,3 +1,3 @@
-class Numeric
+class Fixnum
   include CurrencyConvertible
 end

data/lib/core_ext/float.rb

@@ -0,0 +1,3 @@
+class Float
+  include CurrencyConvertible
+end

data/lib/simple_currency.rb

@@ -1,2 +1,3 @@
 require 'simple_currency/currency_convertible'
-require 'core_ext/numeric'
+require 'core_ext/fixnum'
+require 'core_ext/float'

data/lib/simple_currency/currency_convertible.rb

@@ -4,53 +4,82 @@ require 'crack/xml'
 
 module CurrencyConvertible
 
-  def method_missing(method, *args, &block)
-    return _from(method.to_s) if method.to_s.length == 3 # Presumably a currency ("eur", "gbp"...)
+  Operators = {:+ => :add,
+               :- => :subtract}
 
-    # Now capture methods like to_eur, to_gbp, to_usd... 
-    if @original && !(method.to_s =~ /^to_(utc|int|str|ary)/) && method.to_s =~/^to_/ && method.to_s.length == 6
-      return _to(method.to_s.gsub('to_',''))
-    end
+  def add_with_currency(arg)
+    return add_without_currency(arg) unless arg.is_a? CurrencyConvertible::Proxy
+    add_without_currency(arg)
+  end
 
-    super(method,*args,&block)
+  def subtract_with_currency(arg)
+    return subtract_without_currency(arg) unless arg.is_a? CurrencyConvertible::Proxy
+    subtract_without_currency(arg)
   end
 
-  # Historical exchange lookup
-  def at(exchange = nil)
-    begin
-       
-      @exchange_date = exchange.send(:to_date)
-    rescue
-      raise "Must use 'at' with a time or date object"
-    end
-    self
+  def self.included(base)
+    base.send(:alias_method, :add_without_currency, :+)
+    base.send(:undef_method, :+)
+    base.send(:alias_method, :+, :add_with_currency)
+
+    base.send(:alias_method, :subtract_without_currency, :-)
+    base.send(:undef_method, :-)
+    base.send(:alias_method, :-, :subtract_with_currency)
   end
 
-  private
-   
-    # Called from first currency metamethod to set the original currency.
-    # 
-    # 30.eur # => Calls _from and sets @original to 'eur'
-    #
-    def _from(currency)
+  def method_missing(method, *args, &block)
+    return CurrencyConvertible::Proxy.new(self,method.to_s) if method.to_s.length == 3 # Presumably a currency ("eur", "gbp"...)
+    super(method,*args,&block)
+  end
+
+  class Proxy
+    attr_reader :numeric
+
+    def initialize(numeric,currency)
+      @numeric = numeric
+      @currency = currency
       @exchange_date = Time.now.send(:to_date)
-      @original = currency
+    end
+
+    def method_missing(method, *args, &block)
+      if !(method.to_s =~ /^to_(utc|int|str|ary)/) && method.to_s =~/^to_/ && method.to_s.length == 6
+        return _to(method.to_s.gsub('to_',''))
+      end
+      @numeric.send(method, *args, &block)
+    end
+
+    # Historical exchange lookup
+    def at(exchange = nil)
+      begin
+        @exchange_date = exchange.send(:to_date)
+      rescue
+        raise "Must use 'at' with a time or date object"
+      end
       self
     end
 
-    # Called from last currency metamethod to set the target currency.
-    # 
-    #   30.eur.to_usd 
-    #   # => Calls _to and returns the final value, say 38.08
-    #
+    def +(other)
+      return @numeric + other unless other.is_a? CurrencyConvertible::Proxy
+      converted = other.send(:"to_#{@currency}")
+      @numeric + converted
+    end
+
+    def -(other)
+      return @numeric - other unless other.is_a? CurrencyConvertible::Proxy
+      converted = other.send(:"to_#{@currency}")
+      @numeric - converted
+    end
+
+    private
+
     def _to(target)
-      raise unless @original # Must be called after a _from have set the @original currency
+      raise unless @currency
 
-      return 0.0 if self == 0 # Obviously
+      return 0.0 if @numeric == 0 # Obviously
 
-      original = @original
+      original = @currency
 
-      amount = self
+      amount = @numeric
 
       # Check if there's a cached exchange rate for today
       return cached_amount(original, target, amount) if cached_rate(original, target)
@@ -59,16 +88,16 @@ module CurrencyConvertible
       result = exchange(original, target, amount.abs)
 
       # Cache methods
-      cache_currency_methods(original, target)
+      #cache_currency_methods(original, target)
 
       result
     end
 
-    # Main method (called by _to) which calls Xavier or Xurrency strategies
+    # Main method (called by _to) which calls Xavier API
     # and returns a nice result.
     #
     def exchange(original, target, amount)
-      negative = (self < 0)
+      negative = (@numeric < 0)
 
       # Get the result and round it to 2 decimals
       result = sprintf("%.2f", call_xavier_api(original, target, amount)).to_f
@@ -117,8 +146,10 @@ module CurrencyConvertible
             uri = URI.parse(api_url)
             retry
           else
-            raise "404 Not Found"
+            raise NotFoundError.new("404 Not Found")
           end
+        rescue SocketError
+          raise NotFoundError.new("Socket Error")
         end
 
         return nil unless xml_response && parsed_response = Crack::XML.parse(xml_response)
@@ -152,20 +183,6 @@ module CurrencyConvertible
       rate.first['rate'].to_f
     end
 
-    # Caches currency methods to avoid method missing abuse.
-    #
-    def cache_currency_methods(original, target)
-      # Cache the _from method for faster reuse
-      self.class.send(:define_method, original.to_sym) do
-          _from(original)
-      end unless self.respond_to?(original.to_sym)
-
-      # Cache the _to method for faster reuse
-      self.class.send(:define_method, :"to_#{target}") do
-          _to(target)
-      end unless self.respond_to?(:"to_#{target}")
-    end
-
     ##
     # Cache helper methods (only useful in a Rails app)
     ##
@@ -209,6 +226,7 @@ module CurrencyConvertible
       end
       nil
     end
+  end
 
 end
 
@@ -222,4 +240,7 @@ end
   class NoRatesFoundException < StandardError
   end
 
+  class NotFoundError < StandardError
+  end
+
 

data/simple_currency.gemspec

@@ -5,45 +5,47 @@
 
 Gem::Specification.new do |s|
   s.name = %q{simple_currency}
-  s.version = "1.2.2"
+  s.version = "1.3.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Oriol Gual", "Josep M. Bach", "Josep Jaume Rey"]
-  s.date = %q{2010-10-14}
+  s.date = %q{2010-10-24}
   s.description = %q{A really simple currency converter using XavierMedia API. It's Ruby 1.8, 1.9 and JRuby compatible, and it also takes advantage of Rails cache when available.}
   s.email = %q{info@codegram.com}
   s.extra_rdoc_files = [
     "LICENSE",
-    "README.md"
+     "README.md"
   ]
   s.files = [
     ".autotest",
-    ".bundle/config",
-    ".gitignore",
-    ".rspec",
-    ".rvmrc",
-    "Gemfile",
-    "Gemfile.lock",
-    "LICENSE",
-    "README.md",
-    "Rakefile",
-    "VERSION",
-    "autotest/discover.rb",
-    "lib/core_ext/numeric.rb",
-    "lib/simple_currency.rb",
-    "lib/simple_currency/currency_convertible.rb",
-    "simple_currency.gemspec",
-    "spec/simple_currency_spec.rb",
-    "spec/spec_helper.rb",
-    "spec/support/xavier.xml"
+     ".bundle/config",
+     ".gitignore",
+     ".rspec",
+     ".rvmrc",
+     "Gemfile",
+     "Gemfile.lock",
+     "LICENSE",
+     "README.md",
+     "Rakefile",
+     "VERSION",
+     "autotest/discover.rb",
+     "lib/core_ext/fixnum.rb",
+     "lib/core_ext/float.rb",
+     "lib/simple_currency.rb",
+     "lib/simple_currency/currency_convertible.rb",
+     "simple_currency.gemspec",
+     "spec/simple_currency_spec.rb",
+     "spec/spec_helper.rb",
+     "spec/support/xavier.xml"
   ]
   s.homepage = %q{http://github.com/codegram/simple_currency}
+  s.rdoc_options = ["--charset=UTF-8"]
   s.require_paths = ["lib"]
   s.rubygems_version = %q{1.3.7}
   s.summary = %q{A really simple currency converter using XavierMedia API.}
   s.test_files = [
     "spec/simple_currency_spec.rb",
-    "spec/spec_helper.rb"
+     "spec/spec_helper.rb"
   ]
 
   if s.respond_to? :specification_version then
@@ -51,7 +53,6 @@ Gem::Specification.new do |s|
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<crack>, [">= 0"])
       s.add_runtime_dependency(%q<crack>, [">= 0.1.8"])
       s.add_development_dependency(%q<jeweler>, [">= 1.4.0"])
       s.add_development_dependency(%q<rspec>, [">= 2.0.0.beta.22"])
@@ -59,7 +60,6 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<rails>, [">= 3.0.0"])
       s.add_development_dependency(%q<bundler>, [">= 1.0.0"])
     else
-      s.add_dependency(%q<crack>, [">= 0"])
       s.add_dependency(%q<crack>, [">= 0.1.8"])
       s.add_dependency(%q<jeweler>, [">= 1.4.0"])
       s.add_dependency(%q<rspec>, [">= 2.0.0.beta.22"])
@@ -68,7 +68,6 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<bundler>, [">= 1.0.0"])
     end
   else
-    s.add_dependency(%q<crack>, [">= 0"])
     s.add_dependency(%q<crack>, [">= 0.1.8"])
     s.add_dependency(%q<jeweler>, [">= 1.4.0"])
     s.add_dependency(%q<rspec>, [">= 2.0.0.beta.22"])

data/spec/simple_currency_spec.rb

@@ -10,6 +10,44 @@ describe "SimpleCurrency" do
     0.usd.to_eur.should == 0.0 
   end
 
+  describe "operators" do
+
+    let(:today) { Time.now }
+
+    before(:each) do
+      mock_xavier_api(today)
+    end
+
+    describe "#+" do
+      it "adds two money expressions" do
+        (1.eur + 1.27.usd).should == 2
+        (1.27.usd + 1.eur).should == 2.54
+        (1.eur + 1.eur).should == 2
+      end
+
+      it "does not affect non-money expressions" do
+        (1 + 1.27).should == 2.27
+        (38.eur + 1.27).should == 39.27
+        (1.27.usd + 38).should == 39.27
+      end
+    end
+
+    describe "#-" do
+      it "subtracts two money expressions" do
+        (1.eur - 1.27.usd).should == 0
+        (1.27.usd - 1.eur).should == 0
+        (1.eur - 1.eur).should == 0
+      end
+
+      it "does not affect non-money expressions" do
+        (1 - 1.27).should == -0.27
+        (38.eur - 1.27).should == 36.73
+        (1.27.usd - 38).should == -36.73
+      end
+    end
+
+  end
+
   context "using XavierMedia API for exchange" do
 
     let(:today) { Time.now }
@@ -74,10 +112,12 @@ describe "SimpleCurrency" do
         expect {
           begin 
             1.usd.at(the_past).to_eur
+          rescue NotFoundError=>e
+            raise e
           rescue Timeout::Error
             retry
           end
-        }.to raise_error("404 Not Found")
+        }.to raise_error(NotFoundError)
 
       end
 
@@ -139,4 +179,5 @@ describe "SimpleCurrency" do
 
   end
 
+
 end

metadata

@@ -5,9 +5,9 @@ version: !ruby/object:Gem::Version
   prerelease: false
   segments: 
   - 1
-  - 2
-  - 2
-  version: 1.2.2
+  - 3
+  - 0
+  version: 1.3.0
 platform: ruby
 authors: 
 - Oriol Gual
@@ -17,28 +17,13 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-14 00:00:00 +02:00
+date: 2010-10-24 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  type: :runtime
-  prerelease: false
   name: crack
-  version_requirements: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  requirement: *id001
-- !ruby/object:Gem::Dependency 
-  type: :runtime
   prerelease: false
-  name: crack
-  version_requirements: &id002 !ruby/object:Gem::Requirement 
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -49,12 +34,12 @@ dependencies:
         - 1
         - 8
         version: 0.1.8
-  requirement: *id002
+  type: :runtime
+  version_requirements: *id001
 - !ruby/object:Gem::Dependency 
-  type: :development
-  prerelease: false
   name: jeweler
-  version_requirements: &id003 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -65,17 +50,17 @@ dependencies:
         - 4
         - 0
         version: 1.4.0
-  requirement: *id003
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
   name: rspec
-  version_requirements: &id004 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 62196431
+        hash: 3192703827556712861
         segments: 
         - 2
         - 0
@@ -83,12 +68,12 @@ dependencies:
         - beta
         - 22
         version: 2.0.0.beta.22
-  requirement: *id004
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
   name: fakeweb
-  version_requirements: &id005 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -99,12 +84,12 @@ dependencies:
         - 3
         - 0
         version: 1.3.0
-  requirement: *id005
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
   name: rails
-  version_requirements: &id006 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -115,12 +100,12 @@ dependencies:
         - 0
         - 0
         version: 3.0.0
-  requirement: *id006
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id005
+- !ruby/object:Gem::Dependency 
   name: bundler
-  version_requirements: &id007 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -131,7 +116,8 @@ dependencies:
         - 0
         - 0
         version: 1.0.0
-  requirement: *id007
+  type: :development
+  version_requirements: *id006
 description: A really simple currency converter using XavierMedia API. It's Ruby 1.8, 1.9 and JRuby compatible, and it also takes advantage of Rails cache when available.
 email: info@codegram.com
 executables: []
@@ -154,7 +140,8 @@ files:
 - Rakefile
 - VERSION
 - autotest/discover.rb
-- lib/core_ext/numeric.rb
+- lib/core_ext/fixnum.rb
+- lib/core_ext/float.rb
 - lib/simple_currency.rb
 - lib/simple_currency/currency_convertible.rb
 - simple_currency.gemspec
@@ -166,8 +153,8 @@ homepage: http://github.com/codegram/simple_currency
 licenses: []
 
 post_install_message: 
-rdoc_options: []
-
+rdoc_options: 
+- --charset=UTF-8
 require_paths: 
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement