Apps\Model\Admin\Content\FormContentClear

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	labels()	0	6	1
A	make()	0	13	3

<?php

namespace Apps\Model\Admin\Content;

use Apps\ActiveRecord\Content;
use Ffcms\Core\Arch\Model;
use Ffcms\Core\Helper\FileSystem\Directory;
use Illuminate\Database\Eloquent\Collection;

class FormContentClear extends Model
{
    public $count = 0;

    /** @var Content|Collection */
    private $_records;

    /**
     * FormContentClear constructor. Pass content records collection object inside (can be empty collection)
     * @param Content|Collection $records
     */
    public function __construct($records)
    {
        $this->_records = $records;
        $this->count = $this->_records->count();
        parent::__construct();
    }

    /**
     * Form display labels
     * @return array
     */
    public function labels()
    {
        return [
            'count' => __('Trashed content')
        ];
    }

    /**
     * Finally delete content item
     */
    public function make()
    {
        // remove gallery files if exists
        foreach ($this->_records->get() as $record) {

The call to get() misses a required argument $key.

The method get does only exist in Illuminate\Database\Eloquent\Collection, but not in Apps\ActiveRecord\Content.

            $galleryPath = '/upload/gallery/' . (int)$record->id;
            if (Directory::exist($galleryPath)) {
                Directory::remove($galleryPath);
            }
        }

        // finally remove from db
        $this->_records->forceDelete();

The method forceDelete does only exist in Apps\ActiveRecord\Content, but not in Illuminate\Database\Eloquent\Collection.

    }
}