Decide whether or not to display comment based on trusted commenter

jacobemerick · Jun 12, 2016 · 9fb6a24 · 9fb6a24
1 parent 1e04fd1
commit 9fb6a24
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/src/Controller/Comment.php b/src/Controller/Comment.php
@@ -41,17 +41,18 @@ public function createComment(Request $req, Response $res)
 
         // todo option to pass in by commenter id
         $commenterModel = new CommenterModel($this->container->get('dbal'));
-        $commenterId = $commenterModel->findByFields(
+        $commenter = $commenterModel->findByFields(
             $body['commenter']['name'],
             $body['commenter']['email'],
             $body['commenter']['website']
         );
-        if (!$commenterId) {
+        if (!$commenter) {
             $commenterId = $commenterModel->create(
                 $body['commenter']['name'],
                 $body['commenter']['email'],
                 $body['commenter']['website']
             );
+            $commenter = $commenterModel->findById($commenterId);
         }
 
         $bodyModel = new CommentBodyModel($this->container->get('dbal'));
@@ -103,15 +104,20 @@ public function createComment(Request $req, Response $res)
             );
         }
 
+        $shouldDisplay = $commenter['is_trusted'];
+        if (!empty($body['should_display'])) {
+            $shouldDisplay = (int) $body['should_display'];
+        }
+
         $commentModel = new CommentModel($this->container->get('dbal'));
         $commentId = $commentModel->create(
-            $commenterId,
+            $commenter['id'],
             $bodyId,
             $locationId,
             $commentRequestId,
             $body['url'],
             (int) $body['should_notify'],
-            (int) $body['should_display'],
+            $shouldDisplay,
             time()
         );
 

diff --git a/src/Model/Commenter.php b/src/Model/Commenter.php
@@ -73,7 +73,7 @@ public function findById($id)
     public function findByFields($name, $email, $website)
     {
         $query = "
-            SELECT `id`
+            SELECT *
             FROM `commenter`
             WHERE `name` = :name AND
                   `email` = :email AND
@@ -86,7 +86,7 @@ public function findByFields($name, $email, $website)
             'website' => $website,
         ];
 
-        return $this->extendedPdo->fetchValue($query, $bindings);
+        return $this->extendedPdo->fetchOne($query, $bindings);
     }
 
     /**