lib/Ajde/Publisher/Mail/Mail.php

Summary

Maintainability
A
0 mins
Test Coverage

Missing class import via use statement (line '14', column '23').
Open

        $mailer = new Ajde_Mailer();
Severity: Minor
Found in lib/Ajde/Publisher/Mail/Mail.php by phpmd

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 property $_recipients is not named in camelCase.
Open

class Ajde_Publisher_Mail extends Ajde_Publisher
{
    private $_recipients;

    public function setRecipients($addresses)
Severity: Minor
Found in lib/Ajde/Publisher/Mail/Mail.php by phpmd

CamelCasePropertyName

Since: 0.2

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

Example

class ClassName {
    protected $property_name;
}

Source

The class Ajde_Publisher_Mail is not named in CamelCase.
Open

class Ajde_Publisher_Mail extends Ajde_Publisher
{
    private $_recipients;

    public function setRecipients($addresses)
Severity: Minor
Found in lib/Ajde/Publisher/Mail/Mail.php by phpmd

CamelCaseClassName

Since: 0.2

It is considered best practice to use the CamelCase notation to name classes.

Example

class class_name {
}

Source

There are no issues that match your filters.

Category
Status