Hafiz\Commands\MakeEntity::run()

Complexity

Conditions	8
Paths	22

Size

Total Lines	53
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php namespace Hafiz\Commands;
    public function run(array $params = [])
    {
        helper(['inflector', 'filesystem']);
        $file = new FileHandler();

        $name = array_shift($params);
        $ns = $params['-n'] ?? CLI::getOption('n');
        $table = $params['-t'] ?? CLI::getOption('t');

        if (empty($name)) {
            $name = CLI::prompt(lang('Recharge.nameEntity'), null, 'required|string');
        }

        if (empty($name)) {
            CLI::error(lang('Recharge.badName'));
            return;
        }

        //namespace locator
        $nsinfo = $file->getNamespaceInfo($ns, 'App');

        //class & file name
        $name = singular(pascalize($name));
        $ns = $nsinfo['ns'];
        $targetDir = $nsinfo['path'] . '/Entities/';
        $filepath = $targetDir . $name . '.php';

        if ($file->verifyDirectory($filepath)) {

            //do we have to add table info
            if (!empty($table)) {
                $db = new DBHandler();

                if ($db->checkTableExist($table)) {

It seems like $table can also be of type boolean; however, parameter $table of Hafiz\Libraries\DBHandler::checkTableExist() does only seem to accept string, maybe add an additional type check?

                    $properties = $db->getEntityProperties($table);
                    extract($properties);
                }
            }

            $data = ['{namespace}' => $ns, '{name}' => $name, '{created_at}' => date("d F, Y h:i:s A"),
                '{attributes}' => $attributes ?? NULL, '{dates}' => $dates ?? NULL, '{casts}' => $casts ?? NULL];

            //check a directory exist
            if ($file->checkFileExist($filepath) == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

                $template = $file->renderTemplate('entity', $data);


                if (!write_file($filepath, $template)) {
                    CLI::error(lang('Recharge.writeError', [$filepath]));
                    return;
                }

                CLI::write('Created file: ' . CLI::color($filepath, 'green'));
            }