EscolaLMS/Consultations

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

Open

    public function up()
    {
        Schema::table('consultation_user', function (Blueprint $table) {
            $table->dateTime('executed_at')->nullable();
            $table->string('executed_status')->nullable();

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

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

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

Open

    public function up()
    {
        if (DB::connection() instanceof MySqlConnection) {
            DB::statement('ALTER TABLE `consultation_user` DROP INDEX `consultation_user_unique`, ADD INDEX (user_id, consultation_id)');
        } else {

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

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

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

Open

    public function up(): void
    {
        Schema::create('consultations', function (Blueprint $table) {
            $table->id();
            $table->bigInteger('author_id')->unsigned()->nullable();

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

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

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

Open

    public function up(): void
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->dropColumn('duration');
        });

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

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

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

Open

    public function up()
    {
        Schema::table('consultation_terms', function (Blueprint $table) {
            $table->bigInteger('consultation_id')->unsigned()->nullable();
            $table->foreign('consultation_id')->on('consultations')->references('id')->nullOnDelete();

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

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

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

Open

    public function up()
    {
        Schema::table('consultation_user', function (Blueprint $table) {
            $table->bigInteger('product_id')->unsigned()->nullable();
            if (Schema::hasTable('products')) {

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

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

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

Open

    public function up()
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->string('image_path')->nullable();
        });

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

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

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

Open

    public function up(): void
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->renameColumn('started_at', 'active_from');
            $table->renameColumn('finished_at', 'active_to');

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

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

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

Open

    public function up(): void
    {
        Schema::create('consultations_participants', function (Blueprint $table) {
            $table->id();
            $table->bigInteger('consultation_id')->unsigned()->index();

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

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

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

Open

    public function up(): void
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->dropColumn('calendar_url');
        });

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

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

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

Open

    public function up()
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->string('logotype_path')->nullable();
        });

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

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

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

Open

    public function up(): void
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->string('duration')->nullable();
        });

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

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

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

Open

    public function up()
    {
        Schema::create('consultation_proposed_terms', function (Blueprint $table) {
            $table->id();
            $table->bigInteger('consultation_id')->unsigned();

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

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

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

Open

    public function up()
    {
        Schema::create('category_consultation', function (Blueprint $table) {
            $table->id();
            $table->unsignedBigInteger('consultation_id');

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

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

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

Open

    public function up()
    {
        Permission::findOrCreate(ConsultationsPermissionsEnum::CONSULTATION_CHANGE_TERM, 'api');
    }

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

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

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

Open

    public function up(): void
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->unsignedInteger('max_session_students')->default(1);
        });

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

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

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

Open

    public function up(): void
    {
        if (
            Schema::hasTable('orders') &&
            Schema::hasColumns('orders', ['executed_at', 'executed_status'])

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

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

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

Open

    public function up()
    {
        Schema::table('consultations', function (Blueprint $table) {
            $table->dropColumn('base_price');
        });

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

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

Method forCurrentUserResponse has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function forCurrentUserResponse(
        ListConsultationsRequest $listConsultationsRequest
    ): AnonymousResourceCollection {
        $search = $listConsultationsRequest->except(['limit', 'skip', 'order', 'order_by', 'paginate']);
        $consultations = $this->getConsultationsListForCurrentUser($search);

Method getByCurrentUserTutor has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getByCurrentUserTutor(): Collection
    {
        $result = collect();
        $terms = $this->model->newQuery()
            ->whereHas('consultationUser', fn (Builder $query) => $query