App\Http\Controllers\UsersController

Complexity

Total Complexity

31

Size/Duplication

Total Lines	342
Duplicated Lines	0 %

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	getIndex()	0	21	1
A	searchAlma()	0	12	2
A	json()	0	15	2
A	getShow()	0	4	1
A	connectForm()	0	4	1
A	connect()	0	27	5
A	upsert()	0	37	5
A	getEdit()	0	13	2
A	postMerge()	0	16	3
A	sync()	0	14	3
A	getMerge()	0	8	1
A	deleteForm()	0	9	2
A	createForm()	0	6	1

<?php

namespace App\Http\Controllers;

use App\User;
use App\Alma\User as AlmaUser;
use Carbon\Carbon;

The type Carbon\Carbon was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Http\Request;

The type Illuminate\Http\Request was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Http\Response;

The type Illuminate\Http\Response was not found. Maybe you did not declare it correctly or list all dependencies?

use LimitIterator;
use Scriptotek\Alma\Client as AlmaClient;

The type Scriptotek\Alma\Client was not found. Maybe you did not declare it correctly or list all dependencies?

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 (brukernavn@institusjon.no).',
        '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', [

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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);

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    /**
     * 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([]);

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

        }
        $query = 'ALL~' . $request->input('query');
        $users = collect($alma->users->search($query, ['limit' => 5]))->map(function ($u) {

The function collect was not found. Maybe you did not declare it correctly or list all dependencies?

            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', [

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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', [

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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.');

The function back was not found. Maybe you did not declare it correctly or list all dependencies?

        }
        $almaUsers = collect($alma->users->search('identifiers~' . $barcode, ['limit' => 1]))->map(function ($u) {

The function collect was not found. Maybe you did not declare it correctly or list all dependencies?

            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)

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

            ->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.');

The function back was not found. Maybe you did not declare it correctly or list all dependencies?

        }

        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;

The property barcode does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property university_id does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property lastname does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property firstname does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property phone does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property email does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }

        return response()->view('users.edit', array(

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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(), [

The type Validator was not found. Maybe you did not declare it correctly or list all dependencies?

            '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');

The property barcode does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property university_id does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property lastname does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property firstname does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property phone does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property email does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property lang does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The function dd was not found. Maybe you did not declare it correctly or list all dependencies?

        }

        if ($newUser) {
            return redirect()->action('LoansController@getIndex')

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

                ->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(

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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))

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

                ->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', [

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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)

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

                ->with('error', 'Kan ikke slette en bruker med aktive lån.');
        }

        return response()->view('users.delete', [

The function response was not found. Maybe you did not declare it correctly or list all dependencies?

            '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)

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

                ->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).");
    }
}