CurrencyController::update() - Code Metrics - Inspection of "Adds currencies " - pwweb/localisation - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ea1130...b24b06 )

by F

created 2020-04-10 08:41 UTC

CurrencyController::update() A

↳ Parent: CurrencyController

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	7
c	1
b	0
f	0
nc	2
nop	2
dl	0
loc	15
rs	10

<?php

namespace PWWEB\Localisation\Controllers;

use App\Http\Controllers\Controller;
use Flash;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Http\Request;
use PWWEB\Localisation\Repositories\CurrencyRepository;
use PWWEB\Localisation\Requests\CreateCurrencyRequest;
use PWWEB\Localisation\Requests\UpdateCurrencyRequest;

/**
 * PWWEB\Localisation\Controllers\CurrencyController CurrencyController.
 *
 * The CRUD controller for Currency
 * Class CurrencyController
 *
 * @package   pwweb/localisation
 * @author    Frank Pillukeit <[email protected]>
 * @author    Richard Browne <[email protected]
 * @copyright 2020 pw-websolutions.com
 * @license   http://www.opensource.org/licenses/mit-license.html  MIT License
*/
class CurrencyController extends Controller
{
    /** @var  CurrencyRepository */
    private $currencyRepository;

    public function __construct(CurrencyRepository $currencyRepo)
    {
        $this->currencyRepository = $currencyRepo;
    }

    /**
     * Display a listing of the Currency.
     *
     * @param Request $request
     *
     * @return \Illuminate\View\View
     */
    public function index(Request $request)
    {
        $currencies = $this->currencyRepository->all();

        return view('localisation::currencies.index')
            ->with('currencies', $currencies);
    }

    /**
     * Show the form for creating a new Currency.
     *
     * @return \Illuminate\View\View
     */
    public function create()
    {
        return view('localisation::currencies.create');
    }

    /**
     * Store a newly created Currency in storage.
     *
     * @param CreateCurrencyRequest $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function store(CreateCurrencyRequest $request)
    {
        $input = $request->all();

        $currency = $this->currencyRepository->create($input);


        Flash::success('Currency saved successfully.');

        return redirect(route('localisation.currencies.index'));
    }

    /**
     * Display the specified Currency.
     *
     * @param int $id
     *
     * @return \Illuminate\View\View|\Illuminate\Http\RedirectResponse
     */
    public function show($id)
    {
        $currency = $this->currencyRepository->find($id);

        if (empty($currency)) {
            Flash::error('Currency not found');

            return redirect(route('localisation.currencies.index'));
        }

        return view('localisation::currencies.show')->with('currency', $currency);
    }

    /**
     * Show the form for editing the specified Currency.
     *
     * @param int $id
     *
     * @return \Illuminate\View\View|\Illuminate\Http\RedirectResponse
     */
    public function edit($id)
    {
        $currency = $this->currencyRepository->find($id);

        if (empty($currency)) {
            Flash::error('Currency not found');

            return redirect(route('localisation.currencies.index'));
        }

        return view('localisation::currencies.edit')->with('currency', $currency);
    }

