Laztopaz\EmojiRestfulAPI\EmojiController

Complexity

Total Complexity

41

Size/Duplication

Total Lines	356
Duplicated Lines	5.62 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes	21
Bugs	7	Features	4

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	listAllEmoji()	0	13	2
B	getSingleEmoji()	0	25	4
B	createEmoji()	0	43	5
B	updateEmojiByPutVerb()	0	32	4
B	updateEmojiByPatchVerb()	0	24	4
A	deleteEmoji()	0	13	2
A	createEmojiKeywords()	18	18	3
A	formatEmoji()	0	10	2
B	getCurrentUserId()	0	25	3
A	checkForDuplicateEmoji()	0	16	3
C	validateUserInput()	0	33	8

<?php
/**
 * @author   Temitope Olotin <temitope.olotin@andela.com>
 * @license  <https://opensource.org/license/MIT> MIT
 */
namespace Laztopaz\EmojiRestfulAPI;

use Exception;
use Firebase\JWT\JWT;
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface as Request;
use Illuminate\Database\Capsule\Manager as Capsule;

class EmojiController
{
    private $auth;

    public function __construct(Oauth $auth)
    {
        $this->auth = $auth;
    }

    /**
     * This method list all emojis.
     *
     * @param $response
     *
     * @return json $emojis
     */
    public function listAllEmoji(Response $response)
    {
        $emojis = Emoji::with('keywords', 'category', 'created_by')
        ->get()
        ->toArray();

        if (count($emojis) > 0) {
            return $response
            ->withJson($this->formatEmoji($emojis), 200);
        }

        return $response->withJson(['status'], 404);
    }

    /**
     * This method get a single emoji.
     *
     * @param $response
     * @param $args
     *
     * @return json $emoji
     */
    public function getSingleEmoji(Response $response, $args)
    {
        $emoji = null;

        if ($args['id']) {
            $emoji = Emoji::where('id', '=', $args['id'])
            ->with('keywords', 'category', 'created_by')
            ->get();

            if (count($emoji) <= 0) {
                $emoji = Emoji::where('name', '=', strtolower($args['id']))
                ->with('keywords', 'category', 'created_by')
                ->get();
            }
        } 

        $emoji = $emoji ->toArray();

        if (count($emoji) > 0) {
            return $response
            ->withJson($this->formatEmoji($emoji), 200);
        }

        return $response->withJson(['message' => 'Emoji not found'], 404);
    }

    /**
     * This method creates a new emoji.
     *
     * @param $args
     *
     * @return json $response;
     */
    public function createEmoji(Request $request, Response $response)
    {
        $requestParams = $request->getParsedBody();

        if (is_array($requestParams)) {
            $created_at = date('Y-m-d h:i:s');

            $userId = $this->getCurrentUserId($request, $response);

            if (! $this->checkForDuplicateEmoji($requestParams['name'])) {
                // Validate the user input fields
                $validateResponse = $this->validateUserInput([
                    'name', 
                    'char', 
                    'category', 
                    'keywords'
                ], $requestParams);

                if (is_array($validateResponse)) {
                    return $response->withJson($validateResponse, 400);
                }

                $emojiKeyword = $requestParams['keywords'];

                $emoji = Emoji::create(
                [
                    'name'       => strtolower($requestParams['name']),
                    'char'       => $requestParams['char'],
                    'created_at' => $created_at,
                    'category'   => $requestParams['category'],
                    'created_by' => $userId,
                ]);

                if ($emoji->id) {

The property id does not exist on object<Laztopaz\EmojiRestfulAPI\Emoji>. Since you implemented __get, maybe consider adding a @property annotation.

                    $createdKeyword = $this->createEmojiKeywords($emoji->id, $emojiKeyword);

The property id does not exist on object<Laztopaz\EmojiRestfulAPI\Emoji>. Since you implemented __get, maybe consider adding a @property annotation.

$createdKeyword is not used, you could remove the assignment.

                    return $response->withJson($emoji->toArray(), 201);
                }
            }

            return $response->withJson(['message' => 'Emoji cannot be duplicated'], 400);
        }
    }

