wepay-rails 0.1.82 → 0.1.83

data/README.rdoc

@@ -17,27 +17,62 @@ Your model:
 
 Adding wepayable to your model also adds some helpful methods to your model, like save_<your column name>
 
-You will have to set up a few new controllers to handle callbacks from wepay.
+You need to also create a new model called WepayCheckoutRecord. In future versions, I'll include a generator - but for now, here's the migration to use:
+
+    class CreateWepayCheckoutRecords < ActiveRecord::Migration
+        def self.up
+
+            create_table :wepay_checkout_records do |t|
+                t.integer :checkout_id
+                t.integer :account_id
+                t.string :auth_code
+                t.string :state
+                t.string :short_description
+                t.text :long_description
+                t.string :currency
+                t.decimal :amount
+                t.decimal :app_fee
+                t.string :fee_payer
+                t.decimal :gross
+                t.decimal :fee
+                t.string :reference_id
+                t.text :redirect_uri
+                t.text :callback_uri
+                t.text :checkout_uri
+                t.string :payer_email
+                t.string :payer_name
+                t.text :cancel_reason
+                t.text :refund_reason
+                t.boolean :auto_capture
+                t.boolean :require_shipping
+                t.text :shipping_address
+                t.decimal :tax
+
+                t.timestamps
+            end
+
+            add_index :wepay_checkout_records, :checkout_id
+        end
+
+        def self.down
+            drop_table :wepay_checkout_records
+        end
+    end
+
+
+You will have to set up a few new controllers to handle callbacks and redirects from wepay back to your app.
 One is for handling the Oauth handshake. I called mine authorize_controller.rb
-Another is for handling IPN (Instant Payment Notifications) - for me, it's ipn_controller.rb
 
 I created a directory under my controllers directory called wepay so I can keep my wepay callback controllers in one place.
   app
     |_ controllers
                  |_ wepay
-                        |_ application_controller.rb
                         |_ authorize_controller.rb
-                        |_ ipn_controller.rb
                  |_ purchase
                         |_ checkout_controller.rb
                         |_ finalize_controller.rb
 
-    class Wepay::ApplicationController < ApplicationController
-        include WepayRails::Payments
-    end
-
     class Wepay::AuthorizeController < Wepay::ApplicationController
-        before_filter :authenticate_account! # I use devise
         def index
             current_profile.save_wepay_auth_code params[:code]
             initialize_wepay_access_token(params[:code])
@@ -46,6 +81,10 @@ I created a directory under my controllers directory called wepay so I can keep
         end
     end
 
+The wepay-rails gem comes with an IpnController already built in for
+handling requests from wepay about a specific checkout. If you wish to override it,
+you can create an IpnController in your rails app.
+
     class IpnController < Wepay::ApplicationController
         def index
             #Do something with the instant payment notifications back from Wepay
@@ -55,7 +94,6 @@ I created a directory under my controllers directory called wepay so I can keep
 Routes for these:
     namespace :wepay do
         resources :authorize, :only => [:index]
-        resources :ipn, :only => [:index]
     end
 
 wepay.yml will also need these directives. See the section on wepay.yml
