Update m150102_164631_create_rating_table.php

[loveorigami]

timestamp to int

[chiliec]

А чем timestamp плох? И тогда надо ещё эту строчку удалить будет https://github.com/Chiliec/yii2-vote/commit/097a1e8a2795235f353be3fe80a0a7911e1c1488#diff-572fb452d2419a82290c4ba0b568e09dR43

[loveorigami]

видимо, по той же причине... int хранится во всех базах одинаково..

[chiliec]

А нужна ли такая унификация? Я вот подумываю хранить IP в VARBINARY, его тоже может не быть в других базах...

[loveorigami]

Видел реализацию в других проектах. https://github.com/trntv/yii2-starter-kit/blob/master/common/migrations/m140703_123803_article.php#L22

И даже у Александра (SamDark) https://github.com/samdark/yiifeed/blob/master/migrations/m141205_224547_change_news_date_field_type.php

use yii\db\Schema;
use yii\db\Migration;
class m141205_224547_change_news_date_field_type extends Migration
{
protected $tableName = '{{%news}}';
public function up()
{
$this->alterColumn($this->tableName,'created_at',Schema::TYPE_INTEGER);
}
public function down()
{
$this->alterColumn($this->tableName,'created_at',Schema::TYPE_DATETIME);
}
}

Да и сам так делаю. Проще потом дату в разных форматах и видах представлять. date("F j Y H:i:s) или Y-m-d

[chiliec]

Хорошо, я согласен, тем более что дата никак пока и не используется. Только давайте тогда как в вашем примере у Александра и сделаем — отдельной миграцией (чтоб людям с уже установленным модулем не пришлось менять вручную). Изначально то он тоже не тот тип выбрал https://github.com/samdark/yiifeed/blob/master/migrations/m141202_220415_create_news_table.php#L19

[loveorigami]

согласен!