Jawaraegov\Workflows\Traits\WorkflowTransitionTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	91
Duplicated Lines	35.16 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	transitionIndex()	0	8	1
A	transitionCreate()	0	6	1
A	transitionEdit()	7	7	1
A	transitionUpdate()	25	25	2
A	transitionActive()	0	5	1
A	transitionDeActive()	0	5	1
A	transitionStore()	0	25	2

<?php

namespace Jawaraegov\Workflows\Traits;

use Illuminate\Database\Eloquent\Model;
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;

trait WorkflowTransitionTrait
{
    public static function transitionIndex($page = 10)
    {

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

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

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

    public static function transitionStore($request = array())
    {
        $name = str_replace('_','-',\Transliteration::clean_filename(strtolower($req->label)));

The variable $req does not exist. Did you forget to declare it?

        $label = $request->label;
        $from = join(',',$request->from);
        $to = join(',',$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 static function transitionEdit($id)

This method seems to be duplicated in your project.

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

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

    public static function transitionUpdate($request = array(), $id)

This method seems to be duplicated in your project.

    {
        $name = str_replace('_','-',\Transliteration::clean_filename(strtolower($request->label)));
        $label = $request->label;
        $from = join(',',$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 static function transitionActive($id){
        WorkflowTransition::where('id',$id)->update(['status' => 1]);

        return redirect()->back();
    }

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

        return redirect()->back();
    }

}