    /**
     * Update the specified Currency in storage.
     *
     * @param int $id
     * @param UpdateCurrencyRequest $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function update($id, UpdateCurrencyRequest $request)
    {
        $currency = $this->currencyRepository->find($id);

        if (empty($currency)) {
            Flash::error('Currency not found');

            return redirect(route('localisation.currencies.index'));
        }

        $currency = $this->currencyRepository->update($request->all(), $id);


        Flash::success('Currency updated successfully.');

        return redirect(route('localisation.currencies.index'));
    }

    /**
     * Remove the specified Currency from storage.
     *
     * @param int $id
     *
     * @throws \Exception
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function destroy($id)
    {
        $currency = $this->currencyRepository->find($id);

        if (empty($currency)) {
            Flash::error('Currency not found');

            return redirect(route('localisation.currencies.index'));
        }

        $this->currencyRepository->delete($id);

        Flash::success('Currency deleted successfully.');

        return redirect(route('localisation.currencies.index'));
    }
}


1			<?php
2
3			namespace PWWEB\Localisation\Controllers;
4
5			use App\Http\Controllers\Controller;
6			use Flash;
			0 ignored issues – show Bug introduced 2020-04-10 08:44 UTC by Report Bug Copy Issue Report The type `Flash` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
7			use Illuminate\Http\Request;
8			use PWWEB\Localisation\Repositories\CurrencyRepository;
9			use PWWEB\Localisation\Requests\CreateCurrencyRequest;
10			use PWWEB\Localisation\Requests\UpdateCurrencyRequest;
11
12			/**
13			* PWWEB\Localisation\Controllers\CurrencyController CurrencyController.
14			*
15			* The CRUD controller for Currency
16			* Class CurrencyController
17			*
18			* @package pwweb/localisation
19			* @author Frank Pillukeit <[email protected]>
20			* @author Richard Browne <[email protected]
21			* @copyright 2020 pw-websolutions.com
22			* @license http://www.opensource.org/licenses/mit-license.html MIT License
23			*/
24			class CurrencyController extends Controller
25			{
26			/** @var CurrencyRepository */
27			private $currencyRepository;
28
29			public function __construct(CurrencyRepository $currencyRepo)
30			{
31			$this->currencyRepository = $currencyRepo;
32			}
33
34			/**
35			* Display a listing of the Currency.
36			*
37			* @param Request $request
38			*
39			* @return \Illuminate\View\View
40			*/
41			public function index(Request $request)
42			{
43			$currencies = $this->currencyRepository->all();
44
45			return view('localisation::currencies.index')
46			->with('currencies', $currencies);
47			}
48
49			/**
50			* Show the form for creating a new Currency.
51			*
52			* @return \Illuminate\View\View
53			*/
54			public function create()
55			{
56			return view('localisation::currencies.create');
57			}
58
59			/**
60			* Store a newly created Currency in storage.
61			*
62			* @param CreateCurrencyRequest $request
63			*
64			* @return \Illuminate\Http\RedirectResponse
65			*/
66			public function store(CreateCurrencyRequest $request)
67			{
68			$input = $request->all();
69
70			$currency = $this->currencyRepository->create($input);
			0 ignored issues – show Unused Code introduced 2020-04-10 08:44 UTC by Report Bug Copy Issue Report The assignment to `$currency` is dead and can be removed. Loading history...
71
72			Flash::success('Currency saved successfully.');
73
74			return redirect(route('localisation.currencies.index'));
75			}
76
77			/**
78			* Display the specified Currency.
79			*
80			* @param int $id
81			*
82			* @return \Illuminate\View\View\|\Illuminate\Http\RedirectResponse
83			*/
84			public function show($id)
85			{
86			$currency = $this->currencyRepository->find($id);
87
88			if (empty($currency)) {
89			Flash::error('Currency not found');
90
91			return redirect(route('localisation.currencies.index'));
92			}
93
94			return view('localisation::currencies.show')->with('currency', $currency);
95			}
96
97			/**
98			* Show the form for editing the specified Currency.
99			*
100			* @param int $id
101			*
102			* @return \Illuminate\View\View\|\Illuminate\Http\RedirectResponse
103			*/
104			public function edit($id)
105			{
106			$currency = $this->currencyRepository->find($id);
107
108			if (empty($currency)) {
109			Flash::error('Currency not found');
110
111			return redirect(route('localisation.currencies.index'));
112			}
113
114			return view('localisation::currencies.edit')->with('currency', $currency);
115			}
116
117			/**
118			* Update the specified Currency in storage.
119			*
120			* @param int $id
121			* @param UpdateCurrencyRequest $request
122			*
123			* @return \Illuminate\Http\RedirectResponse
124			*/
125			public function update($id, UpdateCurrencyRequest $request)
126			{
127			$currency = $this->currencyRepository->find($id);
128
129			if (empty($currency)) {
130			Flash::error('Currency not found');
131
132			return redirect(route('localisation.currencies.index'));
133			}
134
135			$currency = $this->currencyRepository->update($request->all(), $id);
			0 ignored issues – show Unused Code introduced 2020-04-10 08:44 UTC by Report Bug Copy Issue Report The assignment to `$currency` is dead and can be removed. Loading history...
136
137			Flash::success('Currency updated successfully.');
138
139			return redirect(route('localisation.currencies.index'));
140			}
141
142			/**
143			* Remove the specified Currency from storage.
144			*
145			* @param int $id
146			*
147			* @throws \Exception
148			*
149			* @return \Illuminate\Http\RedirectResponse
150			*/
151			public function destroy($id)
152			{
153			$currency = $this->currencyRepository->find($id);
154
155			if (empty($currency)) {
156			Flash::error('Currency not found');
157
158			return redirect(route('localisation.currencies.index'));
159			}
160
161			$this->currencyRepository->delete($id);
162
163			Flash::success('Currency deleted successfully.');
164
165			return redirect(route('localisation.currencies.index'));
166			}
167			}
168

pwweb / localisation

Push — master ( ea1130...b24b06 )

CurrencyController::update() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like