stripe_invoice 1.1.2 → 1.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f452ed8feae4a4675887cdee229dbb76d45d332e
-  data.tar.gz: 8b09ef6e35a8de238f4fe1b5ab6385c4d201617c
+  metadata.gz: c305da2ef2b69b073b84bf050d0fc793494bae43
+  data.tar.gz: 6ff3e0fe43ecd54d2c19b13bf45f94c812dc3b6c
 SHA512:
-  metadata.gz: e3c5985e3709e544f3cbbc49298d2597f7fcb419699d127eb23dc607109f56d17557f1f0c20e8d2add78610fae451d5bfd048e00ceb0f5069a23b3ac6dd430c5
-  data.tar.gz: 0c0bf0241fe39f4d87b2557dbb35f3388606e9e69aa18c74709d5d2be44124d809cec375e2123efb6b0ab61029c0a140aec60341a1d18dfa6b27fd58a2aebb3b
+  metadata.gz: 548fff04f67664a4c25e578a4272d25c56406aadffe69bcaa362129c77085bd603e0576ddf09e21d7bd1a293f6d342e1df144bc7535432bf910ff7527f4c3adb
+  data.tar.gz: 3e83e38bb47783b44dc62c9982f742e7d8c07a4022508244e9bb8f7c589eeb5a67e97bbff167717944026b1a52ebda6d61e9bd094b953b539a7b34ada23fe1d3

data/app/models/stripe_invoice/charge.rb

@@ -46,14 +46,14 @@ module StripeInvoice
     def self.create_from_stripe(stripe_charge)
       
       unless stripe_charge.paid
-        puts "ignoring unpaid charges"
-        return
+        return puts "[#{self.class.name}##{__method__.to_s}] ignoring unpaid charge #{stripe_charge.id}"
       end
        
       charge = Charge.find_by_stripe_id(stripe_charge[:id])
       
       # for existing invoices just update and be done
       if charge.present?
+        puts "[#{self.class.name}##{__method__.to_s}] updating data for #{stripe_charge.id}"
         charge.update_attribute(:json, stripe_charge)
         return charge 
       end
@@ -93,18 +93,19 @@ module StripeInvoice
       
       # we found them directly, go for it. 
       unless subscription.nil?
+        puts "[#{self.class.name}##{__method__.to_s}] found owner directly for #{stripe_charge.id} - #{subscription.subscription_owner_email}"
         return subscription.subscription_owner
       end 
       
       # koudoku does have a nasty feature/bug in that it deletes the subscription
       # from the database when it is cancelled. This makes it impossible to 
       # match past charges to customers based solely on the subscription's stripe_id
-      # instead we also try to match the email address/owner_id that was send to stripe 
+      # instead we also try to match the email address that was send to stripe 
       # when the account was created
       stripe_customer = Stripe::Customer.retrieve stripe_charge.customer
       return nil if stripe_customer[:deleted] # yes, that can happen :-(
-      owner_id = stripe_customer.description.to_i
-      Koudoku.owner_class.find(owner_id)
+      puts "[#{self.class.name}##{__method__.to_s}] found owner via email for #{stripe_charge.id} - #{stripe_customer.email}"
+      Koudoku.owner_class.try(:find_by_email, stripe_customer.email)
       
     end
   end

data/lib/stripe_invoice/version.rb

@@ -1,3 +1,3 @@
 module StripeInvoice
-  VERSION = "1.1.2"
+  VERSION = "1.1.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stripe_invoice
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.1.3
 platform: ruby
 authors:
 - Christoph Engelhardt
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-13 00:00:00.000000000 Z
+date: 2015-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails