Laztopaz\EmojiRestfulAPI\UploadTableInfo

Complexity

Total Complexity

8

Size/Duplication

Total Lines	73
Duplicated Lines	26.03 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes	5
Bugs	2	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	createCategory()	0	9	1
A	createEmojiKeywords()	18	18	3
A	createEmoji()	0	20	2
A	createUser()	0	13	1

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

class UploadTableInfo
{
    public function __construct()
    {
        $this->createUser();
        $this->createCategory();
        $this->createEmoji();
    }

    public function createUser()
    {
        $user = new UserController();
        $user->createUser([
            'firstname'  => 'Olotin',
            'lastname'   => 'Temitope',
            'username'   => 'laztopaz',
            'password'   => 'tope0852',
            'email'      => 'temitope.olotin@andela.com',
            'created_at' => date('Y-m-d h:i:s'),
            'updated_at' => date('Y-m-d h:i:s'),
        ]);
    }

    public function createEmoji()
    {
        $emojiKeyword = 'eye, face, grin, person';

        $userId = 1;

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

        $emoji = Emoji::create([
            'name'       => 'GRINNING FACE',
            'char'       => '\u{1F606}',
            'created_at' => $created_at,
            'category'   => 1,
            '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.

        }
    }

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

        $category = Category::create([

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

            'category_name' => 'people',
            'created_at'    => $created_at,
        ]);
    }

    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.

    }
}