Add some method annotators, see if this helps with Scrutinizer issues.

This commit is contained in:
James Cole 2018-07-14 23:04:05 +02:00
parent c424bb097d
commit 5dad569d62
9 changed files with 38 additions and 17 deletions

View File

@ -166,7 +166,7 @@ class AttachmentHelper implements AttachmentHelperInterface
$attachment->md5 = md5_file($path); $attachment->md5 = md5_file($path);
$attachment->mime = $mime; $attachment->mime = $mime;
$attachment->size = \strlen($content); $attachment->size = \strlen($content);
$attachment->uploaded = 1; $attachment->uploaded = true;
$attachment->save(); $attachment->save();
return true; return true;

View File

@ -210,7 +210,7 @@ class LinkTypeRepository implements LinkTypeRepositoryInterface
/** /**
* @param User $user * @param User $user
*/ */
public function setUser(User $user) public function setUser(User $user): void
{ {
$this->user = $user; $this->user = $user;
} }

View File

@ -25,6 +25,7 @@ namespace FireflyIII\Repositories\LinkType;
use FireflyIII\Models\LinkType; use FireflyIII\Models\LinkType;
use FireflyIII\Models\TransactionJournal; use FireflyIII\Models\TransactionJournal;
use FireflyIII\Models\TransactionJournalLink; use FireflyIII\Models\TransactionJournalLink;
use FireflyIII\User;
use Illuminate\Support\Collection; use Illuminate\Support\Collection;
/** /**
@ -62,6 +63,13 @@ interface LinkTypeRepositoryInterface
*/ */
public function find(int $id): LinkType; public function find(int $id): LinkType;
/**
* Set the user for this instance.
*
* @param User $user
*/
public function setUser(User $user): void;
/** /**
* Find link type by name. * Find link type by name.
* *

View File

@ -22,11 +22,15 @@ declare(strict_types=1);
namespace FireflyIII\Support\Facades; namespace FireflyIII\Support\Facades;
use FireflyIII\Models\TransactionCurrency;
use FireflyIII\User;
use Illuminate\Support\Facades\Facade; use Illuminate\Support\Facades\Facade;
/** /**
* @codeCoverageIgnore * @codeCoverageIgnore
* Class Amount. * Class Amount.
*
* @method TransactionCurrency getDefaultCurrencyByUser(User $user)
*/ */
class Amount extends Facade class Amount extends Facade
{ {

View File

@ -27,6 +27,9 @@ use Illuminate\Support\Facades\Facade;
/** /**
* @codeCoverageIgnore * @codeCoverageIgnore
* Class Navigation. * Class Navigation.
*
* @method array blockPeriods(\Carbon\Carbon $start, \Carbon\Carbon $end, string $range)
* @method string periodShow(\Carbon\Carbon $theDate, string $repeatFrequency)
*/ */
class Navigation extends Facade class Navigation extends Facade
{ {

View File

@ -22,11 +22,18 @@ declare(strict_types=1);
namespace FireflyIII\Support\Facades; namespace FireflyIII\Support\Facades;
use FireflyIII\Models\Preference;
use FireflyIII\User;
use Illuminate\Support\Facades\Facade; use Illuminate\Support\Facades\Facade;
/** /**
* @codeCoverageIgnore * @codeCoverageIgnore
* Class Preferences. * Class Preferences.
*
* @method Preference set(string $name, $value)
* @method Preference get(string $name, $value)
* @method Preference|null getForUser(User $user, string $name, $default)
* @method void mark()
*/ */
class Preferences extends Facade class Preferences extends Facade
{ {

View File

@ -27,6 +27,7 @@ use Illuminate\Support\Facades\Facade;
/** /**
* @codeCoverageIgnore * @codeCoverageIgnore
* Class Steam. * Class Steam.
* @method string positive(string $amount)
*/ */
class Steam extends Facade class Steam extends Facade
{ {

View File

@ -298,7 +298,7 @@ class Navigation
* *
* @throws \FireflyIII\Exceptions\FireflyException * @throws \FireflyIII\Exceptions\FireflyException
*/ */
public function periodShow(Carbon $theDate, string $repeatFrequency): string public function periodShow(\Carbon\Carbon $theDate, string $repeatFrequency): string
{ {
$date = clone $theDate; $date = clone $theDate;
$formatMap = [ $formatMap = [

View File

@ -79,12 +79,12 @@ class Preferences
} }
/** /**
* @param $name * @param string $name
* @param null $default * @param null $default
* *
* @return \FireflyIII\Models\Preference|null * @return \FireflyIII\Models\Preference|null
*/ */
public function get($name, $default = null) public function get(string $name, $default = null): ?Preference
{ {
/** @var User $user */ /** @var User $user */
$user = auth()->user(); $user = auth()->user();
@ -125,7 +125,7 @@ class Preferences
* *
* @return \FireflyIII\Models\Preference|null * @return \FireflyIII\Models\Preference|null
*/ */
public function getForUser(User $user, $name, $default = null): ?Preference public function getForUser(User $user, string $name, $default = null): ?Preference
{ {
$fullName = sprintf('preference%s%s', $user->id, $name); $fullName = sprintf('preference%s%s', $user->id, $name);
if (Cache::has($fullName)) { if (Cache::has($fullName)) {
@ -177,23 +177,21 @@ class Preferences
} }
/** /**
* @return bool *
*/ */
public function mark(): bool public function mark(): void
{ {
$this->set('lastActivity', microtime()); $this->set('lastActivity', microtime());
Session::forget('first'); Session::forget('first');
return true;
} }
/** /**
* @param $name * @param string $name
* @param $value * @param mixed $value
* *
* @return Preference * @return \FireflyIII\Models\Preference
*/ */
public function set($name, $value): Preference public function set(string $name, $value): Preference
{ {
$user = auth()->user(); $user = auth()->user();
if (null === $user) { if (null === $user) {
@ -210,12 +208,12 @@ class Preferences
/** /**
* @param \FireflyIII\User $user * @param \FireflyIII\User $user
* @param $name * @param string $name
* @param mixed $value * @param mixed $value
* *
* @return Preference * @return Preference
*/ */
public function setForUser(User $user, $name, $value): Preference public function setForUser(User $user, string $name, $value): Preference
{ {
$fullName = sprintf('preference%s%s', $user->id, $name); $fullName = sprintf('preference%s%s', $user->id, $name);
Cache::forget($fullName); Cache::forget($fullName);