Chuckbe\Chuckcms\Controllers\ContentController

Complexity

Total Complexity

31

Size/Duplication

Total Lines	269
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

18 Methods

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

<?php

namespace Chuckbe\Chuckcms\Controllers;

use Chuckbe\Chuckcms\Models\Content;
use Chuckbe\Chuckcms\Models\Resource;
use Chuckbe\Chuckcms\Models\Repeater;
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\Http\Request;
use Illuminate\Support\Facades\View;

use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Routing\Controller as BaseController;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;

class ContentController extends BaseController
{
    use AuthorizesRequests, DispatchesJobs, 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));

It seems like $handle can also be of type false; however, parameter $handle of fputs() does only seem to accept resource, maybe add an additional type check?

        fclose($handle);

It seems like $handle can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

        $headers = array('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);
        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; 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 = array('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 = array('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 = array('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 = array('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 = array('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 = array('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 property slug does not exist on Chuckbe\Chuckcms\Models\Content. Since you implemented __get, consider adding a @property annotation.

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

        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;
    }
}