chamilo/chamilo-lms

View on GitHub
public/main/admin/course_request_accepted.php

Summary

Maintainability
A
0 mins
Test Coverage

The parameter $number_of_items is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseParameterName

Since: 0.2

It is considered best practice to use the camelCase notation to name parameters.

Example

class ClassName {
    public function doSomething($user_name) {
    }
}

Source

Variable "delete_course_request" is not in valid camel caps format
Open

        $result = CourseRequestManager::delete_course_request($delete_course_request);

Variable "course_requests" is not in valid camel caps format
Open

                $course_requests = $_POST['course_request'];

Variable "course_request_id" is not in valid camel caps format
Open

                    foreach ($_POST['course_request'] as $index => $course_request_id) {

Variable "link_to_setting" is not in valid camel caps format
Open

        sprintf('<strong><a href="%s">%s</a></strong>', $link_to_setting, get_lang('Courses validation'))

Variable "course_request" is not in valid camel caps format
Open

    while ($course_request = Database :: fetch_row($res)) {

Variable "is_error_message" is not in valid camel caps format
Open

            $is_error_message = false;

Variable "course_request_code" is not in valid camel caps format
Open

            $message = sprintf(get_lang('The course request %s has not been deleted due to internal error.'), $course_request_code);

Variable "course_request" is not in valid camel caps format
Open

        $course_request[5] = api_get_local_time($course_request[5]);

Variable "course_validation_feature" is not in valid camel caps format
Open

$course_validation_feature = 'true' == api_get_setting('course_validation');

Variable "delete_course_request" is not in valid camel caps format
Open

    if (!empty($delete_course_request)) {

Variable "course_request_code" is not in valid camel caps format
Open

            $message = sprintf(get_lang('The course request %s has been deleted.'), $course_request_code);

Variable "number_of_items" is not in valid camel caps format
Open

    $number_of_items = (int) $number_of_items;

Variable "course_request" is not in valid camel caps format
Open

        $course_request[5] = api_get_local_time($course_request[5]);

Variable "this_section" is not in valid camel caps format
Open

$this_section = SECTION_PLATFORM_ADMIN;

Variable "delete_course_request" is not in valid camel caps format
Open

$delete_course_request = isset($_GET['delete_course_request']) ? (int) ($_GET['delete_course_request']) : '';

Variable "course_requests" is not in valid camel caps format
Open

    return $course_requests;

Variable "tool_name" is not in valid camel caps format
Open

Display :: display_header($tool_name);

Variable "number_of_items" is not in valid camel caps format
Open

    $number_of_items = (int) $number_of_items;

Variable "link_to_setting" is not in valid camel caps format
Open

    $link_to_setting = api_get_path(WEB_CODE_PATH).'admin/settings.php?category=Platform#course_validation';

Variable "is_error_message" is not in valid camel caps format
Open

    $is_error_message = true;

Variable "number_of_items" is not in valid camel caps format
Open

function get_request_data($from, $number_of_items, $column, $direction)

Variable "is_error_message" is not in valid camel caps format
Open

    if ($is_error_message) {

Variable "course_validation_feature" is not in valid camel caps format
Open

if ($course_validation_feature) {

Variable "is_error_message" is not in valid camel caps format
Open

$is_error_message = !empty($_GET['is_error_message']);

Variable "is_error_message" is not in valid camel caps format
Open

            $is_error_message = true;

Consider putting global function "get_request_data" in a static class
Open

function get_request_data($from, $number_of_items, $column, $direction)

Consider putting global function "modify_filter" in a static class
Open

function modify_filter($id)

Variable "is_error_message" is not in valid camel caps format
Open

                    $is_error_message = !$success;

Consider putting global function "get_number_of_requests" in a static class
Open

function get_number_of_requests()

Variable "course_request_table" is not in valid camel caps format
Open

    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);

Variable "course_request_code" is not in valid camel caps format
Open

        $course_request_code = CourseRequestManager::get_course_request_code($delete_course_request);

Variable "course_request_id" is not in valid camel caps format
Open

                        $success &= CourseRequestManager::delete_course_request($course_request_id);

Variable "course_request_table" is not in valid camel caps format
Open

           FROM $course_request_table

Variable "course_requests" is not in valid camel caps format
Open

    $course_requests = [];

Variable "course_requests" is not in valid camel caps format
Open

        $course_requests[] = $course_request;

Variable "delete_course_request" is not in valid camel caps format
Open

        $course_request_code = CourseRequestManager::get_course_request_code($delete_course_request);

Variable "number_of_items" is not in valid camel caps format
Open

    $sql .= " LIMIT $from,$number_of_items";

Variable "tool_name" is not in valid camel caps format
Open

$tool_name = get_lang('Accepted course requests');

Variable "course_validation_feature" is not in valid camel caps format
Open

if (!$course_validation_feature) {

Variable "course_request" is not in valid camel caps format
Open

        $course_requests[] = $course_request;

The variable $course_requests is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_requests is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $number_of_items is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_request is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_requests is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_request_table is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $number_of_items is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_request is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $number_of_items is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_request is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_request_table is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $course_request is not named in camelCase.
Open

function get_request_data($from, $number_of_items, $column, $direction)
{
    $keyword = isset($_GET['keyword']) ? Database::escape_string(trim($_GET['keyword'])) : null;
    $course_request_table = Database::get_main_table(TABLE_MAIN_COURSE_REQUEST);
    $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

There are no issues that match your filters.

Category
Status