ijeffro / laralocker

src/Http/Controllers/JourneyController.php

<?php

namespace Ijeffro\Laralocker\Http\Controllers;

use Illuminate\Http\Request;
use Ijeffro\Laralocker\Facades\LearningLocker;

class JourneyController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        return LearningLocker::journeys()->get();

The method journeys() does not exist on Ijeffro\LaraLocker\Facades\LearningLocker. Since you implemented __callStatic, consider adding a @method annotation.

    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        return LearningLocker::journey($id)->get();

The method journey() does not exist on Ijeffro\LaraLocker\Facades\LearningLocker. Since you implemented __callStatic, consider adding a @method annotation.

    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function save(Request $request)

The parameter $request is not used and could be removed.

    {
        $data = request()->all();
        return LearningLocker::journey()->create($data);
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)

The parameter $request is not used and could be removed.

    {
        $data = request()->all();
        return LearningLocker::journey($id)->update($data);
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    {
        return LearningLocker::journey($id)->delete();
    }

}