public/plugin/pens/pens.php
Missing class import via use statement (line '45', column '42'). Open
Open
$server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));- 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 '43', column '29'). Open
Open
$chamilo_pens = new ChamiloPens($request);- 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 parameter $path_to_package is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 "path_to_archives" is not in valid camel caps format Open
Open
mkdir($path_to_archives, $mode, true);- Exclude checks
Variable "path_to_package" is not in valid camel caps format Open
Open
rename($path_to_package, $path_to_archives.'/'.$request->getFilename());- Exclude checks
Missing function doc comment Open
Open
public function processPackage($request, $path_to_package)- Exclude checks
Variable "path_to_package" is not in valid camel caps format Open
Open
public function processPackage($request, $path_to_package)- Exclude checks
Variable "path_to_archives" is not in valid camel caps format Open
Open
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- Exclude checks
Variable "chamilo_pens" is not in valid camel caps format Open
Open
$chamilo_pens->save();- Exclude checks
Variable "path_to_archives" is not in valid camel caps format Open
Open
if (!is_dir($path_to_archives)) {- Exclude checks
Variable "path_to_archives" is not in valid camel caps format Open
Open
rename($path_to_package, $path_to_archives.'/'.$request->getFilename());- Exclude checks
Missing class doc comment Open
Open
class ChamiloPackageHandler extends PENSPackageHandler- Exclude checks
Variable "chamilo_pens" is not in valid camel caps format Open
Open
$chamilo_pens = new ChamiloPens($request);- Exclude checks
The variable $path_to_archives is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 $chamilo_pens is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 $path_to_archives is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 $path_to_archives is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 $path_to_package is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 $chamilo_pens is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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 $path_to_archives is not named in camelCase. Open
Open
public function processPackage($request, $path_to_package)
{
$server = PENSServer::singleton();
// Moves the package to archive/pens
$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';- 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();
}
}