update_cms_301::up() - Code Metrics - Inspection of "Fix newcomments locale show" - phpffcms/ffcms - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( e3c39f...fe570d )

by Mihail

created 2019-01-11 11:36 UTC

update_cms_301::up() A

↳ Parent: update_cms_301

Complexity

Conditions	3
Paths	4

Size

Total Lines	16
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	7
dl	0
loc	16
rs	10
c	1
b	0
f	0
cc	3
nc	4
nop	0

<?php

use Ffcms\Core\Migrations\MigrationInterface;
use Ffcms\Core\Migrations\Migration;

/**
 * Class update_cms_301.
 */
class update_cms_301 extends Migration implements MigrationInterface
namespace YourVendor;

class YourClass { }
{
    /**
     * Add database changes, provided in time between 3.0.0 and 3.0.1 release is done
     * @return void
     */
    public function up()
    {
        // use important column for content app
        if (!$this->getSchema()->hasColumn('contents', 'important')) {
            $this->getSchema()->table('contents', function ($table) {
                $table->boolean('important')->default(false);
            });
        }
        // add group display color features
        if (!$this->getSchema()->hasColumn('roles', 'color')) {
            $this->getSchema()->table('roles', function ($table) {
                $table->string('color')->default('#777777');
            });
        }

        parent::up();
    }

    /**
     * Seed created table via up() method with some data
     * @return void
     */
    public function seed()
    {

    }

    /**
     * Execute actions when migration is down
     * @return void
     */
    public function down()
    {
        if ($this->getSchema()->hasColumn('contents', 'important')) {
            $this->getSchema()->table('contents', function ($table) {
                $table->dropColumn('important');
            });
        }
        if ($this->getSchema()->hasColumn('roles', 'color')) {
            $this->getSchema()->table('roles', function ($table) {
                $table->dropColumn('color');
            });
        }
        parent::down();
    }
}

1			<?php
2
3			use Ffcms\Core\Migrations\MigrationInterface;
4			use Ffcms\Core\Migrations\Migration;
5
6			/**
7			* Class update_cms_301.
8			*/
9			class update_cms_301 extends Migration implements MigrationInterface
			0 ignored issues – show Coding Style Compatibility introduced 2018-01-08 19:39 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
10			{
11			/**
12			* Add database changes, provided in time between 3.0.0 and 3.0.1 release is done
13			* @return void
14			*/
15			public function up()
16			{
17			// use important column for content app
18			if (!$this->getSchema()->hasColumn('contents', 'important')) {
19			$this->getSchema()->table('contents', function ($table) {
20			$table->boolean('important')->default(false);
21			});
22			}
23			// add group display color features
24			if (!$this->getSchema()->hasColumn('roles', 'color')) {
25			$this->getSchema()->table('roles', function ($table) {
26			$table->string('color')->default('#777777');
27			});
28			}
29
30			parent::up();
31			}
32
33			/**
34			* Seed created table via up() method with some data
35			* @return void
36			*/
37			public function seed()
38			{
39
40			}
41
42			/**
43			* Execute actions when migration is down
44			* @return void
45			*/
46			public function down()
47			{
48			if ($this->getSchema()->hasColumn('contents', 'important')) {
49			$this->getSchema()->table('contents', function ($table) {
50			$table->dropColumn('important');
51			});
52			}
53			if ($this->getSchema()->hasColumn('roles', 'color')) {
54			$this->getSchema()->table('roles', function ($table) {
55			$table->dropColumn('color');
56			});
57			}
58			parent::down();
59			}
60			}

phpffcms / ffcms

Push — master ( e3c39f...fe570d )

update_cms_301::up() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like