@@ -73,13 +111,13 @@ Finally, your checkout controller (or some controller that will interact with th
                 if wepay_access_token_exists? # Temporary Oauth Access token from wepay
 
                     cart = current_account.cart # EXAMPLE - get my shopping cart
-                    tx_id = cart.init_transaction # EXAMPLE - I use a transaction table to capture attempts to checkout
+                    tx_id = cart.transaction_id # EXAMPLE - I use a column in my cart to have a way to look up the cart upon the user's return from wepay
 
                     checkout_params = {
                         :amount => cart.grand_total,
                         :short_description => cart.short_description,
                         :long_description => cart.long_description,
-                        :callback_uri => purchase_finalize_index_url(:txID => tx_id) # Wepay redirects the user back to this url after purchase
+                        :redirect_uri => purchase_finalize_index_url(:txID => tx_id) # Wepay redirects the user back to this url after purchase
                     }
 
                     init_checkout_and_send_user_to_wepay(checkout_params) # Send the customer to wepay to finish payment
@@ -112,39 +150,57 @@ If not, we can initialize the access token. If it is there, go ahead and make an
 
 Configuration is done through config/wepay.yml:
     production:
-        client_id: <your wepay client id>
-        client_secret: <your wepay client secret>
-        authorize_redirect_uri: "http://prod.example.com/wepay/authorize"
-        scope: ['refund_payments','collect_payments','view_balance','view_user']
-        wepay_api_uri: "https://api.wepay.com"
-        wepay_api_version: "v2"
-        ipn_callback_uri: "http://prod.example.com/wepay/ipn"
-        checkout_redirect_uri: "http://prod.example.com/purchase/finalize"
-        fee_payer: Payee
-        checkout_type: GOODS
-        charge_tax: false
-        app_fee: 0
-        auto_capture: true
-        require_shipping: false
-        shipping_fee: 0
-        charge_tax: false
+      client_id: <your client_id from wepay>
+      client_secret: <your client_secret from wepay>
+      authorize_redirect_uri: "http://www.example.com/wepay/authorize"
+      scope: ['refund_payments','collect_payments','view_balance','view_user']
+      #wepay_api_uri: "https://api.wepay.com"
+      wepay_api_uri: "https://stage.wepay.com"
+      wepay_api_version: "v2"
+      ipn_callback_uri: "http://www.example.com/wepay/ipn"
+      checkout_redirect_uri: "http://www.example.com/purchase/finalize"
+      fee_payer: Payee
+      checkout_type: GOODS
+      charge_tax: false
+      app_fee: 0
+      auto_capture: true
+      require_shipping: false
+      shipping_fee: 0
+      charge_tax: false
     development:
-        client_id: <your dev client id>
-        client_secret: <your dev client secret>
-        redirect_uri: "http://dev.example.com/wepay/authorize"
-        scope: ['refund_payments','collect_payments','view_balance','view_user']
-        wepay_api_uri: "https://stage.wepay.com"
-        wepay_api_version: "v2"
-        ipn_callback_uri: "http://dev.example.com/wepay/ipn"
-        checkout_redirect_uri: "http://dev.example.com/purchase/finalize"
-        fee_payer: Payee
-        checkout_type: GOODS
-        charge_tax: false
-        app_fee: 0
-        require_shipping: false
-        shipping_fee: 0
-        charge_tax: false
-        auto_capture: true
+      client_id: <your client_id from wepay>
+      client_secret: <your client_secret from wepay>
+      redirect_uri: "http://dev.example.com/wepay/authorize"
+      scope: ['refund_payments','collect_payments','view_balance','view_user']
+      wepay_api_uri: "https://stage.wepay.com"
+      wepay_api_version: "v2"
+      ipn_callback_uri: "http://dev.example.com/wepay/ipn"
+      checkout_redirect_uri: "http://dev.example.com/purchase/finalize"
+      fee_payer: Payee
+      checkout_type: GOODS
+      charge_tax: false
+      app_fee: 0
+      require_shipping: false
+      shipping_fee: 0
+      charge_tax: false
+      auto_capture: true
+    test:
+      client_id: <your client_id from wepay>
+      client_secret: <your client_secret from wepay>
+      redirect_uri: "http://dev.example.com/wepay/authorize"
+      scope: ['refund_payments','collect_payments','view_balance','view_user']
+      wepay_api_uri: "https://stage.wepay.com"
+      wepay_api_version: "v2"
+      ipn_callback_uri: "http://test.example.com/wepay/ipn"
+      checkout_redirect_uri: "http://dev.example.com/purchase/finalize"
+      fee_payer: Payee
+      checkout_type: GOODS
+      charge_tax: false
+      app_fee: 0
+      auto_capture: true
+      charge_tax: false
+      require_shipping: false
+      shipping_fee: 0
 
 
 == Contributing to wepay-rails

data/VERSION

@@ -1 +1 @@
-0.1.82
+0.1.83

data/app/controllers/wepay/ipn_controller.rb

@@ -1,21 +1,15 @@
 class Wepay::IpnController < Wepay::ApplicationController
   def index
 
-    log = File.open('/tmp/ipn.log','a')
-
     record = WepayCheckoutRecord.find_by_checkout_id(params[:checkout_id])
 
-    log.puts record.inspect
-
     if record.present?
       wepay_gateway.access_token(record.auth_code)
       checkout = wepay_gateway.lookup_checkout(record.checkout_id)
-      log.puts checkout.inspect
       record.update_attributes(checkout)
+      render :text => 'ok'
     else
       raise StandardError.new("Wepay IPN: No record found for checkout_id #{params[:checkout_id]}")
     end
-
-    render :text => 'ok'
   end
 end

data/lib/helpers/controller_helpers.rb

@@ -87,7 +87,6 @@ module WepayRails
       # :charge_tax	No	A boolean value (0 or 1). If set to 1 and the account has a relevant tax entry (see /account/set_tax), then tax will be charged.
       def init_checkout_and_send_user_to_wepay(parms)
         response = wepay_gateway.perform_checkout(parms)
-        File.open('/tmp/noisebytes.log','a') {|f|f.write(response.inspect)}
         raise WepayRails::Exceptions::InitializeCheckoutError.new("A problem occurred while trying to checkout. Wepay didn't send us back a checkout uri") unless response && response.has_key?('checkout_uri')
         redirect_to response['checkout_uri'] and return
       end

data/lib/wepay-rails.rb

@@ -30,9 +30,7 @@ module WepayRails
         @wepay_access_token = args.first
 
         yml = Rails.root.join('config', 'wepay.yml').to_s
-	File.open('/tmp/wepay.log','a') {|f| f.write("Yaml for wepay is found at #{yml}")}
         @config = YAML.load_file(yml)[Rails.env].symbolize_keys
-	File.open('/tmp/wepay.log','a') {|f| f.write("config is #{@config.inspect}")}
 
         @scope = @config.delete(:scope)
 
@@ -97,14 +95,10 @@ module WepayRails
       # retrieved from the first call.
       def wepay_user
         user_api = lambda {|headers|
-          File.open('/tmp/noisebytes.log','a') {|f|f.write("Base uri inside lambda is #{@base_uri}")}
-          File.open('/tmp/noisebytes.log','a') {|f|f.write("Headers inside lambda is #{headers}")}
           response = self.class.get("#{@base_uri}/user", {:headers => headers})
           JSON.parse(response.body)
         }
 
-        File.open('/tmp/noisebytes.log','a'){|f|f.write("User is #{user_api.call(wepay_auth_header)}")}
-
         @wepay_user ||= user_api.call(wepay_auth_header)
       end
 
@@ -157,9 +151,6 @@ module WepayRails
             :account_id       => @config[:account_id]
         }.merge(parms)
 
