Jawaraegov\Workflows\Http\Controllers\WorkflowTransitionController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	101
Duplicated Lines	31.68 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

9 Methods

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

<?php

namespace Jawaraegov\Workflows\Http\Controllers;

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

class WorkflowTransitionController extends Controller
{
    
    public function index()
    {

        $transitions = WorkflowTransition::paginate(10);
        $state = array();

$state is not used, you could remove the assignment.

  
        return view('workflow.transition.index',compact('transitions'));
    }
    
    public function create()
    {
        $states = WorkflowState::all();
        
        return view('workflow.transition.create',compact('states'));
    }

    public function store(Request $request)
    {
        $name = str_replace('_','-',\Transliteration::clean_filename(strtolower($request->label)));
        $label = $request->label;
        $from = $request->from;
        $to = $request->to;
        $message = $request->message;

        try
        {
            WorkflowTransition::create([
                            'name' => $name,
                            'label' => $label,
                    'from' => $from,
                    'to' => $to,
                    'message' => $message
                ]);
        }
        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('transition')->with('message', 'Error');
        }

    	return redirect()->route('transition')->with('message', 'Add data success');
    }

    public function show($id)
    {
        //
    }

    public function edit($id)

This method seems to be duplicated in your project.

    {
        $transition = WorkflowTransition::where('id',$id)->first();
        $states = WorkflowState::all();

        return view('workflow.transitionedit',['transition' => $transition, 'states' => $states]);
    }

    public function update(Request $request, $id)

This method seems to be duplicated in your project.

    {
        $name = str_replace('_','-',\Transliteration::clean_filename(strtolower($request->label)));
        $label = $request->label;
        $from = $request->from;
        $to = $request->to;

        try
        {
          WorkflowTransition::where('id',$id)->update([
                                                      'label' => $label,
                                                      'name' => $name,
                                                      'from' => $from,
                                                      'to' => $to
                                                    ]);
        }
        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('transition')->with('message', 'Error');
        }



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

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

        return redirect()->back();
    }

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

        return redirect()->back();
    }

    public function destroy($id)
    {
        //
    }
}