m160706_223500_vote_updates

Complexity

Total Complexity

2

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	down()	0	4	1
A	up()	0	4	1

<?php

use hauntd\vote\migrations\Migration;

/**
 * @author Alexander Kononenko <contact@hauntd.me>
 */
class m160706_223500_vote_updates extends Migration

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public function up()
    {
        $this->createIndex('vote_target_user_idx', '{{%vote}}', ['entity', 'target_id', 'user_id'], false);
        $this->alterColumn('{{%vote}}', 'value', $this->boolean()->notNull());

        //TODO: check these command and fix migration
        //$this->execute('ALTER TABLE vote ALTER COLUMN value TYPE boolean USING CASE value WHEN 0 THEN false ELSE true END');

86% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    public function down()
    {
        $this->dropIndex('vote_target_user_idx', '{{%vote}}');
        $this->alterColumn('{{%vote}}', 'value', $this->smallInteger(1)->notNull());
    }
}