EscolaLMS/Webinar

View on GitHub

Showing 636 of 636 total issues

Call to method can from undeclared class \EscolaLms\Auth\Models\User (Did you mean class \EscolaLms\Webinar\Models\User)
Open

                $user->can(WebinarPermissionsEnum::WEBINAR_DELETE_OWN)
Severity: Critical
Found in src/Policies/WebinarPolicy.php by phan

Reference to constant class from undeclared class \EscolaLms\Webinar\Http\Controllers\WebinarAPIController
Open

    Route::get('/start-live-stream/{id}', [WebinarAPIController::class, 'startLiveStream']);
Severity: Critical
Found in src/routes.php by phan

Reference to constant class from undeclared class \EscolaLms\Core\Models\User (Did you mean class \EscolaLms\Webinar\Models\User)
Open

        return $this->belongsTo(User::class);
Severity: Critical
Found in src/Models/WebinarUserPivot.php by phan

Call to method where from undeclared class \Illuminate\Database\Eloquent\Relations\BelongsToMany
Open

                && $webinar->trainers()->where('trainer_id', $user->getKey())->exists()
Severity: Critical
Found in src/Policies/WebinarPolicy.php by phan

Return type of apply() is undeclared type \Illuminate\Database\Eloquent\Builder
Open

    public function apply(Builder $query): Builder

syntax error, unexpected 'RelationStrategyContract' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST)
Open

    private RelationStrategyContract $relationStrategyContract;

Reference to constant class from undeclared class \EscolaLms\Webinar\Http\Controllers\WebinarAPIController
Open

    Route::get('/stop-live-stream/{id}', [WebinarAPIController::class, 'stopLiveStream']);
Severity: Critical
Found in src/routes.php by phan

Static call to undeclared method \EscolaLms\Webinar\Database\Factories\WebinarFactory::new
Open

        return WebinarFactory::new();
Severity: Critical
Found in src/Models/Webinar.php by phan

Call to method can from undeclared class \EscolaLms\Auth\Models\User (Did you mean class \EscolaLms\Webinar\Models\User)
Open

                $user->can(WebinarPermissionsEnum::WEBINAR_READ_OWN)
Severity: Critical
Found in src/Policies/WebinarPolicy.php by phan

Return type of getIncomingTerm() is undeclared type \Illuminate\Support\Collection
Open

    public function getIncomingTerm(array $criteria = []): Collection;

syntax error, unexpected '?', expecting function (T_FUNCTION) or const (T_CONST)
Open

    private ?bool $withDuration;

Parameter $webinarDto has undeclared type \EscolaLms\Webinar\Dto\WebinarDto
Open

    public function update(int $id, WebinarDto $webinarDto): Webinar;

Reference to constant class from undeclared class \EscolaLms\Webinar\Http\Controllers\WebinarController
Open

    Route::post('webinars/{id}', [WebinarController::class, 'update']);
Severity: Critical
Found in src/routes.php by phan

Call to method get from undeclared class \Illuminate\Support\Facades\Route
Open

    Route::get('/stop-live-stream/{id}', [WebinarAPIController::class, 'stopLiveStream']);
Severity: Critical
Found in src/routes.php by phan

Call to method group from undeclared class \Illuminate\Support\Facades\Route
Open

Route::group(['prefix' => 'api/webinars'], function () {
Severity: Critical
Found in src/routes.php by phan

Define a constant instead of duplicating this literal "logotype_path" 3 times.
Open

        if ($this->attributes['logotype_path'] ?? null) {
Severity: Critical
Found in src/Models/Webinar.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "user_id" 3 times.
Open

            $table->unsignedBigInteger('user_id');

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "author_id" 3 times.
Open

            $table->renameColumn('author_id', 'trainer_id');

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "nullable" 3 times.
Open

            'duration' => ['nullable', 'string', 'max:80'],

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "prefix" 3 times.
Open

Route::group(['middleware' => ['auth:api'], 'prefix' => 'api/admin'], function () {
Severity: Critical
Found in src/routes.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Severity
Category
Status
Source
Language