killbill-litle 1.9.6 → 1.9.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3f01741442afd85b5bc17c183a3f0390b8b8dc03
-  data.tar.gz: 0a2b5c3e9a1f4c28bd85092fee19fb196ac9c643
+  metadata.gz: bc0d66bea92052455c27f5e608fdca9249f0d435
+  data.tar.gz: d1c626601553b6e6d05981a954245b09dd7c1bab
 SHA512:
-  metadata.gz: 8af245e60186fc8650919daf8f4dac1afad300fe34bdb23814d883e48150d4b0280f119eef4f1a879c0b7e214803846d30222cb6e176eb3a5c6244c2d1e60f1a
-  data.tar.gz: 0f87a847d67b7954d9ef553677db6247d62ed32760b78ac53dff6a2acdb9f370352ce7c2b83b112b348edf0a25f2185cddd8b6b86170b197bc8fb9efddfe6278
+  metadata.gz: 93529b31d8598992e7daac6228d4afe7ef1b7500f622f299ff1e5fc071817893aa12db4258f24df78c5286d6192fdd039a5311b779c676ce047aa652c27e2e8e
+  data.tar.gz: 3f9261b41c28ef0987d480c46a07452a14f1955bd336d080ac1fd98aee1a2f4d8a080d386aef6e66e6d5daebcdcbce4ec2c0773baaf545d5bafcc9662908db53

data/Jarfile

@@ -1,6 +1,6 @@
-jar 'com.ning.billing:killbill-api', '0.7.8'
-jar 'com.ning.billing.plugin:killbill-plugin-api-notification', '0.6.2'
-jar 'com.ning.billing.plugin:killbill-plugin-api-payment', '0.6.2'
-jar 'com.ning.billing.plugin:killbill-plugin-api-currency', '0.6.2'
-jar 'com.ning.billing:killbill-util:tests', '0.7.1'
+jar 'com.ning.billing:killbill-api', '0.8.0'
+jar 'com.ning.billing.plugin:killbill-plugin-api-notification', '0.6.3'
+jar 'com.ning.billing.plugin:killbill-plugin-api-payment', '0.6.3'
+jar 'com.ning.billing.plugin:killbill-plugin-api-currency', '0.6.3'
+jar 'com.ning.billing:killbill-util:tests', '0.8.8'
 jar 'javax.servlet:javax.servlet-api', '3.0.1'

data/NEWS

@@ -1,3 +1,11 @@
+1.9.7
+    Implement payments and refunds search
+    Fix amount to cents conversion
+    Update to latest killbill (2.0.0)
+
+1.9.6
+    Update to latest killbill (1.9.0)
+
 1.9.5
     Bug fixes for currency transaformation
 

data/README.md

@@ -6,6 +6,8 @@ killbill-litle-plugin
 
 Plugin to use Litle & Co. as a gateway.
 
