BanMuteSupportIssue157::up() - Code Metrics - allejo/bzion - Measure and Improve Code Quality continuously with Scrutinizer

BanMuteSupportIssue157::up() A
last analyzed 2018-04-10 03:56 UTC

↳ Parent: BanMuteSupportIssue157

Complexity

Conditions	1
Paths	1

Size

Total Lines	17
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	17
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	11
nc	1
nop	0

<?php

use Phinx\Migration\AbstractMigration;

/**
 * Restructure the `bans` table to better support bans and mutes.
 *
 * - The `allow_server_join` column was never correctly used and therefore has been dropped; due to incorrect use, the
 *   data did not need to be migrated or saved.
 * - The `is_soft_ban` column is being added to differentiate between bans that should penalize players on the site
 *   itself and bans that will only affect players on league servers. A soft ban will not affect a player on BZiON.
 *
 * @link https://github.com/allejo/bzion/issues/157
 */
class BanMuteSupportIssue157 extends AbstractMigration
{
    public function up()
    {
        $bansTable = $this->table('bans');
        $bansTable
            ->removeColumn('allow_server_join')
            ->addColumn('is_soft_ban', 'boolean', [
                'after' => 'author',
                'null' => false,
                'default' => false,
                'comment' => 'A soft ban will not penalize a user on the site or servers',
            ])
            ->update()
        ;

        // Unban any currently banned players
        $this->query("UPDATE players SET status = 'active' WHERE status = 'banned';");
    }

    public function down()
    {
        $bansTable = $this->table('bans');
        $bansTable
            ->removeColumn('is_soft_ban')
            ->addColumn('allow_server_join', 'boolean', [
                'after' => 'reason',
                'null' => false,
                'default' => false,
                'comment' => 'Whether or not to allow players to join servers while banned'
            ])
            ->update()
        ;
    }
}


1			<?php
2
3			use Phinx\Migration\AbstractMigration;
4
5			/**
6			* Restructure the `bans` table to better support bans and mutes.
7			*
8			* - The `allow_server_join` column was never correctly used and therefore has been dropped; due to incorrect use, the
9			* data did not need to be migrated or saved.
10			* - The `is_soft_ban` column is being added to differentiate between bans that should penalize players on the site
11			* itself and bans that will only affect players on league servers. A soft ban will not affect a player on BZiON.
12			*
13			* @link https://github.com/allejo/bzion/issues/157
14			*/
15			class BanMuteSupportIssue157 extends AbstractMigration
16			{
17			public function up()
18			{
19			$bansTable = $this->table('bans');
20			$bansTable
21			->removeColumn('allow_server_join')
22			->addColumn('is_soft_ban', 'boolean', [
23			'after' => 'author',
24			'null' => false,
25			'default' => false,
26			'comment' => 'A soft ban will not penalize a user on the site or servers',
27			])
28			->update()
29			;
30
31			// Unban any currently banned players
32			$this->query("UPDATE players SET status = 'active' WHERE status = 'banned';");
33			}
34
35			public function down()
36			{
37			$bansTable = $this->table('bans');
38			$bansTable
39			->removeColumn('is_soft_ban')
40			->addColumn('allow_server_join', 'boolean', [
41			'after' => 'reason',
42			'null' => false,
43			'default' => false,
44			'comment' => 'Whether or not to allow players to join servers while banned'
45			])
46			->update()
47			;
48			}
49			}
50

allejo / bzion

BanMuteSupportIssue157::up() A last analyzed 2018-04-10 03:56 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

BanMuteSupportIssue157::up() A
last analyzed 2018-04-10 03:56 UTC