    /**
     * This method updates an emoji.
     *
     * @param $request
     * @param $response
     *
     * @return json
     */
    public function updateEmojiByPutVerb(Request $request, Response $response, $args)
    {
        $upateParams = $request->getParsedBody();

        if (is_array($upateParams)) {

            $emoji = Emoji::find($args['id']);

            if (count($emoji) > 0) {
                // Validate the user input fields
                $validateResponse =  $this->validateUserInput([
                    'name', 
                    'char', 
                    'category'
                ], $requestParams);

The variable $requestParams does not exist. Did you mean $request?

                if (is_array($validateResponse)) {
                    return $response->withJson($validateResponse, 400);
                }

                $emoji->name = $upateParams['name'];
                $emoji->char = $upateParams['char'];
                $emoji->category = $upateParams['category'];
                $emoji->updated_at = date('Y-m-d h:i:s');
                $emoji->save();

The method save does only exist in Illuminate\Database\Eloquent\Model, but not in Illuminate\Database\Eloquent\Builder and Illuminate\Database\Eloquent\Collection.

                return $response->withJson(['message' => 'Record updated successfully'], 200);
            }

            return $response->withJson(['message' => 'Record cannot be updated because the id supplied is invalid'], 404);
        }
    }

    /**
     * This method updates an emoji partially.
     *
     * @param $request
     * @param $response
     *
     * @return json
     */
    public function updateEmojiByPatchVerb(Request $request, Response $response, $args)
    {
        $upateParams = $request->getParsedBody();

        if (is_array($upateParams)) {
            $emoji = Emoji::find($args['id']);

            if (count($emoji) > 0) {
                //Validate user inputs 
                $validateResponse = $this->validateUserInput(['name'], $requestParams);

The variable $requestParams does not exist. Did you mean $request?

                if (is_array($validateResponse)) {
                    return $response->withJson($validateResponse, 400);
                }

                $emoji->name = $upateParams['name'];
                $emoji->updated_at = date('Y-m-d h:i:s');
                $emoji->save();

The method save does only exist in Illuminate\Database\Eloquent\Model, but not in Illuminate\Database\Eloquent\Builder and Illuminate\Database\Eloquent\Collection.

                return $response->withJson($emoji->toArray(), 200);

The method toArray does only exist in Illuminate\Database\Eloquent\Collection and Illuminate\Database\Eloquent\Model, but not in Illuminate\Database\Eloquent\Builder.

            }

            return $response->withJson(['message' => 'No record to update because the id supplied is invalid'], 404);
        }
    }

    /**
     * This method deletes an emoji.
     *
     * @param $request
     * @param $response
     * @param $args
     *
     * @return json
     */
    public function deleteEmoji(Request $request, Response $response, $args)

The parameter $request is not used and could be removed.

    {
        $emoji = Emoji::find($args['id']);
        if (count($emoji) > 0) {
            $emoji->delete();

The method delete does only exist in Illuminate\Database\Eloquent\Builder and Illuminate\Database\Eloquent\Model, but not in Illuminate\Database\Eloquent\Collection.

            // Delete keywords associated with the emoji
            Keyword::where('emoji_id', '=', $args['id'])->delete();

            return $response->withJson(['message' => 'Emoji was sucessfully deleted'], 200);
        }

        return $response->withJson(['message' => 'Emoji cannot be deleted because the id supplied is invalid'], 404);
    }

    /**
     * This method creates emoji keywords.
     *
     * @param $request
     * @param $response
     * @param $args
     *
     * @return $id

The doc-type $id could not be parsed: Unknown type name "$id" at position 0. (view supported doc-types)

     */
    public function createEmojiKeywords($emoji_id, $keywords)

This method seems to be duplicated in your project.

