From e26bb0503f962e1ee56dfa138b43af4653065812 Mon Sep 17 00:00:00 2001
From: BoHong Li <a60814billy@gmail.com>
Date: Tue, 28 Mar 2017 15:16:09 +0800
Subject: [PATCH] fix: Support SQlite

Move 'unique' constraint to another statement (SQLite don't support set unique when addColumn)
---
 lib/migrations/20150702001020-update-to-0_3_1.js | 16 ++++++++++++----
 .../20150915153700-change-notes-title-to-text.js |  5 +++++
 lib/migrations/20160420180355-note-add-alias.js  | 11 ++++++++---
 3 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/lib/migrations/20150702001020-update-to-0_3_1.js b/lib/migrations/20150702001020-update-to-0_3_1.js
index a3163d7..40d9c97 100644
--- a/lib/migrations/20150702001020-update-to-0_3_1.js
+++ b/lib/migrations/20150702001020-update-to-0_3_1.js
@@ -3,17 +3,22 @@ module.exports = {
   up: function (queryInterface, Sequelize) {
     return queryInterface.addColumn('Notes', 'shortid', {
       type: Sequelize.STRING,
-      unique: true,
+      defaultValue: '0000000000',
       allowNull: false
+    }).then(function () {
+      return queryInterface.addIndex('Notes', ['shortid'], {
+        indicesType: 'UNIQUE'
+      })
     }).then(function () {
       return queryInterface.addColumn('Notes', 'permission', {
         type: Sequelize.STRING,
-        allowNull: false,
-        defaultValue: 0
+        defaultValue: 'private',
+        allowNull: false
       })
     }).then(function () {
       return queryInterface.addColumn('Notes', 'viewcount', {
-        type: Sequelize.INTEGER
+        type: Sequelize.INTEGER,
+        defaultValue: 0
       })
     })
   },
@@ -23,6 +28,9 @@ module.exports = {
       .then(function () {
         return queryInterface.removeColumn('Notes', 'permission')
       })
+      .then(function () {
+        return queryInterface.removeIndex('Notes', ['shortid'])
+      })
       .then(function () {
         return queryInterface.removeColumn('Notes', 'shortid')
       })
diff --git a/lib/migrations/20150915153700-change-notes-title-to-text.js b/lib/migrations/20150915153700-change-notes-title-to-text.js
index 6f7307d..2b3a82b 100644
--- a/lib/migrations/20150915153700-change-notes-title-to-text.js
+++ b/lib/migrations/20150915153700-change-notes-title-to-text.js
@@ -3,12 +3,17 @@ module.exports = {
   up: function (queryInterface, Sequelize) {
     return queryInterface.changeColumn('Notes', 'title', {
       type: Sequelize.TEXT
+    }).then(function () {
+      // manual added index will be removed in sqlite
+      return queryInterface.addIndex('Notes', ['shortid'])
     })
   },
 
   down: function (queryInterface, Sequelize) {
     return queryInterface.changeColumn('Notes', 'title', {
       type: Sequelize.STRING
+    }).then(function () {
+      return queryInterface.addIndex('Notes', ['shortid'])
     })
   }
 }
diff --git a/lib/migrations/20160420180355-note-add-alias.js b/lib/migrations/20160420180355-note-add-alias.js
index 2c3e453..a043cd5 100644
--- a/lib/migrations/20160420180355-note-add-alias.js
+++ b/lib/migrations/20160420180355-note-add-alias.js
@@ -2,12 +2,17 @@
 module.exports = {
   up: function (queryInterface, Sequelize) {
     return queryInterface.addColumn('Notes', 'alias', {
-      type: Sequelize.STRING,
-      unique: true
+      type: Sequelize.STRING
+    }).then(function () {
+      return queryInterface.addIndex('Notes', ['alias'], {
+        indicesType: 'UNIQUE'
+      })
     })
   },
 
   down: function (queryInterface, Sequelize) {
-    return queryInterface.removeColumn('Notes', 'alias')
+    return queryInterface.removeColumn('Notes', 'alias').then(function () {
+      return queryInterface.removeIndex('Notes', ['alias'])
+    })
   }
 }