Thinktomorrow\Chief\App\Http\Controllers\Back\Assistants\ArchiveController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	unarchive()	0	9	1
A	archive()	0	24	5
A	index()	0	9	1
A	__construct()	0	3	1

<?php

namespace Thinktomorrow\Chief\App\Http\Controllers\Back\Assistants;

use Illuminate\Http\Request;
use Thinktomorrow\Chief\FlatReferences\FlatReferenceCollection;
use Thinktomorrow\Chief\FlatReferences\FlatReferenceFactory;
use Thinktomorrow\Chief\Management\Managers;
use Thinktomorrow\Chief\App\Http\Controllers\Controller;
use Thinktomorrow\Chief\Urls\UrlRecord;

class ArchiveController extends Controller
{
    /** @var Managers */
    private $managers;

    public function __construct(Managers $managers)
    {
        $this->managers = $managers;
    }

    public function index(Request $request, string $key)
    {
        $manager = $this->managers->findByKey($key);

        $managers = $manager->assistant('archive')->findAll();

The method findAll() does not exist on Thinktomorrow\Chief\Management\Assistants\Assistant. It seems like you code against a sub-type of Thinktomorrow\Chief\Management\Assistants\Assistant such as Thinktomorrow\Chief\Management\Assistants\PublishAssistant or Thinktomorrow\Chief\Management\Assistants\ArchiveAssistant.

        return view('chief::back.managers.archive.index', [
            'modelManager' => $manager,
            'managers' => $managers,
        ]);
    }

    public function archive(Request $request, $key, $id)
    {
        $manager = $this->managers->findByKey($key, $id);

        if ($redirectReference = $request->get('redirect_id')) {
            $model = FlatReferenceFactory::fromString($redirectReference)->instance();

            $targetRecords = UrlRecord::getByModel($model);

            // Ok now get all urls from this model and point them to the new records
            foreach (UrlRecord::getByModel($manager->model()) as $urlRecord) {
                if ($targetRecord = $targetRecords->first(function ($record) use ($urlRecord) {
                    return ($record->locale == $urlRecord->locale && !$record->isRedirect());
                })) {
                    $urlRecord->redirectTo($targetRecord);
                }
            }
        }

        $manager->assistant('archive')
                        ->guard('archive')
                        ->archive();

The method archive() does not exist on Thinktomorrow\Chief\Management\Assistants\Assistant. It seems like you code against a sub-type of Thinktomorrow\Chief\Management\Assistants\Assistant such as Thinktomorrow\Chief\Management\Assistants\ArchiveAssistant.

        return redirect()->to($manager->route('index'))->with('messages.success', $manager->details()->title .' is gearchiveerd.');

The property title does not exist on Thinktomorrow\Chief\Management\Details\Details. Since you implemented __get, consider adding a @property annotation.

    }

    public function unarchive($key, $id)
    {
        $manager = $this->managers->findByKey($key, $id);

        $manager->assistant('archive')
            ->guard('unarchive')
            ->unarchive();

The method unarchive() does not exist on Thinktomorrow\Chief\Management\Assistants\Assistant. It seems like you code against a sub-type of Thinktomorrow\Chief\Management\Assistants\Assistant such as Thinktomorrow\Chief\Management\Assistants\ArchiveAssistant.

        return redirect()->to($manager->route('index'))->with('messages.success', $manager->details()->title .' is hersteld.');

The property title does not exist on Thinktomorrow\Chief\Management\Details\Details. Since you implemented __get, consider adding a @property annotation.

    }
}