Sign upLogin

Rafalsky/HomeFinance

View on GitHub
Star
common/migrations/db/m140703_123803_article.php

Summary

Maintainability
A
1 hr
Test Coverage

Method safeUp has 45 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    public function safeUp()
    {
        $tableOptions = null;
        if ($this->db->driverName === 'mysql') {
            $tableOptions = 'CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE=InnoDB';
Severity: Minor
Found in common/migrations/db/m140703_123803_article.php - About 1 hr to fix

    The class m140703_123803_article is not named in CamelCase.
    Open

    class m140703_123803_article extends Migration
{
    public function safeUp()
    {
        $tableOptions = null;
    Severity: Minor
    Found in common/migrations/db/m140703_123803_article.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