public/main/admin/course_list_admin.php
The function get_course_data() has an NPath complexity of 793152. The configured NPath complexity threshold is 200. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
NPathComplexity
Since: 0.1
The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.
Example
class Foo {
function bar() {
// lots of complicated code
}
}Source https://phpmd.org/rules/codesize.html#npathcomplexity
Avoid unused parameters such as '$dataFunctions'. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)- Read upRead up
- Exclude checks
UnusedFormalParameter
Since: 0.2
Avoid passing parameters to methods or constructors and then not using those parameters.
Example
class Foo
{
private function bar($howdy)
{
// $howdy is not used
}
}Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter
The parameter $number_of_items is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 "keyword_category" is not in valid camel caps format Open
Open
if (!empty($keyword_category)) {- Exclude checks
Consider putting global function "get_course_visibility_icon" in a static class Open
Open
function get_course_visibility_icon($visibility)- Exclude checks
Variable "number_of_items" is not in valid camel caps format Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)- Exclude checks
Variable "number_of_items" is not in valid camel caps format Open
Open
$number_of_items = (int) $number_of_items;- Exclude checks
Variable "access_url_rel_course_table" is not in valid camel caps format Open
Open
$access_url_rel_course_table = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_COURSE);- Exclude checks
Variable "keyword_subscribe" is not in valid camel caps format Open
Open
subscribe LIKE '".$keyword_subscribe."' AND- Exclude checks
Variable "keyword_visibility" is not in valid camel caps format Open
Open
$keyword_visibility = Database::escape_string("%".$_GET['keyword_visibility']."%");- Exclude checks
Variable "keyword_language" is not in valid camel caps format Open
Open
course_language LIKE '".$keyword_language."' AND- Exclude checks
Variable "keyword_category" is not in valid camel caps format Open
Open
$sql .= " AND category_code LIKE '".$keyword_category."' ";- Exclude checks
Variable "this_section" is not in valid camel caps format Open
Open
$this_section = SECTION_PLATFORM_ADMIN;- Exclude checks
Variable "number_of_items" is not in valid camel caps format Open
Open
$number_of_items = (int) $number_of_items;- Exclude checks
Variable "keyword_language" is not in valid camel caps format Open
Open
$keyword_language = Database::escape_string("%".$_GET['keyword_language']."%");- Exclude checks
Variable "course_code" is not in valid camel caps format Open
Open
CourseManager::delete_course($course_code);- Exclude checks
Variable "tool_name" is not in valid camel caps format Open
Open
$tpl = new Template($tool_name);- Exclude checks
Variable "keyword_subscribe" is not in valid camel caps format Open
Open
$keyword_subscribe = Database::escape_string($_GET['keyword_subscribe']);- Exclude checks
Consider putting global function "get_course_data" in a static class Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)- Exclude checks
Variable "keyword_title" is not in valid camel caps format Open
Open
$keyword_title = Database::escape_string("%".$_GET['keyword_title']."%");- Exclude checks
Variable "keyword_title" is not in valid camel caps format Open
Open
title LIKE '".$keyword_title."' AND- Exclude checks
Variable "keyword_code" is not in valid camel caps format Open
Open
(code LIKE '".$keyword_code."' OR visual_code LIKE '".$keyword_code."') AND- Exclude checks
Variable "course_codes" is not in valid camel caps format Open
Open
foreach ($course_codes as $course_code) {- Exclude checks
Variable "course_code" is not in valid camel caps format Open
Open
foreach ($course_codes as $course_code) {- Exclude checks
Variable "tool_name" is not in valid camel caps format Open
Open
$form->addElement('header', $tool_name);- Exclude checks
Variable "keyword_unsubscribe" is not in valid camel caps format Open
Open
$keyword_unsubscribe = Database::escape_string($_GET['keyword_unsubscribe']);- Exclude checks
Variable "keyword_visibility" is not in valid camel caps format Open
Open
visibility LIKE '".$keyword_visibility."' AND- Exclude checks
Variable "keyword_unsubscribe" is not in valid camel caps format Open
Open
unsubscribe LIKE '".$keyword_unsubscribe."'";- Exclude checks
Variable "course_codes" is not in valid camel caps format Open
Open
if (count($course_codes) > 0) {- Exclude checks
Variable "keyword_code" is not in valid camel caps format Open
Open
(code LIKE '".$keyword_code."' OR visual_code LIKE '".$keyword_code."') AND- Exclude checks
Variable "tool_name" is not in valid camel caps format Open
Open
$tool_name = get_lang('Search courses');- Exclude checks
Variable "tool_name" is not in valid camel caps format Open
Open
$tool_name = get_lang('CourseList');- Exclude checks
Variable "number_of_items" is not in valid camel caps format Open
Open
$sql .= " LIMIT $from, $number_of_items";- Exclude checks
Consider putting global function "get_number_of_courses" in a static class Open
Open
function get_number_of_courses()- Exclude checks
Variable "access_url_rel_course_table" is not in valid camel caps format Open
Open
$sql .= " INNER JOIN $access_url_rel_course_table url_rel_course- Exclude checks
Variable "keyword_category" is not in valid camel caps format Open
Open
$keyword_category = isset($_GET['keyword_category'])- Exclude checks
Variable "course_codes" is not in valid camel caps format Open
Open
$course_codes = $_POST['course'];- Exclude checks
Variable "keyword_code" is not in valid camel caps format Open
Open
$keyword_code = Database::escape_string("%".$_GET['keyword_code']."%");- Exclude checks
The variable $keyword_subscribe is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_category is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_visibility is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_title is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_category is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_code is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_title is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_language is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_code is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $access_url_rel_course_table is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_subscribe is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_category is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_code is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_unsubscribe is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_visibility is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_language is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $access_url_rel_course_table is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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 $keyword_unsubscribe is not named in camelCase. Open
Open
function get_course_data($from, $number_of_items, $column, $direction, $dataFunctions = [], $getCount = false)
{
$addTeacherColumn = true;
$table = Database::get_main_table(TABLE_MAIN_COURSE);
- Read upRead up
- Exclude checks
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();
}
}