Chuckbe\Chuckcms\Controllers\PageBlockController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	145
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	show()	0	7	1
A	moveDown()	0	6	1
A	addBlockTop()	0	12	2
A	delete()	0	6	1
A	addBlockBottom()	0	12	2
A	update()	0	7	1
A	moveUp()	0	6	1

<?php

namespace Chuckbe\Chuckcms\Controllers;

use Chuckbe\Chuckcms\Chuck\PageBlockRepository;
use Chuckbe\Chuckcms\Models\Page;
use Chuckbe\Chuckcms\Models\PageBlock;
use Chuckbe\Chuckcms\Models\Repeater;
use Chuckbe\Chuckcms\Models\Resource;
use Chuckbe\Chuckcms\Models\Template;
use File;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Http\Request;
use Illuminate\Routing\Controller as BaseController;

class PageBlockController extends BaseController
{
    use AuthorizesRequests;
    use DispatchesJobs;
    use ValidatesRequests;

    protected $template;
    protected $page;
    protected $pageblock;
    protected $pageBlockRepository;
    protected $resource;
    protected $repeater;

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct(Template $template, Page $page, PageBlock $pageblock, PageBlockRepository $pageBlockRepository, Resource $resource, Repeater $repeater)
    {
        $this->template = $template;
        $this->page = $page;
        $this->pageblock = $pageblock;
        $this->pageBlockRepository = $pageBlockRepository;
        $this->resource = $resource;
        $this->repeater = $repeater;
    }

    /**
     * Get rendered pageblock html as a string.
     *
     * @param Request $request
     *
     * @return array $pageblock
     */
    public function show(Request $request)
    {
        // AUTHORIZE ... COMES HERE
        $pb = $this->pageblock->where('id', $request->get('pageblock_id'))->first();

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

        $pageblock = $this->pageBlockRepository->getRenderedByPageBlock($pageblock);

The variable $pageblock seems to be never defined.

        return $pageblock;
    }

    /**
     * Show the application dashboard.
     *
     * @param Request $request
     *
     * @return array $pageblock
     */
    public function update(Request $request)
    {
        // AUTHORIZE ... COMES HERE
        $pageblock = $this->pageblock->getById($request->get('pageblock_id'));
        $pageblock = $this->pageBlockRepository->updateBody($pageblock, $request->get('html'));

        return $pageblock;
    }

    /**
     * Move the resource one place up.
     *
     * @param Request $request
     *
     * @return array $pageblock
     */
    public function moveUp(Request $request)
    {
        // AUTHORIZE ... COMES HERE
        $pageblock = $this->pageBlockRepository->moveUpById($request->get('pageblock_id'));

        return $pageblock;
    }

    /**
     * Move the resource one place down.
     *
     * @param Request $request
     *
     * @return array $pageblock
     */
    public function moveDown(Request $request)
    {
        // AUTHORIZE ... COMES HERE
        $pageblock = $this->pageBlockRepository->moveDownById($request->get('pageblock_id'));

        return $pageblock;
    }

    /**
     * Delete the resource from the page.
     *
     * @param Request $request
     *
     * @return string $tatus
     */
    public function delete(Request $request)
    {
        // AUTHORIZE ... COMES HERE
        $status = $this->pageBlockRepository->deleteById($request->get('pageblock_id'));

        return $status;
    }

    /**
     * Add Block From Location To Top of Page and Store to Database.
     *
     * @param Request $request
     *
     * @return string
     */
    public function addBlockTop(Request $request)
    {
        if ($request->has('lang')) {
            app()->setLocale($request->get('lang'));

The method setLocale() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        }

        // AUTHORIZE ... COMES HERE
        $contents = File::get($request['location']);
        $page = $this->page->getById($request['page_id']);
        $this->pageblock->addBlockTop($contents, $page, $request['name']);
        //return $pageblock;
        return 'success';
    }

    /**
     * Add Block From Location To Bottom of Page and Store to Database.
     *
     * @param Request $request
     *
     * @return string
     */
    public function addBlockBottom(Request $request)
    {
        if ($request->has('lang')) {
            app()->setLocale($request->get('lang'));
        }

        // AUTHORIZE ... COMES HERE
        $contents = File::get($request['location']);
        $page = $this->page->getById($request['page_id']);
        $this->pageblock->addBlockBottom($contents, $page, $request['name']);
        //return $pageblock;
        return 'success';
    }
}