public/main/lp/lp_final_item.php
Missing class import via use statement (line '229', column '25'). Open
Open
$skillRelUser = new SkillRelUserModel();
- 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
Avoid unused parameters such as '$courseCode'. Open
Open
$courseCode,
- 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
Avoid unused parameters such as '$sessionId'. Open
Open
$sessionId = 0,
- 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
Variable "_in_course" is not in valid camel caps format Open
Open
$_in_course = true;
- Exclude checks
Variable "current_course_tool" is not in valid camel caps format Open
Open
$current_course_tool = TOOL_GRADEBOOK;
- Exclude checks
Variable "show_message" is not in valid camel caps format Open
Open
if (empty($show_message) && !api_is_allowed_to_edit() && !api_is_excluded_user_type()) {
- Exclude checks
Consider putting global function "generateLPFinalItemTemplateBadgeLinks" in a static class Open
Open
function generateLPFinalItemTemplateBadgeLinks($userId, $courseId, $sessionId = 0)
- Exclude checks
Variable "show_message" is not in valid camel caps format Open
Open
$show_message = Category::show_message_resource_delete($courseId);
- Exclude checks
Consider putting global function "generateLPFinalItemTemplate" in a static class Open
Open
function generateLPFinalItemTemplate(
- Exclude checks