Bantenprov\Workflow\Http\Controllers\WorkflowStateController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	81
Duplicated Lines	43.21 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	6	1
A	create()	0	4	1
A	store()	19	19	2
A	show()	0	4	1
A	edit()	0	6	1
A	update()	16	16	2
A	Active()	0	5	1
A	DeActive()	0	5	1
A	destroy()	0	4	1

<?php

namespace Bantenprov\Workflow\Http\Controllers;

use App\Http\Controllers\Controller;
use Illuminate\Http\Request;
use Bantenprov\Workflow\Facades\Workflow;
use Bantenprov\Workflow\Models\WorkflowTransition;
use Bantenprov\Workflow\Models\WorkflowState;
use That0n3guy\Transliteration;

class WorkflowStateController extends Controller
{
    
    public function index()
    {
        $states = WorkflowState::paginate(10);
                        
        return view('workflow.state.index',compact('states'));
    }
    
    public function create()
    {
        return view('workflow.state.create');
    }
    
    public function store(Request $request)

This method seems to be duplicated in your project.

    {
        $name = \Transliteration::clean_filename(strtolower($request->label));
        $label = $request->label;

        try
        {
            WorkflowState::create([
                            'name' => $name,
                            'label' => $label,
                ]);
        }
        catch(\Illuminate\Database\QueryException $e){

The class Illuminate\Database\QueryException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            // do what you want here with $e->getMessage();
            return redirect()->route('state')->with('message', 'Error');
        }

        return redirect()->route('state')->with('message', 'Add data success');
    }
    
    public function show($id)
    {
        //
    }
    
    public function edit($id)
    {
        $state = WorkflowState::where('id',$id)->first();
        
        return view('workflow.state.edit',['state' => $state]);
    }
    
    public function update(Request $request, $id)

This method seems to be duplicated in your project.

    {
        $name = \Transliteration::clean_filename(strtolower($request->label));
        $label = $request->label;

        try
        {
          WorkflowState::where('id',$id)->update(['label' => $label, 'name' => $name]);
        }
        catch(\Illuminate\Database\QueryException $e){

The class Illuminate\Database\QueryException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            // do what you want here with $e->getMessage();
            return redirect()->route('state')->with('message', 'Error');
        }

        return redirect()->route('state')->with('message', 'Update data success');
    }

    public function Active($id){
        WorkflowState::where('id',$id)->update(['status' => 1]);

        return redirect()->back();
    }

    public function DeActive($id){
        WorkflowState::where('id',$id)->update(['status' => 0]);

        return redirect()->back();
    }
    
    public function destroy($id)
    {
        //
    }
}