Chuckbe\Chuckcms\Controllers\ContentController

Complexity

Total Complexity

31

Size/Duplication

Total Lines	286
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

18 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	resourceCreate()	0	3	1
A	repeaterIndex()	0	5	1
A	resourceDelete()	0	12	2
A	repeaterEntriesCreate()	0	5	1
A	repeaterEntriesIndex()	0	6	1
A	repeaterJson()	0	11	1
A	repeaterEntriesDelete()	0	6	1
A	resourceSave()	0	22	3
A	repeaterEntriesSave()	0	10	2
B	repeaterImport()	0	55	6
A	resourceIndex()	0	5	1
A	repeaterCreate()	0	5	1
A	repeaterDelete()	0	13	2
A	repeaterEntriesEdit()	0	6	1
A	repeaterEdit()	0	6	1
A	repeaterSave()	0	40	4
A	resourceEdit()	0	5	1

<?php

namespace Chuckbe\Chuckcms\Controllers;

use Chuckbe\Chuckcms\Models\Content;
use Chuckbe\Chuckcms\Models\Repeater;
use Chuckbe\Chuckcms\Models\Resource;
use Chuckbe\Chuckcms\Models\Template;
use Chuckbe\Chuckcms\Models\User;
use ChuckSite;

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

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;
use Illuminate\Support\Facades\View;

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

    private $content;
    private $resource;
    private $repeater;
    private $template;
    private $user;

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct(Content $content, Resource $resource, Repeater $repeater, Template $template, User $user)
    {
        $this->content = $content;
        $this->resource = $resource;
        $this->repeater = $repeater;
        $this->template = $template;
        $this->user = $user;
    }

    public function resourceIndex()
    {
        $resources = $this->resource->get();

        return view('chuckcms::backend.content.resource.index', compact('resources'));
    }

    public function resourceCreate()
    {
        return view('chuckcms::backend.content.resource.create');
    }

    public function resourceEdit($slug)
    {
        $resource = Resource::where('slug', $slug)->first();

        return view('chuckcms::backend.content.resource.edit', compact('resource'));
    }

    public function resourceSave(Request $request)
    {
        //validate the request
        $this->validate(request(), [//@todo create custom Request class for site validation
            'slug'             => 'required',
            'resource_key.*'   => 'required',
            'resource_value.*' => 'required',
        ]);

        $resource = Resource::firstOrNew(['slug' => $request->get('slug')[0]]);
        $resource->slug = $request->get('slug')[0];
        $json = [];
        foreach (ChuckSite::getSupportedLocales() as $langKey => $langValue) {
            $count = count($request->get('resource_key')[$langKey]);
            for ($i = 0; $i < $count; $i++) {
                $json[$langKey][$request->get('resource_key')[$langKey][$i]] = $request->get('resource_value')[$langKey][$i];
            }
        }
        $resource->json = $json;
        $resource->save();

        return redirect()->route('dashboard.content.resources');
    }

    public function resourceDelete(Request $request)
    {
        $this->validate(request(), [//@todo create custom Request class for site validation
            'resource_id' => 'required',
        ]);

        $resource = Resource::where('id', $request->get('resource_id'))->first();

        if ($resource->delete()) {
            return 'success';
        } else {
            return 'error';
        }
    }

    public function repeaterIndex()
    {
        $repeaters = $this->content->where('type', 'repeater')->get();

        return view('chuckcms::backend.content.repeater.index', compact('repeaters'));
    }

    public function repeaterCreate()
    {
        $pageViews = $this->template->getPageViews();

        return view('chuckcms::backend.content.repeater.create', compact('pageViews'));
    }

    public function repeaterEdit($slug)
    {
        $pageViews = $this->template->getPageViews();
        $repeater = Content::where('slug', $slug)->first();

        return view('chuckcms::backend.content.repeater.edit', compact('pageViews', 'repeater'));
    }

    public function repeaterJson($slug)
    {
        $repeater = Content::where('slug', $slug)->first();

        $filename = $repeater->slug.'.json';

The property slug does not exist on Chuckbe\Chuckcms\Models\Content. Since you implemented __get, consider adding a @property annotation.

        $handle = fopen($filename, 'w+');
        fputs($handle, $repeater->toJson(JSON_PRETTY_PRINT));
        fclose($handle);
        $headers = ['Content-type'=> 'application/json'];

        return response()->download($filename, $filename, $headers)->deleteFileAfterSend();

        //return view('chuckcms::backend.content.repeater.edit', compact('pageViews', 'repeater'));
    }

    public function repeaterSave(Request $request)
    {
        //add validation / move to repository...
        $content = [];
        $content_slug = $request->get('content_slug');
        $fields_slug = $request->get('fields_slug');
        $count = count($fields_slug);

It seems like $fields_slug can also be of type null; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

        for ($i = 0; $i < $count; $i++) {
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['label'] = $request->get('fields_label')[$i];
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['type'] = $request->get('fields_type')[$i];
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['class'] = $request->get('fields_class')[$i];
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['placeholder'] = $request->get('fields_placeholder')[$i];
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['validation'] = $request->get('fields_validation')[$i];
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['value'] = $request->get('fields_value')[$i];
            $fieldsCount = count(explode(';', $request->get('fields_attributes_name')[$i]));
            for ($k = 0; $k < $fieldsCount; $k++) {
                $content['fields'][$content_slug.'_'.$fields_slug[$i]]['attributes'][explode(';', $request->get('fields_attributes_name')[$i])[$k]] = explode(';', $request->get('fields_attributes_value')[$i])[$k];
            }
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['required'] = $request->get('fields_required')[$i];
            $content['fields'][$content_slug.'_'.$fields_slug[$i]]['table'] = $request->get('fields_table')[$i];
        }

        $content['actions']['store'] = $request->get('action_store');
        if ($request->get('action_detail') == 'true') {
            $content['actions']['detail']['url'] = $request->get('action_detail_url');
            $content['actions']['detail']['page'] = $request->get('action_detail_page');
        } else {
            $content['actions']['detail'] = 'false';
        }

        $content['files'] = $request->get('files_allowed');

        Content::updateOrCreate(
            ['id' => $request->get('content_id')],
            ['slug'       => $request->get('content_slug'),
                'type'    => $request->get('content_type'),
                'content' => $content, ]
        );

        return redirect()->route('dashboard.content.repeaters');
    }

    public function repeaterImport(Request $request)
    {
        $this->validate(request(), [//@todo create custom Request class for page validation
            'slug' => 'required',
            'file' => 'required|file|mimetypes:application/json,application/octet-stream,text/plain',
        ]);

        $file_contents = file_get_contents($request->file('file'));

It seems like $request->file('file') can also be of type Illuminate\Http\UploadedFile[] and array and null; however, parameter $filename of file_get_contents() does only seem to accept string, maybe add an additional type check?

        $new_slug = $request->get('slug');
        $old_slug = json_decode($file_contents, true)['slug'];

        $json_string = str_replace($old_slug, $new_slug, $file_contents);
        $json_file_array = json_decode($json_string, true);

        if (!array_key_exists('type', $json_file_array)) {
            $notification = ['type' => 'error', 'message' => 'The "type" key was not present in the JSON file.'];

            return redirect()->route('dashboard.content.repeaters')->with('notification', $notification);
        }

        if (!array_key_exists('content', $json_file_array)) {
            $notification = ['type' => 'error', 'message' => 'The "content" key was not present in the JSON file.'];

            return redirect()->route('dashboard.content.repeaters')->with('notification', $notification);
        }

        if (!array_key_exists('fields', $json_file_array['content'])) {
            $notification = ['type' => 'error', 'message' => 'The "fields" key was not present in the JSON file.'];

            return redirect()->route('dashboard.content.repeaters')->with('notification', $notification);
        }

        if (!array_key_exists('actions', $json_file_array['content'])) {
            $notification = ['type' => 'error', 'message' => 'The "actions" key was not present in the JSON file.'];

            return redirect()->route('dashboard.content.repeaters')->with('notification', $notification);
        }

        if (!array_key_exists('files', $json_file_array['content'])) {
            $notification = ['type' => 'error', 'message' => 'The "files" key was not present in the JSON file.'];

            return redirect()->route('dashboard.content.repeaters')->with('notification', $notification);
        }

        Content::updateOrCreate(
            ['id' => null],
            ['slug'       => $new_slug,
                'type'    => $json_file_array['type'],
                'content' => $json_file_array['content'], ]
        );

        $notification = ['type' => 'success', 'message' => 'The JSON file was successfully imported.'];

        return redirect()->route('dashboard.content.repeaters')->with('notification', $notification);
    }

    public function repeaterDelete(Request $request)
    {
        $this->validate(request(), [//@todo create custom Request class for site validation
            'content_id' => 'required',
        ]);

        $content = Content::where('id', $request->get('content_id'))->first();
        $repeaters = Repeater::where('slug', $content->slug)->delete();

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

The property slug does not exist on Chuckbe\Chuckcms\Models\Content. Since you implemented __get, consider adding a @property annotation.

        if ($content->delete()) {
            return 'success';
        } else {
            return 'error';
        }
    }

    public function repeaterEntriesIndex($slug)
    {
        $content = Content::where('slug', $slug)->first();
        $repeaters = $this->repeater->where('slug', $slug)->get();

        return view('chuckcms::backend.content.repeater.entries.index', compact('content', 'repeaters'));
    }

    public function repeaterEntriesCreate($slug)
    {
        $content = Content::where('slug', $slug)->first();

        return view('chuckcms::backend.content.repeater.entries.create', compact('content'));
    }

    public function repeaterEntriesSave(Request $request)
    {
        $slug = $request->get('content_slug');
        $content = $this->content->getBySlug($slug);
        $rules = $content->getRules();
        $this->validate(request(), $rules);
        $store = $content->storeEntry($request);
        if ($store == 'success') {
            return redirect()->route('dashboard.content.repeaters.entries', ['slug' => $slug]);
        } else {
            // error catching ... ?
        }
    }

    public function repeaterEntriesEdit($slug, $id)
    {
        $content = Content::where('slug', $slug)->first();
        $repeater = Repeater::where('id', $id)->first();

        return view('chuckcms::backend.content.repeater.entries.edit', compact('content', 'repeater'));
    }

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

        return $status;
    }
}