wicked_pdf 1.0.3 → 1.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3a9bd19353fd98c1329694e7e28958e05dd01d97
-  data.tar.gz: 33413fa1202def89fa910d38996ea732ac86b706
+  metadata.gz: 6bb2f6a218b4808fb3f117148379101db09cffb8
+  data.tar.gz: 3d87582716a2d79e4f7b635f4297e50744d47e0a
 SHA512:
-  metadata.gz: a7c925ec176e0976cede93924098016466bd8627a99af611b89c283792ccbd9a154a450a2d9370079d9f6f72d1d48c8c57e0ef5ab3d25f235b242222aa4deabe
-  data.tar.gz: 44b674f5bc1efd1bdb54354f0cc05d13442af7427c53c903821951491f17923466d9d8223303e0519d6b37ba8e2fce8ba534b1d88fcda33c139f61db9d7e1b61
+  metadata.gz: 0579b69ec2897374a6bac52f44306e6c15da9252f09f3f0d6bbbc89a3161f5c4cf47218593a70d9884b9a043f0ca8c5d8c8fdebf38edb9457510cf6706cf2d2a
+  data.tar.gz: be80b3dd951f1a12065237cc5db3ab019b5e048c337e8c3666cc21d7a3685bcee3b5415d566a19ca73631adf7c5757c14115769460b96b13b9497ce18ec85e46

data/.travis.yml

@@ -1,5 +1,8 @@
 language:
   - ruby
+before_install:
+  - sudo apt-get update -qq
+  - gem update bundler
 bundler_args: --verbose
 sudo: required
 rvm:
@@ -26,6 +29,7 @@ matrix:
   include:
     - rvm: 1.8.7
       before_install:
+        - sudo apt-get update -qq
         - gem update --system 1.8.25
         - gem --version
       gemfile: gemfiles/2.3.gemfile

data/{CHANGLOG.md → CHANGELOG.md}

