Soved\Laravel\Magic\Auth\Traits\CanMagicallyLogin

Complexity

Total Complexity

3

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getEmailForMagicLink()	0	3	1
A	sendMagicLinkNotification()	0	3	1
A	viaMagicLink()	0	3	1

<?php

namespace Soved\Laravel\Magic\Auth\Traits;

use Soved\Laravel\Magic\Auth\Notifications\MagicLink as MagicLinkNotification;

trait CanMagicallyLogin
{
    /**
     * Get the e-mail address where magic links are sent.
     *
     * @return string
     */
    public function getEmailForMagicLink()
    {
        return $this->email;
    }

    /**
     * Send the magic link notification.
     *
     * @param  string  $link
     * @return void
     */
    public function sendMagicLinkNotification(string $link)
    {
        $this->notify(new MagicLinkNotification($link));

It seems like notify() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Determine if the user was authenticated via a magic link.
     *
     * @return bool
     */
    public function viaMagicLink()
    {
        return session('viaMagicLink', false);

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

    }
}