DavideCasiraghi\LaravelEventsCalendar\Http\Controllers\RegionController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	236
Duplicated Lines	0 %

Test Coverage

Coverage

88.31%

Importance

Changes	9
Bugs	7	Features	2

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	update()	0	12	1
A	saveOnDb()	0	8	1
A	updateRegionsDropdown()	0	24	2
A	store()	0	18	2
A	create()	0	6	1
A	show()	0	3	1
A	index()	0	40	3
A	destroy()	0	6	1
A	edit()	0	6	1

<?php

namespace DavideCasiraghi\LaravelEventsCalendar\Http\Controllers;

use DavideCasiraghi\LaravelEventsCalendar\Models\Country;
use DavideCasiraghi\LaravelEventsCalendar\Models\Region;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Str;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization;
use Validator;

class RegionController extends Controller
{
    /* Restrict the access to this resource just to logged in users except show view */
    public function __construct()
    {
        $this->middleware('admin', ['except' => ['show', 'updateRegionsDropdown']]);
    }

    /***************************************************************************/

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\View\View
     */
    public function index(Request $request)
    {
        $countries = Country::getCountries();
        //$countries = Country::getActiveCountries();

        $searchKeywords = $request->input('keywords');
        $searchCountry = $request->input('country_id');

        if ($searchKeywords || $searchCountry) {
            //$regions = DB::table('regions')
            $regions = Region::
                        select('region_translations.region_id AS id', 'name', 'timezone', 'locale', 'country_id')
                        ->join('region_translations', 'regions.id', '=', 'region_translations.region_id')
                        ->where('locale', 'en')
                        ->when($searchKeywords, function ($query, $searchKeywords) {
                            return $query->where('name', $searchKeywords)->orWhere('name', 'like', '%'.$searchKeywords.'%');
                        })
                        ->when($searchCountry, function ($query, $searchCountry) {
                            return $query->where('country_id', '=', $searchCountry);
                        })
                        ->orderBy('name')
                        ->paginate(20);
        } else {
            $regions = Region::
                        select('region_translations.region_id AS id', 'name', 'timezone', 'locale', 'country_id')
                        ->join('region_translations', 'regions.id', '=', 'region_translations.region_id')
                        ->where('locale', 'en')
                        ->orderBy('name')
                        ->paginate(20);
        }

        // Countries available for translations
        $countriesAvailableForTranslations = LaravelLocalization::getSupportedLocales();

        return view('laravel-events-calendar::regions.index', compact('regions'))
            ->with('i', (request()->input('page', 1) - 1) * 20)
            ->with('countriesAvailableForTranslations', $countriesAvailableForTranslations)
            ->with('searchKeywords', $searchKeywords)
            ->with('searchCountry', $searchCountry)
            ->with('countries', $countries);
    }

    /***************************************************************************/

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\View\View
     */
    public function create()
    {
        $countries = Country::getCountries();

        return view('laravel-events-calendar::regions.create')
                ->with('countries', $countries);
    }

    /***************************************************************************/

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function store(Request $request)
    {
        // Validate form datas
        $validator = Validator::make($request->all(), [
            'name' => 'required',
            'country_id' => 'required',
            'timezone' => 'required',
        ]);
        if ($validator->fails()) {
            return back()->withErrors($validator)->withInput();
        }

        $region = new Region();

        $this->saveOnDb($request, $region);

        return redirect()->route('regions.index')
                        ->with('success', __('laravel-events-calendar::messages.region_added_successfully'));
    }

    /***************************************************************************/

    /**
     * Display the specified resource.
     *
     * @param  \DavideCasiraghi\LaravelEventsCalendar\Models\Region  $region
     * @return \Illuminate\View\View
     */
    public function show(Region $region)
    {
        return view('laravel-events-calendar::regions.show', compact('region'));
    }

    /***************************************************************************/

    /**
     * Show the form for editing the specified resource.
     *
     * @param  \DavideCasiraghi\LaravelEventsCalendar\Models\Region  $region
     * @return \Illuminate\View\View
     */
    public function edit(Region $region)
    {
        $countries = Country::getCountries();

        return view('laravel-events-calendar::regions.edit', compact('region'))
                    ->with('countries', $countries);
    }

    /***************************************************************************/

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \DavideCasiraghi\LaravelEventsCalendar\Models\Region  $region
     * @return \Illuminate\Http\RedirectResponse
     */
    public function update(Request $request, Region $region)
    {
        request()->validate([
            'name' => 'required',
            'country_id' => 'required',
            'timezone' => 'required',
        ]);

        $this->saveOnDb($request, $region);

        return redirect()->route('regions.index')
                        ->with('success', __('laravel-events-calendar::messages.region_updated_successfully'));
    }

    /***************************************************************************/

    /**
     * Remove the specified resource from storage.
     *
     * @param  \DavideCasiraghi\LaravelEventsCalendar\Models\Region  $region
     * @return \Illuminate\Http\RedirectResponse
     */
    public function destroy(Region $region)
    {
        $region->delete();

        return redirect()->route('regions.index')
                        ->with('success', __('laravel-events-calendar::messages.region_deleted_successfully'));
    }

    /***************************************************************************/

    /**
     * Return the single event region datas by cat id.
     *
     * @param  int $cat_id
     * @return \DavideCasiraghi\LaravelEventsCalendar\Models\Region
     */
    /*public function eventregiondata($cat_id)
    {
        $ret = DB::table('regions')->where('id', $cat_id)->first();
        //dump($ret);

        return $ret;
    }*/

    /***************************************************************************/

    /**
     * Save/Update the record on DB.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param \DavideCasiraghi\LaravelEventsCalendar\Models\Region $region
     * @return void
     */
    public function saveOnDb(Request $request, Region $region): void
    {
        $region->name = $request->get('name');

The property name does not seem to exist on DavideCasiraghi\LaravelEventsCalendar\Models\Region. Are you sure there is no database migration missing?

        $region->country_id = $request->get('country_id');
        $region->timezone = $request->get('timezone');
        $region->slug = Str::slug($region->name, '-');

The property slug does not seem to exist on DavideCasiraghi\LaravelEventsCalendar\Models\Region. Are you sure there is no database migration missing?

        $region->save();
    }

    /***************************************************************************/

    /**
     * Return and HTML with the updated regions dropdown for the homepage
     * after a country get selected.
     *
     * @param \Illuminate\Http\Request  $request
     * @return string $ret
     */
    public static function updateRegionsDropdown(Request $request)
    {
        /*$regions = Region::join('region_translations', 'regions.id', '=', 'region_translations.region_id')
                ->where('locale', 'en')
                ->where('country_id', $request->input('country_id'))
                ->orderBy('name')
                ->pluck('name','region_translations.region_id AS id'); */

        $regions = Region::
                select('name', 'region_translations.region_id AS id')
                ->join('region_translations', 'regions.id', '=', 'region_translations.region_id')
                ->where('locale', 'en')
                ->where('country_id', $request->input('country_id'))
                ->orderBy('name')
                ->get();

        // GENERATE the HTML to return
        $ret = "<select name='region_id' id='region_id' class='selectpicker' title='".__('homepage-serach.select_a_region')."'>";

Are you sure __('homepage-serach.select_a_region') of type array|string can be used in concatenation?

        foreach ($regions as $key => $region) {
            $ret .= "<option value='".$region->id."'>".$region->name.'</option>';
        }
        $ret .= '</select>';

        return response($ret, 200)->header('Content-Type', 'text/html');
    }

    /***************************************************************************/
}