+Release builds are available on [Maven Central](http://search.maven.org/#search%7Cga%7C1%7Cg%3A%22com.ning.killbill.ruby%22%20AND%20a%3A%22litle-plugin%22) with coordinates `com.ning.killbill.ruby:litle-plugin`.
+
 Requirements
 ------------
 

data/VERSION

@@ -1 +1 @@
-1.9.6
+1.9.7

data/killbill-litle.gemspec

@@ -22,9 +22,10 @@ Gem::Specification.new do |s|
 
   s.rdoc_options << '--exclude' << '.'
 
-  s.add_dependency 'killbill', '~> 1.9.0'
+  s.add_dependency 'killbill', '~> 2.0.0'
   s.add_dependency 'activemerchant', '~> 1.36.0'
   s.add_dependency 'activerecord', '~> 3.2.1'
+  s.add_dependency 'money', '~> 6.0.0'
   s.add_dependency 'sinatra', '~> 1.3.4'
   # LitleOnline gem dependencies
   s.add_dependency 'LitleOnline', '~> 8.16.0'

data/lib/litle.rb

@@ -1,6 +1,7 @@
 require 'active_record'
 require 'activemerchant'
 require 'bigdecimal'
+require 'money'
 require 'pathname'
 require 'sinatra'
 require 'singleton'

data/lib/litle/api.rb

@@ -14,7 +14,8 @@ module Killbill::Litle
     end
 
     def process_payment(kb_account_id, kb_payment_id, kb_payment_method_id, amount, currency, call_context = nil, options = {})
-      amount_in_cents = (amount * 100).to_i
+      # Use Money to compute the amount in cents, as it depends on the currency (1 cent of BTC is 1 Satoshi, not 0.01 BTC)
+      amount_in_cents = Money.new_with_amount(amount, currency).cents.to_i
 
       # If the payment was already made, just return the status
       # TODO Should we set the Litle Id field to check for dups (https://www.litle.com/mc-secure/DupeChecking_V1.2.pdf)?
@@ -55,7 +56,8 @@ module Killbill::Litle
     end
 
     def process_refund(kb_account_id, kb_payment_id, amount, currency, call_context = nil, options = {})
-      amount_in_cents = (amount * 100).to_i
+      # Use Money to compute the amount in cents, as it depends on the currency (1 cent of BTC is 1 Satoshi, not 0.01 BTC)
+      amount_in_cents = Money.new_with_amount(amount, currency).cents.to_i
 
       # Check for currency conversion
       actual_amount, actual_currency = convert_amount_currency_if_required(amount_in_cents, currency, kb_payment_id)
@@ -75,9 +77,9 @@ module Killbill::Litle
     def get_refund_info(kb_account_id, kb_payment_id, tenant_context = nil, options = {})
       # We assume the refund is immutable in Litle and only look at our tables since there
       # doesn't seem to be a Litle API to fetch details for a given transaction.
-      litle_transaction = LitleTransaction.refund_from_kb_payment_id(kb_payment_id)
+      litle_transactions = LitleTransaction.refunds_from_kb_payment_id(kb_payment_id)
 
-      litle_transaction.litle_response.to_refund_response
+      litle_transactions.map { |t| t.litle_response.to_refund_response }
     end
 
     def add_payment_method(kb_account_id, kb_payment_method_id, payment_method_props, set_default, call_context = nil, options = {})
@@ -165,6 +167,14 @@ module Killbill::Litle
       end
     end
 
+    def search_payments(search_key, offset = 0, limit = 100, call_context = nil, options = {})
+      LitleResponse.search(search_key, offset, limit, :payment)
+    end
+
+    def search_refunds(search_key, offset = 0, limit = 100, call_context = nil, options = {})
+      LitleResponse.search(search_key, offset, limit, :refund)
+    end
+
     def search_payment_methods(search_key, offset = 0, limit = 100, call_context = nil, options = {})
       LitlePaymentMethod.search(search_key, offset, limit)
     end

data/lib/litle/models/litle_response.rb

@@ -84,6 +84,52 @@ module Killbill::Litle
       to_killbill_response :refund
     end
 
+    # VisibleForTesting
+    def self.search_query(api_call, search_key, offset = nil, limit = nil)
+      t = self.arel_table
+
+      # Exact matches only
+      where_clause =     t[:params_litleonelineresponse_saleresponse_id].eq(search_key)
+                     .or(t[:params_litleonelineresponse_saleresponse_litle_txn_id].eq(search_key))
+                     .or(t[:params_litleonelineresponse_saleresponse_order_id].eq(search_key))
+                     .or(t[:params_litleonelineresponse_saleresponse_auth_code].eq(search_key))
+
+      # Only search successful payments and refunds
+      where_clause = where_clause.and(t[:api_call].eq(api_call))
+                                 .and(t[:success].eq(true))
+
+      query = t.where(where_clause)
+               .order(t[:id])
+
+      if offset.blank? and limit.blank?
+        # true is for count distinct
+        query.project(t[:id].count(true))
+      else
+        query.skip(offset) unless offset.blank?
+        query.take(limit) unless limit.blank?
+        query.project(t[Arel.star])
+        # Not chainable
+        query.distinct
+      end
+      query
+    end
+
+    def self.search(search_key, offset = 0, limit = 100, type = :payment)
+      api_call = type == :payment ? 'charge' : 'refund'
+      pagination = Killbill::Plugin::Model::Pagination.new
+      pagination.current_offset = offset
+      pagination.total_nb_records = self.count_by_sql(self.search_query(api_call, search_key))
+      pagination.max_nb_records = self.where(:api_call => api_call, :success => true).count
+      pagination.next_offset = (!pagination.total_nb_records.nil? && offset + limit >= pagination.total_nb_records) ? nil : offset + limit
+      # Reduce the limit if the specified value is larger than the number of records
+      actual_limit = [pagination.max_nb_records, limit].min
+      pagination.iterator = StreamyResultSet.new(actual_limit) do |offset,limit|
+        self.find_by_sql(self.search_query(api_call, search_key, offset, limit))
+            .map { |x| type == :payment ? x.to_payment_response : x.to_refund_response }
+      end
+      pagination
+    end
+
     private
 
     def to_killbill_response(type)
@@ -97,8 +143,8 @@ module Killbill::Litle
         amount_in_cents = litle_transaction.amount_in_cents
         currency = litle_transaction.currency
         created_date = litle_transaction.created_at
-        first_payment_reference_id = params_litleonelineresponse_saleresponse_id
-        second_payment_reference_id = litle_transaction.litle_txn_id
+        first_reference_id = params_litleonelineresponse_saleresponse_id
+        second_reference_id = litle_transaction.litle_txn_id
       end
 
       effective_date = params_litleonelineresponse_saleresponse_response_time || created_date
@@ -107,26 +153,29 @@ module Killbill::Litle
 
       if type == :payment
         p_info_plugin = Killbill::Plugin::Model::PaymentInfoPlugin.new
-        p_info_plugin.amount = BigDecimal.new(amount_in_cents.to_s) / 100.0 if amount_in_cents
+        p_info_plugin.kb_payment_id = kb_payment_id
+        p_info_plugin.amount = Money.new(amount_in_cents, currency).to_d if currency
         p_info_plugin.currency = currency
         p_info_plugin.created_date = created_date
         p_info_plugin.effective_date = effective_date
         p_info_plugin.status = (success ? :PROCESSED : :ERROR)
         p_info_plugin.gateway_error = gateway_error
         p_info_plugin.gateway_error_code = gateway_error_code
-        p_info_plugin.first_payment_reference_id = first_payment_reference_id
-        p_info_plugin.second_payment_reference_id = second_payment_reference_id
+        p_info_plugin.first_payment_reference_id = first_reference_id
+        p_info_plugin.second_payment_reference_id = second_reference_id
         p_info_plugin
       else
         r_info_plugin = Killbill::Plugin::Model::RefundInfoPlugin.new
-        r_info_plugin.amount = BigDecimal.new(amount_in_cents.to_s) / 100.0 if amount_in_cents
+        r_info_plugin.kb_payment_id = kb_payment_id
+        r_info_plugin.amount = Money.new(amount_in_cents, currency).to_d if currency
         r_info_plugin.currency = currency
         r_info_plugin.created_date = created_date
         r_info_plugin.effective_date = effective_date
         r_info_plugin.status = (success ? :PROCESSED : :ERROR)
         r_info_plugin.gateway_error = gateway_error
         r_info_plugin.gateway_error_code = gateway_error_code
-        r_info_plugin.reference_id = first_payment_reference_id
+        r_info_plugin.first_refund_reference_id = first_reference_id
+        r_info_plugin.second_refund_reference_id = second_reference_id
         r_info_plugin
       end
     end

data/lib/litle/models/litle_transaction.rb

@@ -4,18 +4,11 @@ module Killbill::Litle
     attr_accessible :amount_in_cents, :currency, :api_call, :kb_payment_id, :litle_txn_id
 
     def self.from_kb_payment_id(kb_payment_id)
-      single_transaction_from_kb_payment_id :charge, kb_payment_id
+      transaction_from_kb_payment_id :charge, kb_payment_id, :single
     end
 
-    def self.refund_from_kb_payment_id(kb_payment_id)
-      single_transaction_from_kb_payment_id :refund, kb_payment_id
-    end
-
-    def self.single_transaction_from_kb_payment_id(api_call, kb_payment_id)
-      litle_transactions = find_all_by_api_call_and_kb_payment_id(api_call, kb_payment_id)
-      raise "Unable to find Litle transaction id for payment #{kb_payment_id}" if litle_transactions.empty?
-      raise "Killbill payment mapping to multiple Litle transactions for payment #{kb_payment_id}" if litle_transactions.size > 1
-      litle_transactions[0]
+    def self.refunds_from_kb_payment_id(kb_payment_id)
+      transaction_from_kb_payment_id :refund, kb_payment_id, :multiple
     end
 
     def self.find_candidate_transaction_for_refund(kb_payment_id, amount_in_cents)
@@ -34,5 +27,18 @@ module Killbill::Litle
 
       litle_transactions.first
     end
+
+    private
+
+    def self.transaction_from_kb_payment_id(api_call, kb_payment_id, how_many)
+      litle_transactions = find_all_by_api_call_and_kb_payment_id(api_call, kb_payment_id)
+      raise "Unable to find Litle transaction id for payment #{kb_payment_id}" if litle_transactions.empty?
+      if how_many == :single
+        raise "Killbill payment mapping to multiple Litle transactions for payment #{kb_payment_id}" if litle_transactions.size > 1
+        litle_transactions[0]
+      else
+        litle_transactions
+      end
+    end
   end
 end

data/pom.xml

@@ -25,7 +25,7 @@
     <groupId>com.ning.killbill.ruby</groupId>
     <artifactId>litle-plugin</artifactId>
     <packaging>pom</packaging>
-    <version>1.9.6</version>
+    <version>1.9.7</version>
     <name>litle-plugin</name>
     <url>http://github.com/killbill/killbill-litle-plugin</url>
     <description>Plugin for accessing Litle as a payment gateway</description>

data/spec/litle/litle_response_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+describe Killbill::Litle::LitleResponse do
+  before :all do
+    Killbill::Litle::LitleResponse.delete_all
+  end
+
+  it 'should generate the right SQL query' do
+    # Check count query (search query numeric)
+    expected_query = "SELECT COUNT(DISTINCT \"litle_responses\".\"id\") FROM \"litle_responses\"  WHERE (((\"litle_responses\".\"params_litleonelineresponse_saleresponse_id\" = '1234' OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_litle_txn_id\" = '1234') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_order_id\" = '1234') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_auth_code\" = '1234') AND \"litle_responses\".\"api_call\" = 'charge' AND \"litle_responses\".\"success\" = 't' ORDER BY \"litle_responses\".\"id\""
+    # Note that Kill Bill will pass a String, even for numeric types
+    Killbill::Litle::LitleResponse.search_query('charge', '1234').to_sql.should == expected_query
+
+    # Check query with results (search query numeric)
+    expected_query = "SELECT  DISTINCT \"litle_responses\".* FROM \"litle_responses\"  WHERE (((\"litle_responses\".\"params_litleonelineresponse_saleresponse_id\" = '1234' OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_litle_txn_id\" = '1234') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_order_id\" = '1234') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_auth_code\" = '1234') AND \"litle_responses\".\"api_call\" = 'charge' AND \"litle_responses\".\"success\" = 't' ORDER BY \"litle_responses\".\"id\" LIMIT 10 OFFSET 0"
+    # Note that Kill Bill will pass a String, even for numeric types
+    Killbill::Litle::LitleResponse.search_query('charge', '1234', 0, 10).to_sql.should == expected_query
+
+    # Check count query (search query string)
+    expected_query = "SELECT COUNT(DISTINCT \"litle_responses\".\"id\") FROM \"litle_responses\"  WHERE (((\"litle_responses\".\"params_litleonelineresponse_saleresponse_id\" = 'XXX' OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_litle_txn_id\" = 'XXX') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_order_id\" = 'XXX') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_auth_code\" = 'XXX') AND \"litle_responses\".\"api_call\" = 'charge' AND \"litle_responses\".\"success\" = 't' ORDER BY \"litle_responses\".\"id\""
+    Killbill::Litle::LitleResponse.search_query('charge', 'XXX').to_sql.should == expected_query
+
+    # Check query with results (search query string)
+    expected_query = "SELECT  DISTINCT \"litle_responses\".* FROM \"litle_responses\"  WHERE (((\"litle_responses\".\"params_litleonelineresponse_saleresponse_id\" = 'XXX' OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_litle_txn_id\" = 'XXX') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_order_id\" = 'XXX') OR \"litle_responses\".\"params_litleonelineresponse_saleresponse_auth_code\" = 'XXX') AND \"litle_responses\".\"api_call\" = 'charge' AND \"litle_responses\".\"success\" = 't' ORDER BY \"litle_responses\".\"id\" LIMIT 10 OFFSET 0"
+    Killbill::Litle::LitleResponse.search_query('charge', 'XXX', 0, 10).to_sql.should == expected_query
+  end
+
+  it 'should search all fields' do
+    do_search('foo').size.should == 0
+
+    pm = Killbill::Litle::LitleResponse.create :api_call => 'charge',
+                                               :kb_payment_id => '11-22-33-44',
+                                               :params_litleonelineresponse_saleresponse_id => '55-66-77-88',
+                                               :params_litleonelineresponse_saleresponse_litle_txn_id => 38102343,
+                                               :params_litleonelineresponse_saleresponse_order_id => 'order-id-1',
+                                               :params_litleonelineresponse_saleresponse_auth_code => 'auth-code-1',
+                                               :success => true
+
+    # Wrong api_call
+    ignored1 = Killbill::Litle::LitleResponse.create :api_call => 'add_payment_method',
+                                                     :kb_payment_id => pm.kb_payment_id,
+                                                     :params_litleonelineresponse_saleresponse_id => pm.params_litleonelineresponse_saleresponse_id,
+                                                     :params_litleonelineresponse_saleresponse_litle_txn_id => pm.params_litleonelineresponse_saleresponse_litle_txn_id,
+                                                     :params_litleonelineresponse_saleresponse_order_id => pm.params_litleonelineresponse_saleresponse_order_id,
+                                                     :params_litleonelineresponse_saleresponse_auth_code => pm.params_litleonelineresponse_saleresponse_auth_code,
+                                                     :success => true
+
+    # Not successful
+    ignored2 = Killbill::Litle::LitleResponse.create :api_call => 'charge',
+                                                     :kb_payment_id => pm.kb_payment_id,
+                                                     :params_litleonelineresponse_saleresponse_id => pm.params_litleonelineresponse_saleresponse_id,
+                                                     :params_litleonelineresponse_saleresponse_litle_txn_id => pm.params_litleonelineresponse_saleresponse_litle_txn_id,
+                                                     :params_litleonelineresponse_saleresponse_order_id => pm.params_litleonelineresponse_saleresponse_order_id,
+                                                     :params_litleonelineresponse_saleresponse_auth_code => pm.params_litleonelineresponse_saleresponse_auth_code,
+                                                     :success => false
+
+    do_search('foo').size.should == 0
+    do_search(pm.params_litleonelineresponse_saleresponse_id).size.should == 1
+    do_search(pm.params_litleonelineresponse_saleresponse_litle_txn_id).size.should == 1
+    do_search(pm.params_litleonelineresponse_saleresponse_order_id).size.should == 1
+    do_search(pm.params_litleonelineresponse_saleresponse_auth_code).size.should == 1
+
+    pm2 = Killbill::Litle::LitleResponse.create :api_call => 'charge',
+                                                :kb_payment_id => '11-22-33-44',
+                                                :params_litleonelineresponse_saleresponse_id => '11-22-33-44',
+                                                :params_litleonelineresponse_saleresponse_litle_txn_id => pm.params_litleonelineresponse_saleresponse_litle_txn_id,
+                                                :params_litleonelineresponse_saleresponse_order_id => 'order-id-2',
+                                                :params_litleonelineresponse_saleresponse_auth_code => 'auth-code-2',
+                                                :success => true
+
+    do_search('foo').size.should == 0
+    do_search(pm.params_litleonelineresponse_saleresponse_id).size.should == 1
+    do_search(pm.params_litleonelineresponse_saleresponse_litle_txn_id).size.should == 2
+    do_search(pm.params_litleonelineresponse_saleresponse_order_id).size.should == 1
+    do_search(pm.params_litleonelineresponse_saleresponse_auth_code).size.should == 1
+    do_search(pm2.params_litleonelineresponse_saleresponse_id).size.should == 1
+    do_search(pm2.params_litleonelineresponse_saleresponse_litle_txn_id).size.should == 2
+    do_search(pm2.params_litleonelineresponse_saleresponse_order_id).size.should == 1
+    do_search(pm2.params_litleonelineresponse_saleresponse_auth_code).size.should == 1
+  end
+
+  private
+
+  def do_search(search_key)
+    pagination = Killbill::Litle::LitleResponse.search(search_key)
+    pagination.current_offset.should == 0
+    results = pagination.iterator.to_a
+    pagination.total_nb_records.should == results.size
+    results
+  end
+end

data/spec/litle/remote/integration_spec.rb

@@ -92,10 +92,11 @@ describe Killbill::Litle::PaymentPlugin do
     response.success.should be_true
 
     # Check we can retrieve the refund
-    refund_response = @plugin.get_refund_info pm.kb_account_id, kb_payment_id
+    refund_responses = @plugin.get_refund_info pm.kb_account_id, kb_payment_id
+    refund_responses.size.should == 1
     # Apparently, Litle returns positive amounts for refunds
-    refund_response.amount.should == amount
-    refund_response.status.should == :PROCESSED
+    refund_responses[0].amount.should == amount
+    refund_responses[0].status.should == :PROCESSED
 
     # Make sure we can charge again the same payment method
     second_amount = BigDecimal.new("294.71")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: killbill-litle
 version: !ruby/object:Gem::Version
-  version: 1.9.6
+  version: 1.9.7
 platform: ruby
 authors:
 - Kill Bill core team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2014-01-03 00:00:00.000000000 Z
+date: 2014-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: killbill
@@ -16,12 +16,12 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.9.0
+        version: 2.0.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.9.0
+        version: 2.0.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
@@ -52,6 +52,20 @@ dependencies:
         version: 3.2.1
   prerelease: false
   type: :runtime
+- !ruby/object:Gem::Dependency
+  name: money
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 6.0.0
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 6.0.0
+  prerelease: false
+  type: :runtime
 - !ruby/object:Gem::Dependency
   name: sinatra
   version_requirements: !ruby/object:Gem::Requirement
@@ -215,6 +229,7 @@ files:
 - spec/litle/base_plugin_spec.rb
 - spec/litle/currency_conversion_spec.rb
 - spec/litle/litle_payment_method_spec.rb
+- spec/litle/litle_response_spec.rb
 - spec/litle/remote/integration_spec.rb
 - spec/litle/utils_spec.rb
 - spec/spec_helper.rb
@@ -248,6 +263,7 @@ test_files:
 - spec/litle/base_plugin_spec.rb
 - spec/litle/currency_conversion_spec.rb
 - spec/litle/litle_payment_method_spec.rb
+- spec/litle/litle_response_spec.rb
 - spec/litle/remote/integration_spec.rb
 - spec/litle/utils_spec.rb
 - spec/spec_helper.rb