aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
-rw-r--r--app/DoctrineMigrations/Version20171008195606.php63
-rw-r--r--src/Wallabag/CoreBundle/Entity/Entry.php4
2 files changed, 65 insertions, 2 deletions
diff --git a/app/DoctrineMigrations/Version20171008195606.php b/app/DoctrineMigrations/Version20171008195606.php
new file mode 100644
index 00000000..7c202151
--- /dev/null
+++ b/app/DoctrineMigrations/Version20171008195606.php
@@ -0,0 +1,63 @@
1<?php
2
3namespace Application\Migrations;
4
5use Doctrine\DBAL\Migrations\AbstractMigration;
6use Doctrine\DBAL\Schema\Schema;
7use Symfony\Component\DependencyInjection\ContainerAwareInterface;
8use Symfony\Component\DependencyInjection\ContainerInterface;
9
10/**
11 * Changed reading_time field to prevent null value
12 */
13class Version20171008195606 extends AbstractMigration implements ContainerAwareInterface
14{
15 /**
16 * @var ContainerInterface
17 */
18 private $container;
19
20 public function setContainer(ContainerInterface $container = null)
21 {
22 $this->container = $container;
23 }
24
25 /**
26 * @param Schema $schema
27 */
28 public function up(Schema $schema)
29 {
30 $this->skipIf($this->connection->getDatabasePlatform()->getName() === 'sqlite', 'Migration can only be executed safely on \'mysql\' or \'postgresql\'.');
31
32 switch ($this->connection->getDatabasePlatform()->getName()) {
33 case 'mysql':
34 $this->addSql('ALTER TABLE ' . $this->getTable('entry') . ' CHANGE reading_time reading_time INT(11) NOT NULL;');
35 break;
36 case 'postgresql':
37 $this->addSql('ALTER TABLE ' . $this->getTable('entry') . ' ALTER COLUMN reading_time SET NOT NULL;');
38 break;
39 }
40 }
41
42 /**
43 * @param Schema $schema
44 */
45 public function down(Schema $schema)
46 {
47 $this->skipIf($this->connection->getDatabasePlatform()->getName() === 'sqlite', 'Migration can only be executed safely on \'mysql\' or \'postgresql\'.');
48
49 switch ($this->connection->getDatabasePlatform()->getName()) {
50 case 'mysql':
51 $this->addSql('ALTER TABLE ' . $this->getTable('entry') . ' CHANGE reading_time reading_time INT(11);');
52 break;
53 case 'postgresql':
54 $this->addSql('ALTER TABLE ' . $this->getTable('entry') . ' ALTER COLUMN reading_time DROP NOT NULL;');
55 break;
56 }
57 }
58
59 private function getTable($tableName)
60 {
61 return $this->container->getParameter('database_table_prefix') . $tableName;
62 }
63}
diff --git a/src/Wallabag/CoreBundle/Entity/Entry.php b/src/Wallabag/CoreBundle/Entity/Entry.php
index 4367902e..cfb8db75 100644
--- a/src/Wallabag/CoreBundle/Entity/Entry.php
+++ b/src/Wallabag/CoreBundle/Entity/Entry.php
@@ -180,11 +180,11 @@ class Entry
180 /** 180 /**
181 * @var int 181 * @var int
182 * 182 *
183 * @ORM\Column(name="reading_time", type="integer", nullable=true) 183 * @ORM\Column(name="reading_time", type="integer", nullable=false)
184 * 184 *
185 * @Groups({"entries_for_user", "export_all"}) 185 * @Groups({"entries_for_user", "export_all"})
186 */ 186 */
187 private $readingTime; 187 private $readingTime = 0;
188 188
189 /** 189 /**
190 * @var string 190 * @var string