Showing 31 of 159 total issues
Missing class import via use statement (line '63', column '60'). Confirmed
throw_unless(count($this->manual_ids) <= 1000, new \Exception('Character ids must not exceed 1000'));
- Read upRead up
- Exclude checks
Missing class import via use statement (line '144', column '34'). Confirmed
default => throw new \Exception("Unknown recipient type {$recipient_type}"),
- Read upRead up
- Exclude checks
Missing class import via use statement (line '61', column '38'). Confirmed
default => throw new \Exception("Unexpected key: {$key}")
- Read upRead up
- Exclude checks
The method handle uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them. Confirmed
} else { RefreshToken::with(['corporation', 'character.roles']) ->cursor() ->map(fn (RefreshToken $token) => $token->corporation?->corporation_id) ->filter()
- Read upRead up
- Exclude checks
The method incrementEsiRateLimit uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them. Open
} else { cache()->set($this->ratelimit_key, $amount, carbon('now') ->addMinutes($this->ratelimit_duration)); }
- Read upRead up
- Exclude checks
TODO found Confirmed
// TODO check which columns are actually can be updated
- Exclude checks
TODO found Confirmed
return $query // TODO merge tables like assets for improved db performance
- Exclude checks
TODO found Open
// TODO: Missing Affiliations from character_users, character_info and contacts
- Exclude checks
TODO found Confirmed
// TODO: Update CorporationInfo and AllianceInfo
- Exclude checks
TODO found Confirmed
// TODO: Missing character_info from character_users
- Exclude checks
TODO found Confirmed
return parent::delete(); // TODO: Change the autogenerated stub
- Exclude checks