From 57acba4bfb28eb5038a15d743a9b67e888ebb99c Mon Sep 17 00:00:00 2001
From: Vipul Basapati <vsbasapati@cynget-infotech.com>
Date: Wed, 16 Aug 2017 13:02:11 +0530
Subject: [PATCH] Added author names on the main page and added a link to
 gravatar to point to author's post

---
 Hax/functions.php          | 26 +++++++++++++-------------
 Hax/single.php             |  4 ++--
 Hax/views/article-list.php | 20 ++++++++++++++++++--
 3 files changed, 33 insertions(+), 17 deletions(-)

diff --git a/Hax/functions.php b/Hax/functions.php
index 22ac6ab..deb460c 100644
--- a/Hax/functions.php
+++ b/Hax/functions.php
@@ -52,16 +52,16 @@ function mozhacks_admin_init(){
 
 // Render the Social Sharing checkbox on the settings page
 function mozhacks_settings_field_share_posts() {
-	?>
-	<div class="layout share-posts">
-	<label>
-		<input type="checkbox" id="mozhacks_share_posts" name="mozhacks_share_posts" value="1" <?php checked( '1', get_option('mozhacks_share_posts') ); ?> />
-		<span>
-			<?php _e('Add social sharing buttons to posts and pages', 'mozhacks'); ?>
-		</span>
-	</label>
-	</div>
-	<?php
+  ?>
+  <div class="layout share-posts">
+  <label>
+    <input type="checkbox" id="mozhacks_share_posts" name="mozhacks_share_posts" value="1" <?php checked( '1', get_option('mozhacks_share_posts') ); ?> />
+    <span>
+      <?php _e('Add social sharing buttons to posts and pages', 'mozhacks'); ?>
+    </span>
+  </label>
+  </div>
+  <?php
 }
 
 // Renders the Twitter account setting field to share via.
@@ -323,8 +323,8 @@ function fc_honeypot( array $data ){
     wp_die( $message, $title, $args );
     exit(0);
   } else {
-	   return $data;
-	}
+     return $data;
+  }
 }
 add_filter('preprocess_comment','fc_honeypot');
 
@@ -879,7 +879,7 @@ function fc_custom_loop($args) {
        If 'post__in' is in args and 'orderby' is set to 'none', just grab those posts,
        in the order provided in the 'post__in' array.
     */
-    if($loop_orderby && $loop_orderby == 'none' && $loop_post__in)
+    if(isset($loop_orderby) && $loop_orderby == 'none' && $loop_post__in)
     {
         foreach($loop_post__in as $post_id)
             $loop_posts[] = get_post($post_id);
diff --git a/Hax/single.php b/Hax/single.php
index ef2f964..7168834 100644
--- a/Hax/single.php
+++ b/Hax/single.php
@@ -15,7 +15,7 @@
   <h1 class="post__title"><?php the_title(); ?></h1>
   <div class="byline">
     <h3 class="post__author">
-      <?php if (function_exists(coauthors)) : ?>
+      <?php if (function_exists('coauthors')) : ?>
         <?php
           $authors = get_coauthors($post->ID);
           $authorPos = 0;
@@ -76,7 +76,7 @@
   <article class="post" role="article">
     <?php the_content(); ?>
     <section class="about">
-      <?php if (function_exists(coauthors)) : ?>
+      <?php if (function_exists('coauthors')) : ?>
         <?php $authors = get_coauthors($post->ID); ?>
         <?php foreach ($authors as $author) : ?>
           <h2 class="about__header">About
diff --git a/Hax/views/article-list.php b/Hax/views/article-list.php
index b8791ca..6289295 100644
--- a/Hax/views/article-list.php
+++ b/Hax/views/article-list.php
@@ -1,6 +1,18 @@
 <li class="list-item row listing">
-  <?php $authors = get_coauthors($post->ID); ?>
-  <?php echo get_avatar($authors[0], 72) ?>
+  <?php 
+    $authors = get_coauthors($post->ID);
+  ?>
+  <a href="<?php echo get_author_posts_url($authors[0]->ID); ?>" title="<?php echo $authors[0]->display_name; ?>">
+    <?php echo get_avatar($authors[0], 72) ?>
+  </a>
+  <?php 
+    $author_names = [];
+    foreach($authors as $author)
+    {
+      $author_names[] = $author->display_name;
+    }
+    $authors = implode($author_names, ',');
+  ?>
   <div class="block block--1">
     <h3 class="post__title">
       <a href="<?php the_permalink() ?>"><?php the_title(); ?></a>
@@ -11,6 +23,10 @@
       <abbr class="published" title="<?php the_time('Y-m-d\TH:i:sP'); ?>">
         <?php the_time(get_option('date_format')); ?>
       </abbr>
+      &nbsp;by
+      <abbr title="<?php echo $authors ?>">
+        <?php echo $authors ?>
+      </abbr>
     </div>
   </div>
 </li>