UsersController::deleteForm() - Code Metrics - Inspection of "Add user sync command" - scriptotek/bibrex - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 1fd74f...a7048c )

by Dan Michael O.
created 2018-07-30 18:19 UTC
UsersController::deleteForm() A

↳ Parent: UsersController
Complexity

Conditions	2
Paths	2
Size

Total Lines	9
Code Lines	5
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
cc	2
eloc	5
nc	2
nop	2
dl	0
loc	9
rs	10
c	0
b	0
f	0
<?php

namespace App\Http\Controllers;

use App\User;
use App\Alma\User as AlmaUser;
use Carbon\Carbon;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Http\Request;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Http\Response;
filter:
    dependency_paths: ["lib/*"]
use LimitIterator;
use Scriptotek\Alma\Client as AlmaClient;
filter:
    dependency_paths: ["lib/*"]

class UsersController extends Controller
{

    private $messages = [
        'barcode.regex' => 'Låne-ID er ikke på riktig format.',
        'university_id.regex' => 'Feide-ID er ikke på riktig format ([email protected]).',
        'barcode.unique' => 'Det finnes allerede en annen bruker med denne strekkoden.',
        'university_id.unique' => 'Det finnes allerede en annen bruker med denne Feide-IDen.',
        'lastname.required' => 'Etternavn må fylles inn.',
        'firstname.required' => 'Fornavn må fylles inn.',
        'email.required_without' => 'Enten e-post eller telefonnummer må fylles inn.',
        'lang.required' => 'Språk må fylles inn.'
    ];

