propelorm/Propel2

src/Propel/Generator/Behavior/AutoAddPk/AutoAddPkBehavior.php
A

The variable name, b, is too short

                $b = clone $this;
Severity: Minor
Found in src/Propel/Generator/Behavior/AutoAddPk/AutoAddPkBehavior.php - About 5 mins to fix

There are no issues that match your filters.

Category
<?php

/**
 * This file is part of the Propel package.
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @license MIT License
 */

namespace Propel\Generator\Behavior\AutoAddPk;

use Propel\Generator\Model\Behavior;

/**
 * Adds a primary key to models defined without one
 *
 * @author Fran├žois Zaninotto
 */
class AutoAddPkBehavior extends Behavior
{
    // default parameters value
    protected $parameters = [
        'name'          => 'id',
        'autoIncrement' => 'true',
        'type'          => 'INTEGER'
    ];

    /**
     * Copy the behavior to the database tables
     * Only for tables that have no Pk
     */
    public function modifyDatabase()
    {
        foreach ($this->getDatabase()->getTables() as $table) {
            if (!$table->hasPrimaryKey()) {
                $b = clone $this;
                $table->addBehavior($b);
            }
        }
    }

    /**
     * Add the primary key to the current table
     */
    public function modifyTable()
    {
        $table = $this->getTable();
        if (!$table->hasPrimaryKey() && !$table->hasBehavior('concrete_inheritance')) {
            $columnAttributes = array_merge(['primaryKey' => 'true'], $this->getParameters());
            $this->getTable()->addColumn($columnAttributes);
        }
    }
}

Size

Lines of code
28