    {
        if ($keywords) {
            $splittedKeywords = explode(',', $keywords);

            $created_at = date('Y-m-d h:i:s');

            foreach ($splittedKeywords as $keyword) {
                $emojiKeyword = Keyword::create([
                        'emoji_id'     => $emoji_id,
                        'keyword_name' => $keyword,
                        'created_at'   => $created_at,
                ]);
            }
        }

        return $emojiKeyword->id;

The property id does not exist on object<Laztopaz\EmojiRestfulAPI\Keyword>. Since you implemented __get, maybe consider adding a @property annotation.

The variable $emojiKeyword does not seem to be defined for all execution paths leading up to this point.

    }

    /**
     * This method format emoji result
     *
     * @param $emojis
     *
     * @return array $emojis
     */
    public function formatEmoji(array $emojis)
    {
        foreach ($emojis as $key => &$value) {
            $value['created_by'] = $value['created_by']['firstname'].' '.$value['created_by']['lastname'];
            $value['category'] = $value['category']['category_name'];
            $value['keywords'] = array_map(function ($key) { return $key['keyword_name']; }, $value['keywords']);
        }

        return $emojis;
    }

    /**
     * This method authenticate and return user id.
     */
    public function getCurrentUserId($request, $response)
    {
        $loadEnv = DatabaseConnection::loadEnv();

Are you sure the assignment to $loadEnv is correct as \Laztopaz\EmojiRestfulAPI\DatabaseConnection::loadEnv() (which targets Laztopaz\EmojiRestfulAPI\DatabaseConnection::loadEnv()) seems to always return null.

$loadEnv is not used, you could remove the assignment.

        $jwtoken = $request->getHeader('HTTP_AUTHORIZATION');

        try {
            if (isset($jwtoken)) {
                $secretKey = base64_decode(getenv('secret'));

                $jwt = json_decode($jwtoken[0], true);

                //decode the JWT using the key from config
                $decodedToken = JWT::decode($jwt['jwt'], $secretKey, ['HS512']);

                $tokenInfo = (array) $decodedToken;

                $userInfo = (array) $tokenInfo['dat'];

                return $userInfo['id'];
            }
        } catch (Exception $e) {
            return $response->withJson(['status' => $e->getMessage()], 401);
        }
    }

    /**
     * This method checks for duplicate emoji
     *
     * @param $name
     *
     * @return boolean true
     */
    public function checkForDuplicateEmoji($emojiName)
    {
        if (isset($emojiName)) {
            $emojiFound = Capsule::table('emojis')
            ->where('name', '=', strtoupper($emojiName))
            ->orWhere('name', '=', strtolower($emojiName))
            ->orWhere('name', '=', ucwords($emojiName))
            ->orWhere('name', '=', $emojiName)
            ->get();
            
            if (count($emojiFound) > 0) {
                return true;
            }
        }
        return false;
    }

    /**
     * This method will
     * verify the fields supplied by the user when posting to the API
     * and also validate their input for empty values
     *
     * @param $expectedFields
     * @param $suppliedFields
     * 
     * @return json response
     */
    public function validateUserInput(array $expectedFields, array $suppliedFields)
    {
        $counter = 0;

        if (count($suppliedFields) < count($expectedFields)) {
            return ['message' => 'All fields must be supplied'];

        } else { // Check whether the field supplied by the user is what we expect from them
            foreach ($suppliedFields as $key => $value) {
                if (! in_array($key, $expectedFields)) {
                    $counter++;
                }
            }
            if ($counter > 0) {
                $counter = 0;

$counter is not used, you could remove the assignment.

                return ['message' => 'Unwanted fields must be removed'];

            } else { // Check whether all fields have corresponding values
                foreach ($suppliedFields as $key => $value) {
                    if ($value == "") {
                        $counter++;
                    }
                }
                if ($counter > 0) {
                    return ['message' => 'All fields are required'];

                } else {
                    return true;

                }
            }
        }
    }

}