Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge v2.2.2 from upstream #63

Merged
merged 17 commits into from
Jul 14, 2017

Conversation

yukoff
Copy link
Contributor

@yukoff yukoff commented Jul 12, 2017

No description provided.

@yukoff yukoff force-pushed the feature/merge-upstream-2.2.2 branch from c06e5a2 to 4855f21 Compare July 12, 2017 05:48
@konfuciusu
Copy link
Contributor

Було би добре перед цим вирішити #46

@yukoff
Copy link
Contributor Author

yukoff commented Jul 14, 2017

Там в базі всього дві колонки в таблиці дропнули - не думаю, що завадить #46

diff --git a/install/sql/mysql.sql b/install/sql/mysql.sql
index 9c5cdb2f..7949a0dd 100644
--- a/install/sql/mysql.sql
+++ b/install/sql/mysql.sql
@@ -856,8 +856,6 @@ INSERT INTO `bb_quota_limits` (`quota_limit_id`, `quota_desc`, `quota_limit`) VA
 CREATE TABLE `bb_ranks` (
   `rank_id` smallint(5) UNSIGNED NOT NULL,
   `rank_title` varchar(50) NOT NULL DEFAULT '',
-  `rank_min` mediumint(8) NOT NULL DEFAULT 0,
-  `rank_special` tinyint(1) NOT NULL DEFAULT 1,
   `rank_image` varchar(255) NOT NULL DEFAULT '',
   `rank_style` varchar(255) NOT NULL DEFAULT ''
 ) ENGINE=InnoDB DEFAULT CHARSET=utf8;
@@ -866,8 +864,8 @@ CREATE TABLE `bb_ranks` (
 -- Dumping data for table `bb_ranks`
 --

-INSERT INTO `bb_ranks` (`rank_id`, `rank_title`, `rank_min`, `rank_special`, `rank_image`, `rank_style`) VALUES
-(1, 'Адміністратор', -1, 1, 'styles/images/ranks/admin.png', 'colorAdmin');
+INSERT INTO `bb_ranks` (`rank_id`, `rank_title`, `rank_image`, `rank_style`) VALUES
+(1, 'Адміністратор', 'styles/images/ranks/admin.png', 'colorAdmin');

 -- --------------------------------------------------------

Як людина з немалим досвідом розробки автоматсистем міграції - Phinx не айс :( Плюс якщо використовувати програмні міграції (у Phinx можна або SQL, або PHP код юзати в міграціях), то згенерується дещо інша структура - типу все, що mediumint(X) стане mediumint(8) тощо, можу додати повний diff, якщо цікаво. В laravel є міграції (поки не знаю, чи можна їх заюзати без всього фреймворку), а взагалі щоб не городити щось своє я б радив або міграції з ruby/rails використати, або liquibase - це хоча й джава, але щонайменше там є diff.

@konfuciusu
Copy link
Contributor

Ще є doctrine із Symfony, її точно можна використовувати окремо.

В крайньому випадку можна просто .sql файли і мініатюрний скрипт, який перевіряє версію і виконує ті файли, яких не вистачає.

@konfuciusu konfuciusu merged commit d3382e2 into hurtom:master Jul 14, 2017
@yukoff yukoff deleted the feature/merge-upstream-2.2.2 branch July 14, 2017 19:18
@yukoff
Copy link
Contributor Author

yukoff commented Jul 14, 2017

@konfuciusu Хм, якось я доктрину й не врахував, подумав, що весь фреймворк потребує. Зараз дивлюсь на їх міграції - і поки що цікавше наче за phinx виглядає. Помацаю doctrine, поки @AIlkiv зробить PR з тим, що в нього вийшло з phinx

@yukoff yukoff self-assigned this Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants