raystech / laravel-starter-kit

src/Traits/HasMeta.php

<?php 

namespace Raystech\StarterKit\Traits;

use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Collection;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\MorphMany;
use Illuminate\Database\Query\JoinClause;

trait HasMeta
{
    protected $metaData = NULL;
    
    protected function newMeta($meta, $extra = array())

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

    {
        // dd(__class__);
        // dd($meta);
        $metaModel = new $this->meta_model();

The property meta_model does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

        if($this->meta_global == true) {

The property meta_global does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

            $metaModel->model_type  = __class__;
        }
        $metaModel->{$this->meta_key_name}    = $meta[$this->meta_key_name];

The property meta_key_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

        $metaModel->{$this->meta_value_name}  = $meta[$this->meta_value_name];

The property meta_value_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

        $metaModel->{$this->meta_foreign_key} = $this->{$this->meta_primary_key};

The property meta_primary_key does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

The property meta_foreign_key does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

        // if(!empty($extra)) {
        //  foreach($extra as $key => $value) {

        //  }
        // }
        return $metaModel;
    }


    /**
     * Get the data associated with this model
     *
     * @return hasOne

The type Raystech\StarterKit\Traits\hasOne was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function meta()
    {
        // return $this->morphMany($this->meta_model, $this->meta_foreign_key, $this->primaryKey);
        
        return $this->hasMany($this->meta_model, $this->meta_foreign_key, $this->primaryKey);

The property primaryKey does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

It seems like hasMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

The property meta_model does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

The property meta_foreign_key does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

    }

    /**
     * Dynamically retrieve attributes on the model.
     *
     * @param  string  $key
     * @return mixed
     */
    public function __get($key)
    {
        $value = $this->getAttribute($key);

It seems like getAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (is_null($value)) {
            try {
                if(isset($this->getMeta()->$key)) {
                    $value = $this->getMeta()->$key->{$this->meta_value_name};

The property meta_value_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                } else {
                    $value = NULL;
                }
            } catch (Exception $e) {

The type Raystech\StarterKit\Traits\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

                $value = NULL;
            }
        }

        return $value;
    }

    /**
     * Dynamically set attributes on the model.
     *
     * @param  string  $key
     * @param  mixed   $value
     * @return void
     */
    public function __set($key, $value)
    {
        if (array_key_exists($key, $this->getAttributes())) {

It seems like getAttributes() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->setAttribute($key, $value);

It seems like setAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return;
        }

        try {
            if (isset($this->getMeta()->$key)) {
                $this->getMeta()->$key->{$this->meta_value_name} = $value;

The property meta_value_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                return;
            }
        } catch (Exception $e) { }

Consider adding a comment why this CATCH block is empty.

        if(\Config::get('database.default') === 'sqlite') {
            $columns = \DB::select('PRAGMA table_info('.$this->table.')');

The property table does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

            foreach ($columns as $column) {
                if ($column->name === $key) {
                    $this->setAttribute($key, $value);
                    return;
                }
            }
        } else {
            $columns = \DB::getSchemaBuilder()->getColumnListing($this->getTable());

It seems like getTable() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            // $columns = \DB::select('DESCRIBE '.$this->table);

            // dd($columns);
            foreach ($columns as $column) {
                if ($column === $key) {
                    $this->setAttribute($key, $value);
                    return;
                }
            }
        }
        $this->getMeta()->$key = $this->newMeta(array($this->meta_key_name => $key, $this->meta_value_name => $value));

The property meta_key_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

    }


    public function __isset($key)
    {
        if ((isset($this->attributes[$key]) 

The property attributes does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

            || isset($this->relations[$key])) 

The property relations does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

            || ($this->hasGetMutator($key) && ! is_null($this->getAttributeValue($key)))) {

It seems like getAttributeValue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like hasGetMutator() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return true;
        }
        return isset($this->getMeta()->$key->{$this->meta_value_name});

The property meta_value_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

    }

    public function push() 
    {
        return parent::push() && $this->deleteMeta() && $this->saveMeta();
    }

    public function save(array $options = array())

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

    {
        return parent::save() && $this->deleteMeta() && $this->saveMeta();
    }

    /**
     * Get the data associated with this model in a usable format
     *
     * @return array
     */
    protected function getMeta()
    {
        if (is_null($this->metaData)) {
            $primaryKey = $this->primaryKey;

The property primaryKey does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

The assignment to $primaryKey is dead and can be removed.

            if (is_null($this->primaryKey)) {
                // If nothing has been set and there is no ID then there will be no data
                $this->metaData = (object)[];
            } else {
                $meta = $this->meta;

The property meta does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                $niceDataArray = array();

                foreach ($meta as $data) {
                    $niceDataArray[$data->{$this->meta_key_name}] = $data;

The property meta_key_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                }

                $this->metaData = (object)$niceDataArray;
            }
        }
        return $this->metaData;
    }

    protected function saveMeta()
    {
        $primaryKey = $this->primaryKey;

The property primaryKey does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

        foreach ((array)$this->getMeta() as $data) {
            
            if (is_null($data->{$this->meta_foreign_key})) {

The property meta_foreign_key does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                $data->{$this->meta_foreign_key} = $this->$primaryKey;
            }

            if(!is_null($data->{$this->meta_value_name})) {

The property meta_value_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                $data->save();
            }
        }
        
        return true;
    }

    protected function deleteMeta() 
    {
        foreach ((array)$this->getMeta() as $data) {
            if(is_null($data->{$this->meta_value_name})) {

The property meta_value_name does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                $dataID = $this->meta_primary_key;

The property meta_primary_key does not exist on Raystech\StarterKit\Traits\HasMeta. Since you implemented __get, consider adding a @property annotation.

                $data->destroy($data->$dataID);
            }
        }
        
        return true;
    }


    
}