plezi 0.12.16 → 0.12.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6f6fba282cfa6309e0cccf3aff5da8e19de6fc5e
-  data.tar.gz: 9aa0efac03e405a8d4db93455d56b51ea328c956
+  metadata.gz: 09beaf8c38e5eed4e1cc3580fe3ce78be7628f8c
+  data.tar.gz: dd718e677a9e7a00ca747c5cb06e936e1248453c
 SHA512:
-  metadata.gz: f447f812e27e63bd503c8ed4d945cd7202f5a6e38524b248368730295407a7a49336af8b45bc618e4f13b1f1112df0e0b0a1c73f1c96c0a199abb7277f4701f0
-  data.tar.gz: ab4818fb422e3d0a275f8a73338d50e70d3d092b7776b6bab3a6c8373f86f9b186a85f58e9dc1ee2ef08cdc7a1249aec18189b9786abcf9164c739707bf8aeef
+  metadata.gz: f20d81b4969cec3edd49f65aeef948d08e90ee8a650411bc3c043cc2a80658d1e99df322fecce7c0cc7fd62ed3d61aeddf8caf4fe85a2324321fb5d174170e2d
+  data.tar.gz: 8913e9cfbe636463d0493182de4e7306a0354ea34c210afadba38cf8879f57ca9cc7787af197c02a82bb5939571d49cebfe8359bafbfc39d45157eff83b119ec

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ***
 
+Change log v.0.12.17
+
+**Fix**: error template format would (and should) fallback to 'html' if the originally requested format (i.e. 'json') was missing. An issue with nested template rendering (when the error template would call `render` to render a partial or other template) caused the fallback to revert to the original (missing) format when searching for the nested template. This issue is now fixed, by instructing ErrorCtrl to set the global format rather than the local one.
+
+**Update**: when the `ENV["ENV"]` or `ENV["ENV"]` are set to `"production"`, the SASS asset render engine will be set to compress the css (minify), for smaller transfers.
+
+***
+
 Change log v.0.12.16
 
 **Fix**: fixed an issue with the `#url_for` method, which couldn't be used as a class method.

data/lib/plezi/common/cache.rb

@@ -12,8 +12,6 @@ module Plezi
 		CACHE_LOCK = Mutex.new
 		CACHABLE = (%w{cache object slim haml css map js html scss sass coffee txt xml json yaml rb}).to_set
 
-		@cache_to_disk = true
-
 		# this class holds cached objects (data and modification times)
 		class CacheObject
 			# Cached attributes
@@ -92,11 +90,30 @@ module Plezi
 			( CACHE_STORE[filename] || File.exist?(filename) ) && true
 		end
 
-		# returns true if the file has been update since data was last cached.
+		# returns true if the {#allow_cache_update?} is true and the file has been update since data was last cached.
 		def cache_needs_update? filename
-			return true if CACHE_STORE[filename].nil? || CACHE_STORE[filename].mtime < File.mtime(filename)
+			return true if CACHE_STORE[filename].nil? || (CACHE_STORE[filename].mtime < File.mtime(filename))
 			false
 		end
+
+		# # # The following was discarded because benchmarks show the difference is negligible
+		# @refresh_cache ||= (ENV['ENV'] != 'production')
+		# # get the cache refresh directive for {#cache_needs_update}. Defaults to ENV['ENV'] != 'production'
+		# def allow_cache_update?
+		# 	@refresh_cache ||= (ENV['ENV'] != 'production')
+		# end
+		# # set the cache refresh directive for {#cache_needs_update}
+		# def allow_cache_update= val
+		# 	@refresh_cache = val
+		# end
+		# # returns true if the {#allow_cache_update?} is true and the file has been update since data was last cached.
+		# def cache_needs_update? filename
+		# 	return true if CACHE_STORE[filename].nil? || (allow_cache_update? && (CACHE_STORE[filename].mtime < File.mtime(filename)))
+		# 	false
+		# end
+
+
+
 	end
 
 	public

data/lib/plezi/common/renderer.rb

@@ -41,13 +41,14 @@ module Plezi
 
 		module SASSExt
 			module_function
+			SASS_OPTIONS = { style: (ENV['SASS_STYLE'] || ((ENV['ENV'] || ENV['RACK_ENV']) == 'production' ? :compressed : :nested )) }
 
 			def call filename, context, &block
 				return false unless defined? ::Sass
