familytree365 / backend

app/Http/Controllers/PersonNameFoneController.php

<?php

namespace App\Http\Controllers;

use App\Models\PersonNameFone;
use Illuminate\Http\Request;

class PersonNameFoneController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index(Request $request)
    {
        $query = PersonNameFone::query();

        if ($request->has('searchTerm')) {
            $columnsToSearch = ['name', 'email', 'phone'];
            $search_term = json_decode($request->searchTerm)->searchTerm;
            if (! empty($search_term)) {
                $searchQuery = '%'.$search_term.'%';
                foreach ($columnsToSearch as $column) {
                    $query->orWhere($column, 'LIKE', $searchQuery);
                }
            }
        }

        if ($request->has('columnFilters')) {
            $filters = get_object_vars(json_decode($request->columnFilters));

            foreach ($filters as $key => $value) {
                if (! empty($value)) {
                    $query->orWhere($key, 'like', '%'.$value.'%');
                }
            }
        }

        if ($request->has('sort.0')) {
            $sort = json_decode($request->sort[0]);
            $query->orderBy($sort->field, $sort->type);
        }

        if ($request->has('perPage')) {
            $rows = $query->paginate($request->perPage);
        }

        return $rows;

The variable $rows does not seem to be defined for all execution paths leading up to this point.

    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        //
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        $request->validate([
            'group' => 'required',
            'gid' => 'required',
            'type' => 'required',
            'name' => 'required',
            'npfx' => 'required',
            'givn' => 'required',
            'nick' => 'required',
            'spfx' => 'required',
            'surn' => 'required',
            'nsfx' => 'required',
        ]);

        return PersonNameFone::create([

The expression return App\Models\PersonNameFone::create(array('group' => $request->group, 'gid' => $request->gid, 'type' => $request->type, 'name' => $request->name, 'npfx' => $request->npfx, 'givn' => $request->givn, 'nick' => $request->nick, 'spfx' => $request->spfx, 'surn' => $request->surn, 'nsfx' => $request->nsfx)) also could return the type App\Models\PersonNameFone which is incompatible with the documented return type Illuminate\Http\Response.

            'group' => $request->group,
            'gid' => $request->gid,
            'type' => $request->type,
            'name' => $request->name,
            'npfx' => $request->npfx,
            'givn' => $request->givn,
            'nick' => $request->nick,
            'spfx' => $request->spfx,
            'surn' => $request->surn,
            'nsfx' => $request->nsfx,
        ]);
    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        return PersonNameFone::find($id);

The expression return App\Models\PersonNameFone::find($id) also could return the type App\Models\PersonNameFone which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function edit($id)

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

    {
        //
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)
    {
        $request->validate([
            'group' => 'required',
            'gid' => 'required',
            'type' => 'required',
            'name' => 'required',
            'npfx' => 'required',
            'givn' => 'required',
            'nick' => 'required',
            'spfx' => 'required',
            'surn' => 'required',
            'nsfx' => 'required',
        ]);

        $personnamefone = PersonNameFone::find($id);
        $personnamefone->group = $request->group;
        $personnamefone->gid = $request->gid;
        $personnamefone->type = $request->type;
        $personnamefone->name = $request->name;
        $personnamefone->npfx = $request->npfx;
        $personnamefone->givn = $request->givn;
        $personnamefone->nick = $request->nick;
        $personnamefone->spfx = $request->spfx;
        $personnamefone->surn = $request->surn;
        $personnamefone->nsfx = $request->nsfx;
        $personnamefone->save();

        return $personnamefone;

The expression return $personnamefone also could return the type App\Models\PersonNameFone which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    {
        $personnamefonefone = PersonNameFone::find($id);
        if ($personnamefonefone) {
            $personnamefonefone->delete();

            return 'true';

The expression return 'true' returns the type string which is incompatible with the documented return type Illuminate\Http\Response.

        }

        return 'false';

The expression return 'false' returns the type string which is incompatible with the documented return type Illuminate\Http\Response.

    }
}