academico-sis / academico

app/Http/Controllers/ContactController.php

<?php

namespace App\Http\Controllers;

use App\Http\Requests\ContactRequest as StoreRequest;
use App\Models\Contact;
use App\Models\PhoneNumber;
use App\Models\Profession;
use Illuminate\Http\Request;
use Prologue\Alerts\Facades\Alert;

class ContactController extends Controller
{
    public function __construct()
    {
        parent::__construct();
        $this->middleware(backpack_middleware());
    }

    /**
     * Store a newly created resource in storage.
     */
    public function store(StoreRequest $request)
    {
        $request->validate([
            'firstname' => 'required',
            'lastname' => 'required',
            'email' => 'required',
            'address' => 'required',
            'phone_number' => 'required',
            'idnumber' => 'required',
            'contact_type' => 'required',
        ]);

        $contact = new Contact();
        $contact->student_id = $request->input('student_id');
        $contact->firstname = $request->input('firstname');
        $contact->lastname = $request->input('lastname');
        $contact->idnumber = $request->input('idnumber');
        $contact->address = $request->input('address');
        $contact->email = $request->input('email');
        $contact->relationship_id = $request->input('contact_type');
        $contact->save();

        // register the phone number
        if ($request->input('phone_number') !== null) {
            $phone = new PhoneNumber();
            $phone->phoneable_id = $contact->id;
            $phone->phoneable_type = Contact::class;
            $phone->phone_number = $request->input('phone_number');
            $phone->save();
        }

        Alert::success(__('The information has successfully been saved'))->flash();

It seems like __('The information has successfully been saved') can also be of type array and array; however, parameter $text of Prologue\Alerts\Facades\Alert::success() does only seem to accept string, maybe add an additional type check?

        return redirect()->back();
    }

    public function update(Contact $contact, Request $request)
    {
        // check if the user is allowed to edit the contact
        if (! backpack_user()->can('update', $contact)) {
            abort(403);
        }

        $request->validate([
            'firstname' => 'required|max:255',
            'lastname' => 'required|max:255',
            'email' => 'required',
        ]);

        $contact->update([
            'firstname' => $request->firstname,
            'lastname' => $request->lastname,
            'idnumber' => $request->idnumber,
            'address' => $request->address,
            'email' => $request->email,
            'relationship_id' => $request->contact_type,
        ]);

        if ($request->profession) {
            $profession = Profession::firstOrCreate([
                'name' => $request->profession,
            ]);

            $contact->update([
                'profession_id' => $profession->id,
            ]);
        }

        Alert::success(__('The information has successfully been saved'))->flash();

It seems like __('The information has successfully been saved') can also be of type array and array; however, parameter $text of Prologue\Alerts\Facades\Alert::success() does only seem to accept string, maybe add an additional type check?

        return redirect($request->redirect_path);
    }

    // open a page to update contact information
    public function edit(Contact $contact)
    {
        // check if the user is allowed to edit the contact
        if (! backpack_user()->can('update', $contact)) {
            abort(403);
        }

        return view('students.edit-contact', [
            'contact' => $contact,
            'redirect_url' => url()->previous(),
        ]);
    }

    // delete additional contact information
    public function destroy(Contact $contact)
    {
        $contact->delete();
    }
}