seatplus/eveapi

Method handle has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Wontfix

    public function handle(int $location_id, Collection $tracking): ?RefreshToken
    {
        $character_ids_to_ignore = $tracking->pluck('character_id');

        $corporation_member_tracking_query = CorporationMemberTracking::query()

Avoid too many return statements within this method.

Invalid

        return $refresh_token;

Avoid excessively long variable names like $corporation_member_tracking_query. Keep variable name length under 20.

Wontfix

        $corporation_member_tracking_query = CorporationMemberTracking::query()

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $character_ids_to_ignore. Keep variable name length under 20.

Wontfix

        $character_ids_to_ignore = $tracking->pluck('character_id');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}