Use a terrible placeholder for users with missing gravatar URLs.

We won't skip users with missing icons because then the ++ number won't add up. Someone with a better icon can fix this or we leave it as is in order to motivate us to clean up the cases where users blew away the original URL and haven't been able to replace them. I believe there is an open issue for this.
metacpan · May 13, 2014 · 3eb60fc · 3eb60fc
1 parent d5de020
commit 3eb60fc
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 12 deletions.
diff --git a/root/inc/breadcrumbs.html b/root/inc/breadcrumbs.html
@@ -51,13 +51,17 @@
 <% FOREACH plusser IN plusser_authors%>
 <!-- display the gravatar url for each pause plusser alongwith their author page as a link.-->
 
-<a href="/author/<% plusser.id %>">
-	<img src="<% plusser.pic %>" width="20" height="20">
-</a>
+<a href="/author/<% plusser.id %>"><img src="<% plusser.pic || '/static/icons/user.png' %>" width="20" height="20"></a>
 
 <% IF loop.count%35 == 0 %>
 <!--To display 35 plussers in one row. -->
 <p> </p>
 <% END %>
 <% END %>
+
 </div>
+
+<!-- Display the count of non pauseid plussers-->
+<% IF plusser_others %>
+<p><b>and <% plusser_others %> non-PAUSE user(s)</b> ++ed this.</b></p>
+<% END %>
diff --git a/root/pod.html b/root/pod.html
@@ -5,10 +5,6 @@
 
 <% INCLUDE inc/breadcrumbs.html plussers_div = plusser_authors%>
 
-<% IF plusser_others %>
-<p><b><% plusser_others %> user(s) ++ed this. </b></p>
-<% END %>
-
 <ul class="nav nav-list slidepanel"  itemscope itemtype="http://schema.org/SoftwareApplication">
   <li class="visible-phone search">
     <form action="<% IF req.uri.path == '/mirrors'; req.uri.path; ELSE; '/search'; END %>">

diff --git a/root/release.html b/root/release.html
@@ -6,11 +6,6 @@
 <!-- Pass the plussers with pauseids details to breadcrumbs.html -->
 <% INCLUDE inc/breadcrumbs.html plussers_div = plusser_authors%>
 
-<!-- Display the count of non pauseid plussers-->
-<% IF plusser_others %>
-<p><b><% plusser_others %> user(s) ++ed this. </b></p>
-<% END %>
-
 <ul class="nav nav-list slidepanel">
     <li class="visible-phone search">
       <form action="<% IF req.uri.path == '/mirrors'; req.uri.path; ELSE; '/search'; END %>">