From 8b72fa3fbf78e399c84a1e9b5bf08c2a3aeb8d22 Mon Sep 17 00:00:00 2001 From: ashmaroli Date: Mon, 29 Jan 2018 22:06:38 +0530 Subject: [PATCH] move duplicate code to a single private method (#6593) Merge pull request 6593 --- lib/jekyll/site.rb | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/lib/jekyll/site.rb b/lib/jekyll/site.rb index 8cd86fff..fafa3dd6 100644 --- a/lib/jekyll/site.rb +++ b/lib/jekyll/site.rb @@ -458,10 +458,7 @@ module Jekyll def render_docs(payload) collections.each_value do |collection| collection.docs.each do |document| - if regenerator.regenerate?(document) - document.output = Jekyll::Renderer.new(self, document, payload).run - document.trigger_hooks(:post_render) - end + render_regenerated(document, payload) end end end @@ -469,11 +466,15 @@ module Jekyll private def render_pages(payload) pages.flatten.each do |page| - if regenerator.regenerate?(page) - page.output = Jekyll::Renderer.new(self, page, payload).run - page.trigger_hooks(:post_render) - end + render_regenerated(page, payload) end end + + private + def render_regenerated(document, payload) + return unless regenerator.regenerate?(document) + document.output = Jekyll::Renderer.new(self, document, payload).run + document.trigger_hooks(:post_render) + end end end