From 4d85355bd2441e07a8624324c0f7a83eab3459e1 Mon Sep 17 00:00:00 2001
From: eddiejaoude <eddieaboujaoude@Eddies-MacBook-Pro.local>
Date: Sat, 31 Aug 2013 10:04:48 +0100
Subject: [PATCH] Updated mapper with docblocs & removed unused (unreachable)
 code

---
 DataMapper/UserMapper.php | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/DataMapper/UserMapper.php b/DataMapper/UserMapper.php
index 22ce20d..ce9fbea 100644
--- a/DataMapper/UserMapper.php
+++ b/DataMapper/UserMapper.php
@@ -26,6 +26,9 @@ namespace DesignPatterns\DataMapper;
 class UserMapper
 {
 
+    /**
+     * @var DBAL
+     */
     protected $_adapter;
 
     public function __construct(DBAL $dbLayer)
@@ -36,15 +39,16 @@ class UserMapper
     /**
      * saves a user object from memory to Database
      *
+     * @param User $user
      * @return boolean
      */
     public function save(User $user)
     {
-        /* $data keys shoulds correspond to valid Table columns on the Database */
+        /* $data keys should correspond to valid Table columns on the Database */
         $data = array(
             'userid'   => $user->getUserId(),
             'username' => $user->getUsername(),
-            'email'   => $user->getEmail(),
+            'email'    => $user->getEmail(),
         );
 
         /* if no ID specified create new user else update the one in the Database */
@@ -56,14 +60,14 @@ class UserMapper
             $this->_adapter->update($data, array('userid = ?' => $id));
             return true;
         }
-
-        return false;
     }
 
     /**
      * finds a user from Database based on ID and returns a User object located
      * in memory
      *
+     * @param $id
+     * @throws \InvalidArgumentException
      * @return User
      */
     public function findById($id)