From c90716606b3556c1d2cc17c17c4931df41dfc911 Mon Sep 17 00:00:00 2001
From: Mirv <reski.raptor@gmail.com>
Date: Wed, 31 May 2017 19:24:51 +0000
Subject: [PATCH] formatting for reading, fixed error with user_id, made helper
 to remove some logic out of the way

---
 app/helpers/recipes_helper.rb                | 18 ++++++++++-----
 app/views/recipes/_gravatar_section.html.erb |  2 +-
 app/views/recipes/show.html.erb              | 24 +++++++++++++-------
 3 files changed, 29 insertions(+), 15 deletions(-)

diff --git a/app/helpers/recipes_helper.rb b/app/helpers/recipes_helper.rb
index 7ef181b..74d231b 100644
--- a/app/helpers/recipes_helper.rb
+++ b/app/helpers/recipes_helper.rb
@@ -1,9 +1,15 @@
 module RecipesHelper
-    def is_owner?
-      if current_user.id == @recipe.user_id then
-          return true
-      else
-          return false
-      end
+  def is_owner?
+    if current_user.id == @recipe.user_id then
+        return true
+    else
+        return false
     end
+  end
+  
+  def displayGravator?(user)
+    link_to gravatar_for(user, size: 100), user_path(user) unless user.nil?
+    
+  end
+  
 end
\ No newline at end of file
diff --git a/app/views/recipes/_gravatar_section.html.erb b/app/views/recipes/_gravatar_section.html.erb
index 4971b94..6913a0e 100644
--- a/app/views/recipes/_gravatar_section.html.erb
+++ b/app/views/recipes/_gravatar_section.html.erb
@@ -1,2 +1,2 @@
-<%= link_to gravatar_for(user, size: 100), user_path(user) %>
+<%= displayGravator?(user) %>
 <h5> User name: <%= user.first_name %></h5>
diff --git a/app/views/recipes/show.html.erb b/app/views/recipes/show.html.erb
index 75a5d55..5fd735b 100644
--- a/app/views/recipes/show.html.erb
+++ b/app/views/recipes/show.html.erb
@@ -27,7 +27,8 @@
 				<h3>Directions</h3>
 				<ol>
 					<%@recipe.directions.each do |direction| %>
-					<li><%= direction.step %></li><%end %>
+						<li><%= direction.step %></li>
+					<%end %>
 				</ol>
 				<%if @recipe.styles.any? %>
 					<p>Styles: <%= render @recipe.styles %></p>
@@ -65,13 +66,20 @@
 			<%= render partial: 'gravatar_section', locals: {user: @user} %>
 			<p></p><br>
 			<h3><br>
-			Reviews</h3><%if @recipe.reviews.any? %><%@recipe.reviews.each do |c| %>
-			<div class="well">
-				<div class="col-md-2 pull-right">
-					<strong>User: <%= c.user.username.downcase.capitalize %></strong>
-				</div>
-				<p><%= c.body %></p><i><small><%= c.updated_at.strftime("%m/%d/%Y") %></small></i> <%if c.user == current_user %> <%= link_to "Delete", review_recipe_path(:revid => c.id), class:"btn btn-danger btn-small pull-right", method: :delete, data: { confirm: "Are you sure you want to delete this review?" }  %> <%end %>
-			</div><%end %><%else %>
+			Reviews</h3>
+			<%if @recipe.reviews.any? %>
+				<%@recipe.reviews.each do |c| %>
+					<div class="well">
+						<div class="col-md-2 pull-right">
+							<strong>User: <%= c.user.username.downcase.capitalize %></strong>
+						</div>
+						<p><%= c.body %></p><i><small><%= c.updated_at.strftime("%m/%d/%Y") %></small></i> 
+						<%if c.user == current_user %> 
+							<%= link_to "Delete", review_recipe_path(:revid => c.id), class:"btn btn-danger btn-small pull-right", method: :delete, data: { confirm: "Are you sure you want to delete this review?" }  %> 
+						<%end %>
+					</div>
+				<%end %>
+			<%else %>
 			
 			<h5>Be the first to post a review!</h5><%end %>
 			<div class="form-group">