App\Http\Controllers\CrudController::postGenerator()

Complexity

Conditions	18
Paths	> 20000

Size

Total Lines	104
Code Lines	53

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function postGenerator(Request $request)
    {
        $commandArg = [];
        $commandArg['name'] = $request->crud_name;


        /**
         * Make sure we do not already have this module
         */
        $directories = glob(base_path() . '/resources/views/*' , GLOB_ONLYDIR);
        foreach( $directories as $dir ){
            $this_name = str_replace(base_path() . '/resources/views/','',$dir);
            $used_names[] = $this_name;
        }
        //dd($request->crud_name);

        if(in_array(strtolower($request->crud_name),$used_names)){

The variable $used_names seems to be defined by a foreach iteration on line 48. Are you sure the iterator is never empty, otherwise this variable is not defined?

            return Redirect::back()->withErrors(['Name Exists or Restricted']);

The expression return Redirect::back()->withErrors(array('Name Exists or Restricted')) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Response.

        }





        if ($request->has('fields')) {
            $fieldsArray = [];
            $validationsArray = [];
            $x = 0;
            foreach ($request->fields as $field) {
                if ($request->fields_required[$x] == 1) {
                    $validationsArray[] = $field;
                }

                $fieldsArray[] = $field . '#' . $request->fields_type[$x];

                $x++;
            }

            $commandArg['--fields'] = implode(";", $fieldsArray);
        }

        if (!empty($validationsArray)) {
            $commandArg['--validations'] = implode("#required;", $validationsArray) . "#required";
        }

        if ($request->has('route')) {
            $commandArg['--route'] = $request->route;
        }

        if ($request->has('view_path')) {
            $commandArg['--view-path'] = $request->view_path;
        }

        if ($request->has('controller_namespace')) {
            $commandArg['--controller-namespace'] = $request->controller_namespace;
        }

        if ($request->has('model_namespace')) {
            $commandArg['--model-namespace'] = $request->model_namespace;
        }

        if ($request->has('route_group')) {
            $commandArg['--route-group'] = $request->route_group;
        }

        if ($request->has('relationships')) {
            $commandArg['--relationships'] = $request->relationships;
        }

        if ($request->has('form_helper')) {
            $commandArg['--form-helper'] = $request->form_helper;
        }

        if ($request->has('soft_deletes')) {
            $commandArg['--soft-deletes'] = $request->soft_deletes;
        }

        try {
            Artisan::call('crud:generate', $commandArg);

            $menus = json_decode(File::get(base_path('resources/laravel-admin/menus.json')));

            $name = $commandArg['name'];
            $routeName = ($commandArg['--route-group']) ? $commandArg['--route-group'] . '/' . snake_case($name, '-') : snake_case($name, '-');

            $menus->menus = array_map(function ($menu) use ($name, $routeName) {
                if ($menu->section == 'Resources') {
                    array_push($menu->items, (object) [
                        'title' => $name,
                        'url' => '/' . $routeName,
                    ]);
                }

                return $menu;
            }, $menus->menus);

            File::put(base_path('resources/laravel-admin/menus.json'), json_encode($menus));

            Artisan::call('migrate');
        } catch (\Exception $e) {
            return Response::make($e->getMessage(), 500);

The expression return Response::make($e->getMessage(), 500) returns the type Illuminate\Http\Response which is incompatible with the documented return type Response.

        }

        return redirect('crud')->with('flash_message', 'Your CRUD has been generated. See on the menu.');

The expression return redirect('crud')->with('flash_message', 'Your CRUD has been generated. See on the menu.') returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Response.

    }
}