Issues (256)

src/Http/Controllers/ClientController.php (2 issues)

Severity
1
<?php
2
3
namespace Ijeffro\Laralocker\Http\Controllers;
4
5
use LearningLocker;
6
use Illuminate\Http\Request;
7
8
class ClientController extends Controller
9
{
10
    /**
11
     * Display a listing of the resource.
12
     *
13
     * @return LearningLocker
14
     */
15
    public function index()
16
    {
17
        return LearningLocker::clients()->get();
18
    }
19
20
    /**
21
     * Display the specified resource.
22
     *
23
     * @param  int  $id
24
     * @return LearningLocker
25
     */
26
    public function show($id)
27
    {
28
        $select = ["_id"];
29
        return LearningLocker::client($id)->get($select);
30
    }
31
32
    /**
33
     * Store a newly created resource in storage.
34
     *
35
     * @param  \Illuminate\Http\Request  $request
36
     * @return LearningLocker
37
     */
38
    public function save(Request $request)
0 ignored issues
show
The parameter $request is not used and could be removed. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-unused  annotation

38
    public function save(/** @scrutinizer ignore-unused */ Request $request)

This check looks for parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
39
    {
40
        $data = request()->all();
41
        return LearningLocker::client()->create($data);
42
    }
43
44
    /**
45
     * Update the specified resource in storage.
46
     *
47
     * @param  \Illuminate\Http\Request $request
48
     * @param  int  $id
49
     * @return LearningLocker
50
     */
51
    public function update(Request $request, $id)
0 ignored issues
show
The parameter $request is not used and could be removed. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-unused  annotation

51
    public function update(/** @scrutinizer ignore-unused */ Request $request, $id)

This check looks for parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
52
    {
53
        $data = request()->all();
54
        return LearningLocker::client($id)->update($data);
55
    }
56
57
    /**
58
     * Remove the specified resource from storage.
59
     *
60
     * @param  int  $id
61
     * @return LearningLocker
62
     */
63
    public function destroy($id)
64
    {
65
        return LearningLocker::client($id)->delete();
66
    }
67
68
}
69