SleepingOwl\Admin\Display\Column\Editable\DateTime

Complexity

Total Complexity

16

Size/Duplication

Total Lines	168
Duplicated Lines	12.5 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

0%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	toArray()	0	18	2
A	getFormatedDate()	12	12	3
A	getValueFromModel()	0	7	2
A	hasSeconds()	0	4	1
A	setSeconds()	0	6	1
A	setModelAttribute()	9	9	2
A	save()	0	18	1
A	getPickerFormat()	0	4	2
A	setCurrentDate()	0	6	1

<?php

namespace SleepingOwl\Admin\Display\Column\Editable;

use Carbon\Carbon;
use Illuminate\Http\Request;
use SleepingOwl\Admin\Form\FormDefault;
use SleepingOwl\Admin\Traits\DateFormat;
use SleepingOwl\Admin\Traits\DatePicker;
use SleepingOwl\Admin\Contracts\Display\ColumnEditableInterface;

class DateTime extends EditableColumn implements ColumnEditableInterface
{
    use DatePicker, DateFormat;

    /**
     * @var string
     */
    protected $format = 'Y-m-d H:i:s';
//    protected $format = 'YYYY-MM-DD';

45% of this comment could be valid code. Did you maybe forget this after debugging?

    protected $type = 'combodate';

    /**
     * @var string
     */
    protected $timezone;

    /**
     * @var bool
     */
    protected $seconds = false;

    /**
     * @var string
     */
    protected $view = 'column.editable.datetime';

    /**
     * Text constructor.
     *
     * @param             $name
     * @param             $label
     */
    public function __construct($name, $label = null)
    {
        parent::__construct($name, $label);
    }

    /**
     * @return array
     */
    public function toArray()
    {
        $value = $this->getModelValue();

        return parent::toArray() + [
                'id'             => $this->getModel()->getKey(),
                'value'          => $this->getFormatedDate($value),
                'isEditable'     => $this->getModelConfiguration()->isEditable($this->getModel()),
                'url'            => $this->getUrl(),

                'format'          => $this->getJsPickerFormat(),
                'viewformat'      => $this->getJsPickerFormat(),
                'data-date-pickdate'   => 'true',
                'data-date-picktime'   => 'false',
                'data-date-useseconds' => $this->hasSeconds() ? 'true' : 'false',
                'type'                 => $this->type,
            ];
    }

    /**
     * @param string $date
     *
     * @return null|string
     */
    protected function getFormatedDate($date)

This method seems to be duplicated in your project.

    {
        if (! is_null($date)) {
            if (! $date instanceof Carbon) {
                $date = Carbon::parse($date);
            }

            $date = $date->timezone($this->getTimezone())->format($this->getFormat());
        }

        return $date;
    }

    /**
     * @return $this|NamedFormElement|mixed|null|string
     */
    public function getValueFromModel()
    {
        $value = parent::getValueFromModel();

It seems like you code against a specific sub-type and not the parent class SleepingOwl\Admin\Display\Column\Editable\EditableColumn as the method getValueFromModel() does only exist in the following sub-classes of SleepingOwl\Admin\Display\Column\Editable\EditableColumn: SleepingOwl\Admin\Display\Column\Editable\DateTime. Maybe you want to instanceof check for one of these explicitly?

        if (! empty($value)) {
            return $this->parseValue($value);

The method parseValue() does not seem to exist on object<SleepingOwl\Admin\Display\Column\Editable\DateTime>.

        }
    }

    /**
     * @return bool
     */
    public function hasSeconds()
    {
        return (bool) $this->seconds;
    }

    /**
     * @param bool $seconds
     *
     * @return $this
     */
    public function setSeconds($seconds)
    {
        $this->seconds = $seconds;

        return $this;
    }

    /**
     * @param mixed $value
     *
     * @return void
     */
    public function setModelAttribute($value)

This method seems to be duplicated in your project.

    {
        $value = ! empty($value)
            ? Carbon::createFromFormat($this->getPickerFormat(), $value, $this->getTimezone())
                ->timezone(config('app.timezone'))->format($this->getFormat())
            : null;

        parent::setModelAttribute($value);

The method setModelAttribute() does not exist on SleepingOwl\Admin\Display\Column\Editable\EditableColumn. Did you maybe mean setModel()?

    }

    /**
     * @param Request $request
     *
     * @return void
     */
    public function save(Request $request)
    {
        $form = new FormDefault([
            new \SleepingOwl\Admin\Form\Element\Text(
                $this->getName()
            ),
        ]);

        $model = $this->getModel();

        $request->offsetSet($this->getName(), $request->input('value', null));

        $form->setModelClass(get_class($model));
        $form->initialize();
        $form->setId($model->getKey());

        $form->saveForm($request);
    }

    /**
     * @return string
     */
    public function getPickerFormat()
    {
        return $this->pickerFormat ?: config('sleeping_owl.datetimeFormat');
    }

    /**
     * @return $this
     *
     * SMELLS This function does more than it says.
     */
    public function setCurrentDate()
    {
        $this->defaultValue = Carbon::now()->timezone($this->getTimezone())->format($this->getFormat());

The property defaultValue does not exist. Did you maybe forget to declare it?

        return $this;
    }
}