EscolaLMS/Webinar

View on GitHub

Showing 636 of 636 total issues

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

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

    Route::get('webinars/users/assignable', [WebinarController::class, 'assignableUsers']);
Severity: Critical
Found in src/routes.php by phan

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

    Route::get('/me', [WebinarAPIController::class, 'forCurrentUser']);
Severity: Critical
Found in src/routes.php by phan

Reference to undeclared property \EscolaLms\Webinar\Models\Webinar->duration
Open

        return $this->duration ?? '';
Severity: Minor
Found in src/Models/Webinar.php by phan

Call to undeclared function \app()
Open

        $webinarServiceContract = app(WebinarServiceContract::class);
Severity: Critical
Found in src/Models/Webinar.php by phan

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

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

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

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

Reference to undeclared property \EscolaLms\Webinar\Repositories\Criteria\WebinarSearch->value
Open

        return $query->where('webinars.name', $like, '%' . $this->value . '%');

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('/start-live-stream/{id}', [WebinarAPIController::class, 'startLiveStream']);
Severity: Critical
Found in src/routes.php by phan

Return type of users() is undeclared type \Illuminate\Database\Eloquent\Relations\BelongsToMany
Open

    public function users(): BelongsToMany
Severity: Minor
Found in src/Models/Webinar.php by phan

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

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

Class extends undeclared class \EscolaLms\Core\Repositories\Criteria\Criterion
Open

class WebinarSearch extends Criterion

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

    private Webinar $webinar;

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

    Route::get('/', [WebinarAPIController::class, 'index']);
Severity: Critical
Found in src/routes.php by phan

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

                && $webinar->trainers()->where('trainer_id', $user->getKey())->exists()
Severity: Critical
Found in src/Policies/WebinarPolicy.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 "webinar_authors" 3 times.
Open

        Schema::rename('webinar_authors', 'webinar_trainers');

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 "string" 6 times.
Open

            'name' => ['required', 'string', 'max:255', 'min:3'],

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.

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.

Severity
Category
Status
Source
Language