-        File.open('/tmp/noisebytes.log','a') {|f| f.write({:headers => wepay_auth_header}.merge!(defaults).inspect)}
-
-
         response = self.class.get("#{@base_uri}/checkout/create", {:headers => wepay_auth_header}.merge!(:query => defaults))
         JSON.parse(response.body)
       end

data/wepay-rails.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{wepay-rails}
-  s.version = "0.1.82"
+  s.version = "0.1.83"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = [%q{Adam Medeiros}]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wepay-rails
 version: !ruby/object:Gem::Version
-  version: 0.1.82
+  version: 0.1.83
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-09-04 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
-  requirement: &19124380 !ruby/object:Gem::Requirement
+  requirement: &19031220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *19124380
+  version_requirements: *19031220
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &19123780 !ruby/object:Gem::Requirement
+  requirement: &19030640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *19123780
+  version_requirements: *19030640
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &19123180 !ruby/object:Gem::Requirement
+  requirement: &19030120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *19123180
+  version_requirements: *19030120
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &19122580 !ruby/object:Gem::Requirement
+  requirement: &19029520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *19122580
+  version_requirements: *19029520
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &19121980 !ruby/object:Gem::Requirement
+  requirement: &19028920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +65,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *19121980
+  version_requirements: *19028920
 description: Rails gem that interfaces with the WePay API
 email: adammede@gmail.com
 executables: []
@@ -108,7 +108,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1065642594911744828
+      hash: -782813271623262101
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: