Skip to content

Commit

Permalink
Fix transaction notifications
Browse files Browse the repository at this point in the history
  • Loading branch information
sagmor committed Jan 27, 2012
1 parent 0f0b5b8 commit a9d01ff
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 14 deletions.
1 change: 1 addition & 0 deletions lib/pag_seguro.rb
Expand Up @@ -5,4 +5,5 @@
require "pag_seguro/version"
require "pag_seguro/item"
require "pag_seguro/payment"
require "pag_seguro/transaction"
require "pag_seguro/client"
8 changes: 4 additions & 4 deletions lib/pag_seguro/client.rb
Expand Up @@ -16,9 +16,9 @@ def payment(attributes)
payment
end

def notification(params)
def notification(code)
Transaction.new(
self.get("transactions/notifications/#{params[:notificationCode]}")["transaction"]
self.get("transactions/notifications/#{code}")["transaction"]
)
end

Expand All @@ -33,15 +33,15 @@ def redirect_url_for(code)
end

def get(url,params = {})
url = API_HOST+url
url = API_HOST + url

params = params.merge({
:email => self.email,
:token => self.token
})

MultiXml.parse(
RestClient.get(url, params)
RestClient.get(url, :params => params)
)
end

Expand Down
6 changes: 3 additions & 3 deletions lib/pag_seguro/payment.rb
Expand Up @@ -39,9 +39,9 @@ def to_xml(options={})
checkout.extraAmount extra_amount if extra_amount
checkout.maxUses max_uses || 1
checkout.maxAge max_age if max_age
end
builder.shipping do |shipping|
shipping.type 3
checkout.shipping do |shipping|
shipping.type 3
end
end
end

Expand Down
14 changes: 7 additions & 7 deletions lib/pag_seguro/transaction.rb
@@ -1,13 +1,13 @@
module PagSeguro
class Transaction < Hashie::Mash
module STATUS
PENDING = 1
ON_ANALYSIS = 2
PAYED = 3
AVAILABLE = 4
DISPUTED = 5
REFUNDED = 6
CANCELED = 7
PENDING = "1"
ON_ANALYSIS = "2"
PAYED = "3"
AVAILABLE = "4"
DISPUTED = "5"
REFUNDED = "6"
CANCELED = "7"
end

def success?
Expand Down

0 comments on commit a9d01ff

Please sign in to comment.