Merge pull request #7 from BranHill21/debugging_cov

fixing deployed homepage error
omarkhater-school · Dec 6, 2024 · 74b1afb · 74b1afb
2 parents 4ef36ac + f70e337
commit 74b1afb
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/app/views/layout/_homepage.html.erb b/app/views/layout/_homepage.html.erb
@@ -95,11 +95,13 @@
             <% @all_galleries = Gallery.all %>
             <% map = {} %>
             <% @all_galleries.each do |gal| %>
-              <% total = gal.reviews.sum(&:rating).to_i %>
-              <% count = gal.reviews.size %>
+              <% total = gal.reviews.sum(&:rating).to_i if gal.reviews.present? %>
+              <% total ||= 0 %> 
+              <% count = gal.reviews.size if gal.reviews.present? %>
+              <% count ||= 0 %>
               <% gal.gallery_totalRate ||= total %>
               <% gal.gallery_totalRator ||= count %>
-              <% if map.size < 50 && (gal.gallery_totalRate / gal.gallery_totalRator >= 3) %>
+              <% if map.size < 50 && gal.gallery_totalRator > 0 && (gal.gallery_totalRate / gal.gallery_totalRator >= 3) %>
                 <% map[gal] = gal.gallery_totalRate / gal.gallery_totalRator %>
               <% end %>
             <% end %>