    /**
     * Display a listing of the resource.
     *
     * @param  Request $request
     * @return Response
     */
    public function getIndex(Request $request)
    {
        $users = User::with('loans')
            ->where('lastname', '!=', '(anonymisert)')
            ->orderBy('lastname')
            ->get()
            ->map(function ($user) {
                return [
                    'id' => $user->id,
                    'primaryId' => $user->alma_primary_id,
                    'group' => $user->alma_user_group,
                    'name' => $user->lastname . ', ' . $user->firstname,
                    'barcode' => $user->barcode,
                    'in_alma' => $user->in_alma,
                    'created_at' => $user->created_at->toDateTimestring(),
                    'note' => $user->note,
                ];
            });

        return response()->view('users.index', [
        return /** @scrutinizer ignore-call */ response()->view('users.index', [
            'users' => $users,
        ]);
    }

    /**
     * Display a listing of the resource as json.
     *
     * @param  Request $request
     * @return Response
     */
    public function json(Request $request)
    {
        $users = [];
        foreach (User::get() as $user) {
            $users[] = [
                'id' => $user->id,
                'primaryId' => $user->alma_primary_id,
                'group' => $user->alma_user_group,
                'name' => $user->lastname . ', ' . $user->firstname,
                'barcode' => $user->barcode,
                'type' => 'local',
            ];
        }

        return response()->json($users);
        return /** @scrutinizer ignore-call */ response()->json($users);
    }

    /**
     * Display a listing of the resource.
     *
     * @param AlmaClient $alma
     * @param  Request $request
     * @return Response
     */
    public function searchAlma(AlmaClient $alma, Request $request)
    {
        if (is_null($alma->key)) {
            \Log::warning('Cannot search Alma users since no Alma API key is configured.');
            return response()->json([]);
            return /** @scrutinizer ignore-call */ response()->json([]);
        }
        $query = 'ALL~' . $request->input('query');
        $users = collect($alma->users->search($query, ['limit' => 5]))->map(function ($u) {
        $users = /** @scrutinizer ignore-call */ collect($alma->users->search($query, ['limit' => 5]))->map(function ($u) {
            return new AlmaUser($u);
        });

        return response()->json($users);
    }

    /**
     * Display the specified resource.
     *
     * @param User $user
     * @return Response
     */
    public function getShow(User $user)
    {
        return response()->view('users.show', [
        return /** @scrutinizer ignore-call */ response()->view('users.show', [
            'user' => $user,
        ]);
    }

    /**
     * Display form for connecting local user to external user.
     *
     * @param User $user
     * @return Response
     */
    public function connectForm(User $user)
    {
        return response()->view('users.connect', [
        return /** @scrutinizer ignore-call */ response()->view('users.connect', [
            'user' => $user,
        ]);
    }

    /**
     * Connect local user to external user.
     *
     * @param User $user
     * @param Request $request
     * @param AlmaClient $alma
     * @return Response
     */
    public function connect(User $user, Request $request, AlmaClient $alma)
    {
        $barcode = $request->barcode;
        if (empty($barcode)) {
            return back()->with('error', 'Du må registrere låne-ID.');
            return /** @scrutinizer ignore-call */ back()->with('error', 'Du må registrere låne-ID.');
        }
        $almaUsers = collect($alma->users->search('identifiers~' . $barcode, ['limit' => 1]))->map(function ($u) {
        $almaUsers = /** @scrutinizer ignore-call */ collect($alma->users->search('identifiers~' . $barcode, ['limit' => 1]))->map(function ($u) {
            return new AlmaUser($u);
        });
        if (!count($almaUsers)) {
            return back()->with('error', 'Fant ikke låne-ID-en ' . $user->barcode . ' i Alma 😭 ');
        }

        $almaUser = $almaUsers[0];

        $barcode = $almaUser->getBarcode();
        $other = User::where('barcode', '=', $barcode)->first();
        if (!is_null($other) && $other->id != $user->id) {
            return back()->with('error', 'Låne-ID-en er allerede koblet til en annen Bibrex-bruker ' .
                '(' . $other->name . '). Du kan slå dem sammen fra brukeroversikten.');
        }

        $user->mergeFromAlmaResponse($almaUser);
        $user->save();

        return redirect()->action('UsersController@getShow', $user->id)
        return /** @scrutinizer ignore-call */ redirect()->action('UsersController@getShow', $user->id)
            ->with('status', 'Bibrex-brukeren ble koblet med Alma-brukeren!');
    }

    /**
     * Import user data from Alma.
     *
     * @param  int  $id
     * @return Response
     */
    public function sync(AlmaClient $alma, User $user)
    {
        if (!$user->barcode) {
            return back()->with('error', 'Du må registrere låne-ID for brukeren før du kan importere.');
            return /** @scrutinizer ignore-call */ back()->with('error', 'Du må registrere låne-ID for brukeren før du kan importere.');
        }

        if (!$user->updateFromAlma($alma)) {
            $user->save();

            return back()->with('error', 'Fant ikke brukeren i Alma 😭');
        }
        $user->save();

        return back()->with('status', 'Brukeropplysninger ble oppdatert fra Alma.');
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param User $user
     * @param Request $request
     * @return Response
     */
    public function getEdit(User $user, Request $request)
    {
        if (!$user->id) {
            $user->barcode = $request->barcode;

            $user->university_id = $request->university_id;

            $user->lastname = $request->lastname;

            $user->firstname = $request->firstname;

            $user->phone = $request->phone;

            $user->email = $request->email;

        }

        return response()->view('users.edit', array(
        return /** @scrutinizer ignore-call */ response()->view('users.edit', array(
            'user' => $user
        ));
    }

    /**
     * Update the specified resource in storage.
     *
     * @param User $user
     * @param  Request $request
     * @return Response
     */
    public function upsert(User $user, Request $request)
    {
        \Validator::make($request->input(), [
filter:
    dependency_paths: ["lib/*"]
            'barcode' => 'nullable|regex:/^[0-9a-zA-Z]{10}$/|unique:users,barcode' . ($user->id ? ',' . $user->id : ''),
            'university_id' => 'nullable|regex:/@/|unique:users,university_id' . ($user->id ? ',' . $user->id : ''),
            'lastname' => 'required',
            'firstname' => 'required',
            'email' => 'requiredWithout:phone',
            'lang' => 'required',
        ], $this->messages)->validate();

        $user->barcode = $request->input('barcode');

        $user->university_id = $request->input('university_id');

        $user->lastname = $request->input('lastname');

        $user->firstname = $request->input('firstname');

        $user->phone = $request->input('phone');

        $user->email = $request->input('email');

        $user->note = $request->input('note');
        $user->lang = $request->input('lang');

        $user->last_loan_at = Carbon::now();
        $newUser = !$user->exists;
        if (!$user->save()) {
            dd('Oi');
            dd('Oi');
        }

        if ($newUser) {
            return redirect()->action('LoansController@getIndex')
            return /** @scrutinizer ignore-call */ redirect()->action('LoansController@getIndex')
                ->with('status', 'Brukeren ble opprettet.')
                ->with('user', [
                    'type' => 'local',
                    'id' => $user->id,
                    'name' => $user->lastname . ', ' . $user->firstname,
                ]);
        }

        return redirect()->action('UsersController@getShow', $user->id)
            ->with('status', 'Brukeren ble lagret.');
    }

    /**
     * Display form to merge two users.
     *
     * @param User $user1
     * @param User $user2
     * @return Response
     */
    public function getMerge(User $user1, User $user2)
    {
        $merged = $user1->getMergeData($user2);

        return response()->view('users.merge', array(
        return /** @scrutinizer ignore-call */ response()->view('users.merge', array(
            'user1' => $user1,
            'user2' => $user2,
            'merged' => $merged
        ));
    }

    /**
     * Merge $user2 into $user1
     *
     * @param Request $request
     * @param User $user1
     * @param User $user2
     * @return Response
     */
    public function postMerge(Request $request, User $user1, User $user2)
    {
        $mergedAttributes = array();
        foreach (User::$editableAttributes as $attr) {
            $mergedAttributes[$attr] = $request->input($attr);
        }

        $errors = $user1->merge($user2, $mergedAttributes);

        if (!is_null($errors)) {
            return redirect()->action('UsersController@getMerge', array($user1->id, $user2->id))
            return /** @scrutinizer ignore-call */ redirect()->action('UsersController@getMerge', array($user1->id, $user2->id))
                ->withErrors($errors);
        }

        return redirect()->action('UsersController@getShow', $user1->id)
            ->with('status', 'Brukerne ble flettet.');
    }

    /**
     * Show the form for creating the specified resource.
     *
     * @param Request $request
     * @return Response
     */
    public function createForm(Request $request)
    {
        $user = User::make();

        return response()->view('users.create', [
        return /** @scrutinizer ignore-call */ response()->view('users.create', [
            'user' => $user,
        ]);
    }

    /**
     * Show the form for deleting the specified resource.
     *
     * @param User $user
     * @param Request $request
     * @return Response
     */
    public function deleteForm(User $user, Request $request)
    {
        if ($user->loans()->count()) {
            return redirect()->action('UsersController@getShow', $user->id)
            return /** @scrutinizer ignore-call */ redirect()->action('UsersController@getShow', $user->id)
                ->with('error', 'Kan ikke slette en bruker med aktive lån.');
        }

        return response()->view('users.delete', [
        return /** @scrutinizer ignore-call */ response()->view('users.delete', [
            'user' => $user,
        ]);
    }

    /**
     * Delte the specified resource from storage.
     *
     * @param User $user
     * @param Request $request
     * @return Response
     */
    public function delete(User $user, Request $request)
    {
        if ($user->loans()->count()) {
            return redirect()->action('UsersController@getShow', $user->id)
            return /** @scrutinizer ignore-call */ redirect()->action('UsersController@getShow', $user->id)
                ->with('error', 'Kan ikke slette en bruker med aktive lån.');
        }

        $user_id = $user->id;
        $name = $user->name;

        $user->delete();
        \Log::info(sprintf('Slettet brukeren med ID %d', $user_id));

        return redirect()->action('UsersController@getIndex')
            ->with('status', "Brukeren $name ble slettet (men slapp av, du har ikke drept noen).");
    }
}

scriptotek / bibrex

Push — master ( 1fd74f...a7048c )

UsersController::deleteForm() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
