Sign upLogin

nabble/ajde

View on GitHub
Star
core/modules/email/model/MailerlogModel.php

Summary

Maintainability
A
50 mins
Test Coverage

Method log has 7 arguments (exceeds 4 allowed). Consider refactoring.
Open

    public static function log($fromEmail, $fromName, $toEmail, $toName, $subject, $body, $status = 0)
Severity: Major
Found in core/modules/email/model/MailerlogModel.php - About 50 mins to fix

    The property $_autoloadParents is not named in camelCase.
    Open

    class MailerlogModel extends Ajde_Model
{
    protected $_autoloadParents = true;
    protected $_displayField = 'subject';
    Severity: Minor
    Found in core/modules/email/model/MailerlogModel.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 property $_displayField is not named in camelCase.
    Open

    class MailerlogModel extends Ajde_Model
{
    protected $_autoloadParents = true;
    protected $_displayField = 'subject';
    Severity: Minor
    Found in core/modules/email/model/MailerlogModel.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

    There are no issues that match your filters.

    Category
    Status