@@ -2,9 +2,7 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
-##
-
-## [1.0.1] - 2015-11-30
+## [master] - Unpublished
 ### Changed
 - The default dpi is now 72. Previously the default would be whatever your `wkhtmltopdf` version specified as the default. This change [speeds up generation of documents that contain `border-radius` dramatically](https://github.com/wkhtmltopdf/wkhtmltopdf/issues/1510)
 

data/lib/wicked_pdf/pdf_helper.rb

@@ -38,7 +38,7 @@ module PdfHelper
   private
 
   def log_pdf_creation
-    logger.info '*' * 15 + 'WICKED' + '*' * 15 unless logger.nil?
+    logger.info '*' * 15 + 'WICKED' + '*' * 15 if logger && logger.respond_to?(:info)
   end
 
   def set_basic_auth(options = {})

data/lib/wicked_pdf/version.rb

@@ -1,3 +1,3 @@
 class WickedPdf
-  VERSION = '1.0.3'
+  VERSION = '1.0.4'
 end

data/lib/wicked_pdf/wicked_pdf_helper/assets.rb

@@ -21,8 +21,7 @@ module WickedPdfHelper
         if Regexp.last_match[1].starts_with?('data:')
           "url(#{Regexp.last_match[1]})"
         else
-          asset = Regexp.last_match[1]
-          "url(#{wicked_pdf_asset_path(asset)})" if asset_exists?(asset)
+          "url(#{wicked_pdf_asset_path(Regexp.last_match[1])})"
         end
       end.html_safe
     end
@@ -57,15 +56,17 @@ module WickedPdfHelper
     URI_REGEXP = %r{^[-a-z]+://|^(?:cid|data):|^//}
 
     def asset_pathname(source)
-      if precompiled_asset?(source)
-        if (pathname = set_protocol(asset_path(source))) =~ URI_REGEXP
+      if precompiled_or_absolute_asset?(source)
+        asset = asset_path(source)
+        if (pathname = set_protocol(asset)) =~ URI_REGEXP
           # asset_path returns an absolute URL using asset_host if asset_host is set
           pathname
         else
-          File.join(Rails.public_path, asset_path(source).sub(/\A#{Rails.application.config.action_controller.relative_url_root}/, ''))
+          File.join(Rails.public_path, asset.sub(/\A#{Rails.application.config.action_controller.relative_url_root}/, ''))
         end
       else
-        Rails.application.assets.find_asset(source).pathname
+        asset = Rails.application.assets.find_asset(source)
+        asset ? asset.pathname : File.join(Rails.public_path, source)
       end
     end
 
@@ -81,25 +82,27 @@ module WickedPdfHelper
       source
     end
 
-    def precompiled_asset?(source)
-      Rails.configuration.assets.compile == false || source.to_s[0] == '/'
+    def precompiled_or_absolute_asset?(source)
+      Rails.configuration.assets.compile == false ||
+        source.to_s[0] == '/' ||
+        source.to_s.match(/\Ahttps?\:\/\//)
     end
 
     def read_asset(source)
-      if precompiled_asset?(source)
-        if set_protocol(asset_path(source)) =~ URI_REGEXP
-          read_from_uri(source)
-        elsif asset_exists?(source)
-          IO.read(asset_pathname(source))
+      if precompiled_or_absolute_asset?(source)
+        if (pathname = asset_pathname(source)) =~ URI_REGEXP
+          read_from_uri(pathname)
+        elsif File.file?(pathname)
+          IO.read(pathname)
         end
       else
         Rails.application.assets.find_asset(source).to_s
       end
     end
 
-    def read_from_uri(source)
+    def read_from_uri(uri)
       encoding = ':UTF-8' if RUBY_VERSION > '1.8'
-      asset = open(asset_pathname(source), "r#{encoding}") { |f| f.read }
+      asset = open(uri, "r#{encoding}") { |f| f.read }
       asset = gzip(asset) if WickedPdf.config[:expect_gzipped_remote_assets]
       asset
     end
@@ -110,9 +113,5 @@ module WickedPdfHelper
       gzipper.read
     rescue Zlib::GzipFile::Error
     end
-
-    def asset_exists?(source)
-      Rails.application.assets.find_asset(source).present?
-    end
   end
 end

data/test/functional/wicked_pdf_helper_assets_test.rb

@@ -6,9 +6,6 @@ class WickedPdfHelperAssetsTest < ActionView::TestCase
 
   if Rails::VERSION::MAJOR > 3 || (Rails::VERSION::MAJOR == 3 && Rails::VERSION::MINOR > 0)
     test 'wicked_pdf_asset_base64 returns a base64 encoded asset' do
-      source = File.new('test/fixtures/wicked.css', 'r')
-      destination = Rails.root.join('app', 'assets', 'stylesheets', 'wicked.css')
-      File.open(destination, 'w') { |f| f.write(source) }
       assert_match /data:text\/css;base64,.+/, wicked_pdf_asset_base64('wicked.css')
     end
 
@@ -18,28 +15,74 @@ class WickedPdfHelperAssetsTest < ActionView::TestCase
     end
 
     test 'wicked_pdf_asset_path should return a url when assets are served by an asset server using HTTPS' do
-      expects(:asset_path => 'https://assets.domain.com/dummy.png', 'precompiled_asset?' => true)
+      Rails.configuration.assets.expects(:compile => false)
+      expects(:asset_path => 'https://assets.domain.com/dummy.png')
       assert_equal 'https://assets.domain.com/dummy.png', wicked_pdf_asset_path('dummy.png')
     end
 
     test 'wicked_pdf_asset_path should return a url with a protocol when assets are served by an asset server with relative urls' do
+      Rails.configuration.assets.expects(:compile => false)
       expects(:asset_path => '//assets.domain.com/dummy.png')
-      expects('precompiled_asset?' => true)
       assert_equal 'http://assets.domain.com/dummy.png', wicked_pdf_asset_path('dummy.png')
     end
 
     test 'wicked_pdf_asset_path should return a url with a protocol when assets are served by an asset server with no protocol set' do
+      Rails.configuration.assets.expects(:compile => false)
       expects(:asset_path => 'assets.domain.com/dummy.png')
-      expects('precompiled_asset?' => true)
       assert_equal 'http://assets.domain.com/dummy.png', wicked_pdf_asset_path('dummy.png')
     end
 
-    test 'wicked_pdf_asset_path should return a path when assets are precompiled' do
-      expects('precompiled_asset?' => false)
+    test 'wicked_pdf_asset_path should return a path' do
+      Rails.configuration.assets.expects(:compile => true)
       path = wicked_pdf_asset_path('application.css')
 
-      assert path.include?('/assets/stylesheets/application.css')
+      assert path.include?('/app/assets/stylesheets/application.css')
       assert path.include?('file:///')
+
+      Rails.configuration.assets.expects(:compile => false)
+      expects(:asset_path => '/assets/application-6fba03f13d6ff1553477dba03475c4b9b02542e9fb8913bd63c258f4de5b48d9.css')
+      path = wicked_pdf_asset_path('application.css')
+
+      assert path.include?('/public/assets/application-6fba03f13d6ff1553477dba03475c4b9b02542e9fb8913bd63c258f4de5b48d9.css')
+      assert path.include?('file:///')
+    end
+
+    # This assets does not exists so probably it doesn't matter what is
+    # returned, but lets ensure that returned value is the same when assets
+    # are precompiled and when they are not
+    test 'wicked_pdf_asset_path should return a path when asset does not exist' do
+      Rails.configuration.assets.expects(:compile => true)
+      path = wicked_pdf_asset_path('missing.png')
+
+      assert path.include?('/public/missing.png')
+      assert path.include?('file:///')
+
+      Rails.configuration.assets.expects(:compile => false)
+      expects(:asset_path => '/missing.png')
+      path = wicked_pdf_asset_path('missing.png')
+
+      assert path.include?('/public/missing.png')
+      assert path.include?('file:///')
+    end
+
+    test 'wicked_pdf_asset_path should return a url when asset is url' do
+      Rails.configuration.assets.expects(:compile => true)
+      expects(:asset_path => 'http://example.com/rails.png')
+      assert_equal 'http://example.com/rails.png', wicked_pdf_asset_path('http://example.com/rails.png')
+
+      Rails.configuration.assets.expects(:compile => false)
+      expects(:asset_path => 'http://example.com/rails.png')
+      assert_equal 'http://example.com/rails.png', wicked_pdf_asset_path('http://example.com/rails.png')
+    end
+
+    test 'wicked_pdf_asset_path should return a url when asset is url without protocol' do
+      Rails.configuration.assets.expects(:compile => true)
+      expects(:asset_path => '//example.com/rails.png')
+      assert_equal 'http://example.com/rails.png', wicked_pdf_asset_path('//example.com/rails.png')
+
+      Rails.configuration.assets.expects(:compile => false)
+      expects(:asset_path => '//example.com/rails.png')
+      assert_equal 'http://example.com/rails.png', wicked_pdf_asset_path('//example.com/rails.png')
     end
 
     test 'WickedPdfHelper::Assets::ASSET_URL_REGEX should match various URL data type formats' do

data/test/test_helper.rb

@@ -16,3 +16,9 @@ end
 require 'wicked_pdf'
 
 Rails.backtrace_cleaner.remove_silencers!
+
+if (assets_dir = Rails.root.join('app/assets')) && File.directory?(assets_dir)
+  destination = assets_dir.join('stylesheets/wicked.css')
+  source = File.read('test/fixtures/wicked.css')
+  File.open(destination, 'w') { |f| f.write(source) }
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: wicked_pdf
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.4
 platform: ruby
 authors:
 - Miles Z. Sterrett
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-02 00:00:00.000000000 Z
+date: 2016-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -121,7 +121,7 @@ files:
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
 - ".travis.yml"
-- CHANGLOG.md
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md