-				@sass_cache ||= Sass::CacheStores::Memory.new if defined?(::Sass)
+				SASS_OPTIONS[:cache_store] ||= Sass::CacheStores::Memory.new
 				# review mtime and render sass if necessary
 				if refresh_sass?(filename)
-					eng = Sass::Engine.for_file(filename, cache_store: @sass_cache)
+					eng = Sass::Engine.for_file(filename, SASS_OPTIONS)
 					Plezi.cache_data filename, eng.dependencies
 					css, map = eng.render_with_sourcemap("#{File.basename(filename, '.*'.freeze)}.map".freeze)
 					Plezi.cache_data filename.sub(/\.s[ac]ss$/, '.map'.freeze), map.to_json( css_uri: File.basename(filename, '.*'.freeze) )
@@ -58,8 +59,8 @@ module Plezi
 			end
 
 			def refresh_sass? sass
-				return false unless File.exists?(sass)
 				return true if Plezi.cache_needs_update?(sass)
+				# return false unless Plezi.allow_cache_update? # no meaningful performance boost.
 				mt = Plezi.file_mtime(sass)
 				Plezi.get_cached(sass).each {|e| return true if File.exists?(e.options[:filename]) && (File.mtime(e.options[:filename]) > mt)} # fn = File.join( e.options[:load_paths][0].root, e.options[:filename]) 
 				false

data/lib/plezi/common/settings.rb

@@ -36,5 +36,17 @@ module Plezi
 		def uuid
 			@uuid ||= SecureRandom.uuid
 		end
+
+
+		# # # The following was discarded because benchmarks show the difference is negligible
+		# # Get the cache refresh directive for the {#cache_needs_update?} implementation. Defaults to ENV['ENV'] != 'production'
+		# def allow_cache_update?
+		# 	Plezi.allow_cache_update?
+		# end
+		# # Set the cache refresh directive for the {#cache_needs_update?} implementation. Defaults to ENV['ENV'] != 'production'
+		# def allow_cache_update= val
+		# 	Plezi.allow_cache_update = val
+		# end
+		# # **Update**: up until now, Plezi would allow refresh any cached assets ot templated when they are edited. This file review is usually a waste of resources after deployment, since the files never get updated in production environments. Plezi now supports a two new settings options: `Plezi.allow_cache_update?` and `Plezi.allow_cache_update=` which will dictate if file review should be performed after caching. The default is now set to AVOID file reviews for production environments (`ENV['ENV'] || ENV['RACK_ENV'] != 'production'`).
 	end
 end

data/lib/plezi/helpers/http_sender.rb

@@ -13,7 +13,9 @@ module Plezi
 				include ::Plezi::ControllerMagic
 
 				def index
-					render(response.status.to_s) || (params[:format] && (params[:format] != 'html'.freeze) && render(response.status.to_s, format: 'html'.freeze)) || ((response['content-type'.freeze] = 'text/plain'.freeze) && response.class::STATUS_CODES[response.status])
+					render(response.status.to_s) ||
+						(params[:format] && (params[:format] != 'html'.freeze) && (params[:format] = 'html'.freeze) && (response['content-type'] = nil).nil? && render(response.status.to_s)) ||
+						((response['content-type'.freeze] = 'text/plain'.freeze) && response.class::STATUS_CODES[response.status])
 				end
 				def requested_method
 					:index

data/lib/plezi/version.rb

@@ -1,3 +1,3 @@
 module Plezi
-    VERSION = "0.12.16"
+    VERSION = "0.12.17"
 end

data/plezi.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "iodine", "~> 0.1.16"
+  spec.add_dependency "iodine", "~> 0.1.17"
   spec.add_development_dependency "bundler", "~> 1.7"
   spec.add_development_dependency "rake", "~> 10.0"
 

data/resources/mini_app.rb

@@ -1,6 +1,7 @@
 # encoding: UTF-8
 
-## Set working directory, load gems and create logs
+## Set environment, working directory, load gems and create logs
+	ENV['ENV'] ||= ENV['RACK_ENV'] # production ENV will render SASS as compressed.
 	## Using pathname extentions for setting public folder
 	require 'pathname'
 	## Set up root object, it might be used by the environment and\or the plezi extension gems.

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: plezi
 version: !ruby/object:Gem::Version
-  version: 0.12.16
+  version: 0.12.17
 platform: ruby
 authors:
 - Boaz Segev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-15 00:00:00.000000000 Z
+date: 2015-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: iodine
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.16
+        version: 0.1.17
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.16
+        version: 0.1.17
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement