move github api fetching to preprocess, only get each github user a maximum of one time per compile from API, and implement ordinal positioning of items for next-previous footer navigation

Author: ajpiano

Rules

@@ -89,23 +89,52 @@ preprocess do
   ]
 
   @chapters = {}
+
+  @github_users = {
+    "jquery" =>  nil
+  }
+
   @items.each do |item|
     item[:chapter] = item[:filename].split('/')[1] 
     item[:chapter_title] = item[:chapter].gsub(/-/, " ").upcase
+    if item[:github]
+      @github_users[ item[:github] ] = nil
+    else
+      item[:github] = "jquery"
+    end
   end
 
+  @github_users.each do |username, wat|
+    request = Curl::Easy.http_get("https://api.github.com/users/"+username) 
+    request.perform
+    @github_users[ username ] = JSON.parse request.body_str
+  end
+
+
   @groupedItems = @items.group_by {|item| item[:chapter]}
 
+  @orderedItems = []
+
   @chapterOrder.each do |folder|
     myitems = @groupedItems[ folder ]
     @chapters [ folder] = {}
     @chapters[ folder ][ :items ] = @groupedItems[folder].sort_by {|i| i[:section] || 0 }
+    @orderedItems = @orderedItems + @chapters[ folder ][ :items ]
     @chapters[ folder ][ :title ] = folder.gsub(/-/, " ").upcase
     @chapters[ folder ][ :folder ] = folder
   end
 
-  @site.config[:chapterOrder] = @chapterOrder
+  @items.each do |item|
+    i = item[:ordinal_index] = @orderedItems.index(item)
+    if i
+      item[:next_item] = @orderedItems[ i+1 ]
+      item[:previous_item] = @orderedItems[ i-1 ]
+    end
+    item[:github_user] = @github_users[ item[:github] ]
+  end
+
   @site.config[:chapters] = @chapters
+  @site.config[:orderedItems] = @orderedItems
 end
 
 compile '/assets/*' do

layouts/article_footer.html

@@ -1,15 +1,7 @@
 <div class="clearfix meta">
 	<div class="github">
-		<%if @item[:github] %>
-		<% github_user = @item[:github] || "jquery" %>
-		<% user_request = Curl::Easy.http_get("https://api.github.com/users/"+github_user) %>
-		<% user_request.perform %>
-		<% user_obj = JSON.parse(user_request.body_str) %>
-		<img src="<%= user_obj["avatar_url"] %>" class="avatar" />
-		<h3 class="author">By <a href="<%= user_obj["html_url"] %>"><%= user_obj["name"] %></a></h3>
-		<% else %>
-		<h3 class="author">By <%= @item[:attribution] %></h3>
-		<% end %>
+		<img src="<%= @item[:github_user]["avatar_url"] %>" class="avatar" />
+		<h3 class="author">By <a href="<%= @item[:github_user]["html_url"] %>"><%= @item[:github_user]["name"] %></a></h3>
 	</div>
 	<div class="feedback">
 		<h4>Suggestions? Problems? Feedback?</h4>
@@ -22,6 +14,12 @@ <h4>Suggestions? Problems? Feedback?</h4>
 </div>
 
 <div class="clearfix pagination">
-	<a href="#" class="previous"><span class="article">jQuery for Designers</span> <span class="direction">Previous</span></a>
-	<a href="#" class="next"><span class="article">$(document).ready()</span> <span class="direction">Next</span></a>
+
+  <% if @item[:previous_item] %>
+  <a href="<%= @item[:previous_item].path %>" class="previous"><span class="article"><%= @item[:previous_item][:title] %></span> <span class="direction">Previous</span></a>
+  <% end %>
+  <% if @item[:next_item] %>
+  <a href="<%= @item[:next_item].path %>" class="next"><span class="article"><%= @item[:next_item][:title] %></span> <span class="direction">Next</span></a>
+  <% end %>
+
 </div>