Pull in old png Solidus logo

The Solidus admin area was recently changed to use an svg logo solidusio/solidus#2995 but that is not supported by Prawn so it broke the pdf embed here. The old png logo is no longer used in Solidus so it's only a matter of time before someone removes it. As such I have pulled it in for use in the pdf header, but kept the same name because I figured it less likely that the png logo ever be updated again in Solidus than maybe someone customized their Solidus install by overwriting the image instead of changing the config. This has the unfortunate affect of no longer automatically following Solidus admin logo customizations, but I don't think it would be worth doing fancy svg detection at that point. We have our own config, folks can use that if needed.
solidusio-contrib · Jan 13, 2019 · aee03d3 · aee03d3
1 parent 1204e9a
commit aee03d3
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 2 deletions.
diff --git a/app/assets/images/logo/solidus_logo.png b/app/assets/images/logo/solidus_logo.png
diff --git a/app/models/spree/print_invoice_configuration.rb b/app/models/spree/print_invoice_configuration.rb
@@ -2,7 +2,7 @@ module Spree
   class PrintInvoiceConfiguration < Preferences::Configuration
 
     preference :print_invoice_next_number, :integer, :default => nil
-    preference :print_invoice_logo_path, :string, :default => Spree::Config[:admin_interface_logo]
+    preference :print_invoice_logo_path, :string, :default => 'logo/solidus_logo.png'
     preference :print_invoice_logo_scale, :integer, :default => 50
     preference :print_invoice_font_face, :string, :default => 'Helvetica'
     preference :print_buttons, :string, :default => 'invoice'

diff --git a/lib/solidus_print_invoice/version.rb b/lib/solidus_print_invoice/version.rb
@@ -1,5 +1,5 @@
 module Spree
   module PrintInvoice
-    VERSION = "1.0.2".freeze
+    VERSION = "1.1.2".freeze
   end
 end