Showing 910 of 910 total issues
Opening brace of a class must be on the line after the definition Open
class ELearn_Views{- Exclude checks
Visibility must be declared on method "preDelete" Open
function preDelete()- Exclude checks
Visibility must be declared on method "render_object" Open
function render_object()- Exclude checks
Visibility must be declared on method "preSave" Open
function preSave($create = false)- Exclude checks
The closing brace for the class must go on the next line after the body Open
}- Exclude checks
Expected "function abc(...)"; found "function abc (...)" Open
function ELearn_Shortcuts_GetPartByNameOr404 ($name)- Exclude checks
Visibility must be declared on method "preSave" Open
function preSave($create = false)- Exclude checks
Visibility must be declared on method "postSave" Open
function postSave($create = false)- Exclude checks
Visibility must be declared on method "preDelete" Open
function preDelete()- Exclude checks
The method preSave has a boolean flag argument $create, which is a certain sign of a Single Responsibility Principle violation. Open
function preSave($create = false)- Read upRead up
- Exclude checks
BooleanArgumentFlag
Since: 1.4.0
A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.
Example
class Foo {
public function bar($flag = true) {
}
}Source https://phpmd.org/rules/cleancode.html#booleanargumentflag
Missing class import via use statement (line '13', column '17'). Open
$item = new ELearn_Topic();- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
Missing class import via use statement (line '150', column '20'). Open
return new Pluf_HTTP_Response_Json($partCopy);- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
The method postSave has a boolean flag argument $create, which is a certain sign of a Single Responsibility Principle violation. Open
function postSave($create = false)- Read upRead up
- Exclude checks
BooleanArgumentFlag
Since: 1.4.0
A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.
Example
class Foo {
public function bar($flag = true) {
}
}Source https://phpmd.org/rules/cleancode.html#booleanargumentflag
Missing class import via use statement (line '37', column '27'). Open
throw new Pluf_Form_Invalid('An error occured when creating the upload path. Please try to send the file again.');- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
The method postSave has a boolean flag argument $create, which is a certain sign of a Single Responsibility Principle violation. Open
function postSave($create = false)- Read upRead up
- Exclude checks
BooleanArgumentFlag
Since: 1.4.0
A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.
Example
class Foo {
public function bar($flag = true) {
}
}Source https://phpmd.org/rules/cleancode.html#booleanargumentflag
Missing class import via use statement (line '261', column '28'). Open
$lastsql = new Pluf_SQL();- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
The method postSave has a boolean flag argument $create, which is a certain sign of a Single Responsibility Principle violation. Open
function postSave($create = false)- Read upRead up
- Exclude checks
BooleanArgumentFlag
Since: 1.4.0
A boolean flag argument is a reliable indicator for a violation of the Single Responsibility Principle (SRP). You can fix this problem by extracting the logic in the boolean flag into its own class or method.
Example
class Foo {
public function bar($flag = true) {
}
}Source https://phpmd.org/rules/cleancode.html#booleanargumentflag
Missing class import via use statement (line '66', column '23'). Open
$course = new ELearn_Course();- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
Missing class import via use statement (line '44', column '19'). Open
throw new Pluf_Exception(__('Part name must not be new, find'));- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}Source http://phpmd.org/rules/cleancode.html#MissingImport
Missing class import via use statement (line '67', column '26'). Open
$paginator = new Pluf_Paginator($course);- Read upRead up
- Exclude checks
MissingImport
Since: 2.7.0
Importing all external classes in a file through use statements makes them clearly visible.
Example
function make() {
return new \stdClass();
}