Bantenprov\Sktm\Http\Controllers\SktmController

Complexity

Total Complexity

21

Size/Duplication

Total Lines	253
Duplicated Lines	28.85 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	create()	0	19	3
B	index()	0	33	6
B	store()	25	42	3
A	show()	11	11	1
A	edit()	0	14	1
A	update()	25	54	4
A	destroy()	12	12	2

<?php

namespace Bantenprov\Sktm\Http\Controllers;

/* Require */
use App\Http\Controllers\Controller;
use Illuminate\Http\Request;
use Bantenprov\Sktm\Facades\SktmFacade;

/* Models */
use Bantenprov\Sktm\Models\Bantenprov\Sktm\Sktm;
use Bantenprov\Sktm\Models\Bantenprov\Sktm\MasterSktm;
use App\User;

/* Etc */
use Validator;

/**
 * The SktmController class.
 *
 * @package Bantenprov\Sktm
 * @author  bantenprov <developer.bantenprov@gmail.com>
 */
class SktmController extends Controller
{
    /**
     * Create a new controller instance.
     *
     * @return void
     */
    protected $sktm;
    protected $master_sktm;
    protected $user;

    public function __construct(Sktm $sktm, MasterSktm $master_sktm, User $user)
    {
        $this->sktm = $sktm;
        $this->master_sktm = $master_sktm;
        $this->user = $user;
    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index(Request $request)
    {
        if (request()->has('sort')) {
            list($sortCol, $sortDir) = explode('|', request()->sort);

            $query = $this->sktm->orderBy($sortCol, $sortDir);
        } else {
            $query = $this->sktm->orderBy('id', 'asc');
        }

        if ($request->exists('filter')) {
            $query->where(function($q) use($request) {
                $value = "%{$request->filter}%";
                $q->where('nomor_un', 'like', $value)
                    ->orWhere('nilai', 'like', $value);
            });
        }

        $perPage = request()->has('per_page') ? (int) request()->per_page : null;
        $response = $query->paginate($perPage);

        foreach($response as $master_sktm){
            array_set($response->data, 'master_sktm', $master_sktm->master_sktm->nama);
        }

        foreach($response as $user){
            array_set($response->data, 'user', $user->user->name);
        }

        return response()->json($response)
            ->header('Access-Control-Allow-Origin', '*')
            ->header('Access-Control-Allow-Methods', 'GET');
    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */

    public function create()
    {
        $users = $this->user->all();
        $master_sktms = $this->master_sktm->all();

        foreach($users as $user){
            array_set($user, 'label', $user->name);
        }

        foreach($master_sktms as $master_sktm){
            array_set($master_sktm, 'label', $master_sktm->nama);
        }
        
        $response['master_sktm'] = $master_sktms;

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        $response['user'] = $users;
        $response['status'] = true;

        return response()->json($response);
    }

    /**
     * Display the specified resource.
     *
     * @param  \App\Sktm  $sktm

There is no parameter named $sktm. Was it maybe removed?

     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        $sktm = $this->sktm;

        $validator = Validator::make($request->all(), [
            'user_id' => 'required|unique:sktms,user_id',
            'nomor_un' => 'required',
            'master_sktm_id' => 'required',
            'no_sktm' => 'required',
            'nilai' => 'required',
        ]);

        if($validator->fails()){

This code seems to be duplicated across your project.

            $check = $sktm->where('user_id',$request->user_id)->whereNull('deleted_at')->count();

            if ($check > 0) {
                $response['message'] = 'Failed, Username ' . $request->user_id . ' already exists';

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

            } else {
                $sktm->user_id = $request->input('user_id');
                $sktm->nomor_un = $request->input('nomor_un');
                $sktm->master_sktm_id = $request->input('master_sktm_id');
                $sktm->no_sktm = $request->input('no_sktm');
                $sktm->nilai = $request->input('nilai');
                $sktm->save();

                $response['message'] = 'success';

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

            }
        } else {
                $sktm->user_id = $request->input('user_id');
                $sktm->nomor_un = $request->input('nomor_un');
                $sktm->master_sktm_id = $request->input('master_sktm_id');
                $sktm->no_sktm = $request->input('no_sktm');
                $sktm->nilai = $request->input('nilai');
                $sktm->save();

            $response['message'] = 'success';

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        }

        $response['status'] = true;

        return response()->json($response);
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request

There is no parameter named $request. Was it maybe removed?

     * @return \Illuminate\Http\Response
     */
    public function show($id)

This method seems to be duplicated in your project.

    {
        $sktm = $this->sktm->findOrFail($id);
        
        $response['user'] = $sktm->user;

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        $response['master_sktm'] = $sktm->master_sktm;
        $response['sktm'] = $sktm;
        $response['status'] = true;

        return response()->json($response);
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  \App\Sktm  $sktm
     * @return \Illuminate\Http\Response

There is no parameter named $sktm. Was it maybe removed?

     */

    public function edit($id)
    {
        $sktm = $this->sktm->findOrFail($id);

        array_set($sktm->user, 'label', $sktm->user->name);
        array_set($sktm->master_sktm, 'label', $sktm->master_sktm->nama);
        
        $response['master_sktm'] = $sktm->master_sktm;

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        $response['sktm'] = $sktm;
        $response['user'] = $sktm->user;
        $response['status'] = true;

        return response()->json($response);
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \App\Sktm  $sktm

There is no parameter named $sktm. Was it maybe removed?

     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)
    {
        $sktm = $this->sktm->findOrFail($id);

        if ($request->input('master_sktm_id') == $request->input('master_sktm_id'))
        {
            $validator = Validator::make($request->all(), [
                'user_id' => 'required|unique:sktms,user_id',
            'nomor_un' => 'required',
            'master_sktm_id' => 'required',
            'no_sktm' => 'required',
            'nilai' => 'required',
                
            ]);
        } else {
            $validator = Validator::make($request->all(), [
                'user_id' => 'required|unique:sktms,user_id',
            'nomor_un' => 'required',
            'master_sktm_id' => 'required',
            'no_sktm' => 'required',
            'nilai' => 'required',
            ]);
        }

        if ($validator->fails()) {

This code seems to be duplicated across your project.

            $check = $sktm->where('user_id',$request->user_id)->whereNull('deleted_at')->count();

            if ($check > 0) {
                $response['message'] = 'Failed, Master SKTM ' . $request->user_id . ' already exists';

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

            } else {
                $sktm->user_id = $request->input('user_id');
                $sktm->nomor_un = $request->input('nomor_un');
                $sktm->master_sktm_id = $request->input('master_sktm_id');
                $sktm->no_sktm = $request->input('no_sktm');
                $sktm->nilai = $request->input('nilai');
                $sktm->save();

                $response['message'] = 'success';

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

            }
        } else {
                $sktm->user_id = $request->input('user_id');
                $sktm->nomor_un = $request->input('nomor_un');
                $sktm->master_sktm_id = $request->input('master_sktm_id');
                $sktm->no_sktm = $request->input('no_sktm');
                $sktm->nilai = $request->input('nilai');
                $sktm->save();

            $response['message'] = 'success';

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        }

        $response['status'] = true;

        return response()->json($response);
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  \App\Sktm  $sktm

There is no parameter named $sktm. Was it maybe removed?

     * @return \Illuminate\Http\Response
     */
    public function destroy($id)

This method seems to be duplicated in your project.

    {
        $sktm = $this->sktm->findOrFail($id);

        if ($sktm->delete()) {
            $response['status'] = true;

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        } else {
            $response['status'] = false;

$response was never initialized. Although not strictly required by PHP, it is generally a good practice to add $response = array(); before regardless.

        }

        return json_encode($response);
    }
}