sant0ro/chocolatey

The class ApiTest has 12 public methods. Consider refactoring ApiTest to keep number of public methods under 10.

Open

class ApiTest extends TestCase
{
    /**
     * Test Index of the API.
     *

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

User has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class User extends Model implements AuthenticatableContract, AuthorizableContract
{
    use Authenticatable, Authorizable, Eloquence, Mappable;

    /**

Method createPoolRoom has 52 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function createPoolRoom(User $user): bool
    {
        $room = (new Room())->store("{$user->name}'s room", "{$user->name} has entered the building", 'model_h', 25, 12, 307, 3104, 1.10, $user->uniqueId, $user->name);

        $user->update(['home_room' => $room->id]);

Avoid excessively long class names like CreateChocolateyUsersPhotosReportedCategoriesTable. Keep class name length under 40.

Open

class CreateChocolateyUsersPhotosReportedCategoriesTable extends Migration
{
    /**
     * Run the migrations.
     *

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

class ATooLongClassNameThatHintsAtADesignProblem {

}

interface ATooLongInterfaceNameThatHintsAtADesignProblem {

}

Avoid excessively long class names like CreateChocolateyShopPaymentCategoriesTable. Keep class name length under 40.

Open

class CreateChocolateyShopPaymentCategoriesTable extends Migration
{
    /**
     * Run the migrations.
     *

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

class ATooLongClassNameThatHintsAtADesignProblem {

}

interface ATooLongInterfaceNameThatHintsAtADesignProblem {

}

Avoid excessively long class names like CreateChocolateyUsersSecurityTrustedTable. Keep class name length under 40.

Open

class CreateChocolateyUsersSecurityTrustedTable extends Migration
{
    /**
     * Run the migrations.
     *

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

class ATooLongClassNameThatHintsAtADesignProblem {

}

interface ATooLongInterfaceNameThatHintsAtADesignProblem {

}

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

Open

    public $buildersClubFurniLimit = 0;

• 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 using short method names like CreateChocolateyUsersPhotosReportedTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_photos_reported', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('photo_id');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyUsersSecurityTrustedTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_security_trusted', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('user_id');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyShopPaymentCheckoutTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_shop_payment_checkout', function (Blueprint $table) {
            $table->increments('id');
            $table->string('category', 50)->default('online');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyUsersIdTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_id', function (Blueprint $table) {
            $table->string('mail', 150);
            $table->enum('mail_verified', ['0', '1'])->default('0');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyShopPaymentCategoriesTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_shop_payment_categories', function (Blueprint $table) {
            $table->increments('id');
            $table->string('country_code', 5)->default('all');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyShopCountriesTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_shop_countries', function (Blueprint $table) {
            $table->increments('id');
            $table->string('countryCode', 5)->default('all');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyUsersPreferencesTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_preferences', function (Blueprint $table) {
            $table->integer('user_id');
            $table->enum('emailFriendRequestNotificationEnabled', ['1', '0'])->default('0');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyShopItemsTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_shop_items', function (Blueprint $table) {
            $table->increments('id');
            $table->string('countryCode', 5)->default('all');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyShopHistoryTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_shop_history', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('user_id');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyArticlesCategoriesTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_articles_categories', function (Blueprint $table) {
            $table->string('link', 50);
            $table->string('translate', 125);

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyUsersMailRequestsTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_mail_requests', function (Blueprint $table) {
            $table->increments('id');
            $table->string('token', 255);

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyUsersPhotosReportedCategoriesTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_photos_reported_categories', function (Blueprint $table) {
            $table->integer('report_category');
            $table->string('description', 255);

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}

Avoid using short method names like CreateChocolateyUsersPhotosLikesTable::up(). The configured minimum method name length is 3.

Open

    public function up()
    {
        Schema::create('chocolatey_users_photos_likes', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('photo_id');

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

class ShortMethod {
    public function a( $index ) { // Violation
    }
}