From d70d7988c14d32729e3b4e69e01916286ff9bf10 Mon Sep 17 00:00:00 2001
From: Ashwin Maroli <ashmaroli@gmail.com>
Date: Fri, 3 Mar 2017 17:39:47 +0530
Subject: [PATCH] Reduce ABC size of method

---
 lib/jekyll/collection.rb | 25 +++++++++++++++----------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/lib/jekyll/collection.rb b/lib/jekyll/collection.rb
index bae979dbbfb..8ff4c0e10e4 100644
--- a/lib/jekyll/collection.rb
+++ b/lib/jekyll/collection.rb
@@ -249,18 +249,23 @@ def rearrange_docs!
         end
       end
 
+      assign_sort_index(docs)
       docs.sort_by! { |d| d.data["sort_index"] }
-    rescue
-      # Collect all documents without the virtual `sort_index` key defined and assign
-      # the key with a value equal to the total no. of documents in the main collection.
-      #
-      # Such documents will be sorted alphabetically and appear at the end of the custom
-      # array.
-      docs.select { |d| d.data["sort_index"].nil? }.each do |doc|
-        doc.data["sort_index"] = docs.size
-      end
+    end
 
-      docs.sort_by! { |d| d.data["sort_index"] }
+    # Assign a virtual `sort_index` to all Jekyll::Document objects in the current
+    # collection.
+    #
+    # Collect all documents without the virtual `sort_index` key defined and assign
+    # the key with a value equal to the total no. of documents in the main collection.
+    # Such documents will be sorted alphabetically and appear at the end of the custom
+    # array.
+    #
+    # documents - the array of Jekyll::Document objects from the current Collection
+    def assign_sort_index(documents)
+      documents.select { |d| d.data["sort_index"].nil? }.each do |document|
+        document.data["sort_index"] = documents.size
+      end
     end
 
     def read_static_file(file_path, full_path)