Custom getConnectionName not called on extended MagicLink model due to usage of self instead of static

[savander]

The latest change to the library broke my code:

laravel-magiclink/src/MagicLink.php

Line 93 in 82c82f9

$magiclink = new self();

I have extended your model class and done some modifications, such as overriding \Illuminate\Database\Eloquent\Model::getConnectionName to my custom one. The self keyword, do break the code, since it refers to the same class in which the new keyword is actually written. In that case, my getConnectionName is ignored and is not called.

The solution would be to use static since it uses late static binding, which refers to the class you called the method on.
It should not break your code, but enable it for further modification/extension.

Is there any reason to do that? Why would you restrict modification of your code?

In this case, other methods will also be affected, for example:

laravel-magiclink/src/MagicLink.php

Lines 165 to 177 in 82c82f9

	return self::where('id', $tokenId)
	->where('token', $tokenSecret)
	->where(function ($query) {
	$query
	->whereNull('available_at')
	->orWhere('available_at', '>=', Carbon::now());
	})
	->where(function ($query) {
	$query
	->whereNull('max_visits')
	->orWhereRaw('max_visits > num_visits');
	})
	->first();

[cesargb]

Please, feel free to submit a PR