From 4bc99e2fb74c384480cdc25529fb253a490a0074 Mon Sep 17 00:00:00 2001 From: Steve Root Date: Sat, 12 May 2012 11:00:57 +0100 Subject: [PATCH] Reworked based on Radar's directions --- .../spree/checkout_controller_decorator.rb | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/app/controllers/spree/checkout_controller_decorator.rb b/app/controllers/spree/checkout_controller_decorator.rb index 347bfc5..6337938 100644 --- a/app/controllers/spree/checkout_controller_decorator.rb +++ b/app/controllers/spree/checkout_controller_decorator.rb @@ -195,14 +195,22 @@ module Spree else user_action = Spree::PaypalExpress::Config[:paypal_express_local_confirm] == "t" ? "continue" : "commit" end + + #asset_url doesn't like Spree::Config[:logo] being an absolute url + #if statement didn't work within hash + if URI.parse(Spree::Config[:logo]).absolute? + chosen_image = Spree::Config[:logo] + else + chosen_image = asset_url(Spree::Config[:logo]) + end { :description => "Goods from #{Spree::Config[:site_name]}", # site details... - #:page_style => "foobar", # merchant account can set named config :background_color => "ffffff", # must be hex only, six chars :header_background_color => "ffffff", :header_border_color => "ffffff", + :header_image => chosen_image :allow_note => true, :locale => user_locale, :req_confirm_shipping => false, # for security, might make an option later @@ -212,13 +220,6 @@ module Spree # they've not been tested and may trigger some paypal bugs, eg not showing order # see http://www.pdncommunity.com/t5/PayPal-Developer-Blog/Displaying-Order-Details-in-Express-Checkout/bc-p/92902#C851 } - #asset_url doesn't like Spree::Config[:logo] being an absolute url - #if statement didn't work within hash - if URI.parse(Spree::Config[:logo]).absolute? - {:header_image => Spree::Config[:logo]} - else - {:header_image => asset_url(Spree::Config[:logo])} - end end def user_locale