From 6eec3a7942540966371a7ef6b78a8323fcede852 Mon Sep 17 00:00:00 2001 From: Parker Moore Date: Tue, 6 Aug 2013 20:56:29 +0200 Subject: [PATCH] Print warning when `paginate` is set to `true`. Related to #1105. Sample output: ~/code/jekyll/tsite$ ../bin/jekyll build --trace Configuration file: /Users/parker/code/jekyll/tsite/_config.yml Config Warning: The `paginate` key must be a positive integer or nil. It's currently set to 'true'. Source: /Users/parker/code/jekyll/tsite Destination: /Users/parker/code/jekyll/tsite/_site Generating... done. --- lib/jekyll/configuration.rb | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/lib/jekyll/configuration.rb b/lib/jekyll/configuration.rb index 5b5dda4b..b560aa1d 100644 --- a/lib/jekyll/configuration.rb +++ b/lib/jekyll/configuration.rb @@ -150,7 +150,7 @@ module Jekyll $stderr.puts "#{err}" end - configuration.backwards_compatibilize + configuration.fix_common_issues.backwards_compatibilize end # Public: Split a CSV string into an array containing its values @@ -205,5 +205,17 @@ module Jekyll config end + def fix_common_issues + config = clone + + if config.has_key?('paginate') && (!config['paginate'].is_a?(Integer) || config['paginate'] < 0) + Jekyll.logger.warn "Config Warning:", "The `paginate` key must be a" + + " positive integer or nil. It's currently set to '#{config['paginate'].inspect}'." + config['paginate'] = nil + end + + config + end + end end