chuckbe / chuckcms

src/Controllers/FrontEndController.php

<?php

namespace Chuckbe\Chuckcms\Controllers;

use Chuckbe\Chuckcms\Chuck\PageBlockRepository;
use Chuckbe\Chuckcms\Models\Page;
use Chuckbe\Chuckcms\Models\PageBlock;
use Chuckbe\Chuckcms\Models\Redirect;
use Chuckbe\Chuckcms\Models\Repeater;
use Chuckbe\Chuckcms\Models\Template;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Routing\Controller as BaseController;
use Illuminate\Support\Facades\Auth;
use Spatie\Permission\Exceptions\UnauthorizedException;
use Spatie\Permission\Models\Role;

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

    private $page;
    private $pageblock;
    private $pageBlockRepository;
    private $redirect;
    private $repeater;
    private $role;

The private property $role is not used, and could be removed.

    private $template;

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct(
        Page $page,
        PageBlock $pageblock,
        PageBlockRepository $pageBlockRepository,
        Redirect $redirect,
        Repeater $repeater,
        Role $role,

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

        Template $template
    ) {
        $this->page = $page;
        $this->pageblock = $pageblock;
        $this->pageBlockRepository = $pageBlockRepository;
        $this->redirect = $redirect;
        $this->repeater = $repeater;
        $this->template = $template;
    }

    public function index($slug = null)
    {
        if ($slug == null) {
            $page = $this->page->where('isHp', 1)->firstOrFail();
        } elseif ($slug !== null) {
            $redirect = $this->redirect->where('slug', $slug)->first();
            if ($redirect !== null) {
                return redirect($redirect->to, $redirect->type);
            }

            $repeater = $this->repeater->where('url', $slug)->first();
            if ($repeater !== null) {
                $templateHintpath = explode('::', (string) $repeater->page)[0];
                $template = $this->template->where('active', 1)->where('hintpath', $templateHintpath)->first();

                return view((string) $repeater->page, compact('template', 'repeater'));
            }

            $page = $this->page->where('slug->'.app()->getLocale(), $slug)->first();

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

            if ($page == null) {
                foreach (\LaravelLocalization::getSupportedLocales() as $localeCode => $properties) {
                    $page = $this->page->where('slug->'.$localeCode, $slug)->first();
                    if ($page !== null && $localeCode == app()->getLocale()) {
                        break;
                    }

                    if ($page !== null && $localeCode !== app()->getLocale()) {
                        //dd(app()->getLocale());
                        app()->setLocale($localeCode);

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?

                        \LaravelLocalization::setLocale($localeCode);

The type LaravelLocalization was not found. Maybe you did not declare it correctly or list all dependencies?

                        return redirect($localeCode.'/'.$slug);
                    }
                }
            }

            if ($page == null) {
                abort(404);
            }
        }

        if ($page->roles !== null) {

The variable $page does not seem to be defined for all execution paths leading up to this point.

            return $this->authorizedIndex($page);
        }

        return $this->getView($page);
    }

    public function concept($slug = null)
    {
        if ($slug == null) {
            return redirect('/');
        }

        $page = $this->page->where('slug->'.app()->getLocale(), $slug)->first();
        if ($page == null) {
            foreach (\LaravelLocalization::getSupportedLocales() as $localeCode => $properties) {
                $page = $this->page->where('slug->'.$localeCode, $slug)->first();
                if ($page !== null && $localeCode == app()->getLocale()) {
                    break;
                }

                if ($page !== null && $localeCode !== app()->getLocale()) {
                    //dd(app()->getLocale());
                    app()->setLocale($localeCode);
                    \LaravelLocalization::setLocale($localeCode);

                    return redirect($localeCode.'/concept/'.$slug);
                }
            }
        }

        if ($page == null) {
            abort(404);
        }

        if ($page->roles !== null) {
            return $this->authorizedIndex($page);
        }

        return $this->getView($page);
    }

    public function authorizedIndex(Page $page)
    {
        if (!Auth::check()) {
            return redirect()->guest('login');
        }
        $roles = Role::whereIn('id', explode('|', $page->roles))->select('name')->get()->toArray();
        if (!Auth::user()->hasAnyRole($roles)) {

The method hasAnyRole() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

            throw UnauthorizedException::forRoles($roles);
        }

        return $this->getView($page);
    }

    public function getView(Page $page)
    {
        $ogpageblocks = $this->pageblock->getAllByPageId($page->id);
        $pageblocks = $this->pageBlockRepository->getRenderedByPageBlocks($ogpageblocks);

        if ($page->page !== null) {
            $template = $this->template->where('active', 1)->where('hintpath', explode('::', $page->page)[0])->first();

            return view($page->page, compact('template', 'page', 'pageblocks'));
        }

        $template = $this->template->where('active', 1)->where('id', $page->template_id)->first();

        return view($template->hintpath.'::templates.'.$template->slug.'.page', compact('template', 'page', 'pageblocks'));
    }

    public function css(Page $page)
    {
        $response = response()->make($page->css);
        $response->header('Content-Type', 'text/css');

        return $response;
    }

    public function js(Page $page)
    {
        $response = response()->make($page->js);
        $response->header('Content-Type', 'text/javascript');

        return $response;
    }
}