m1_initial_schema::update_schema() - Code Metrics - Inspection of "Fix migrations" - blitze/phpBB-ext-sitemaker - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 02e5e5...868d49 )

by Daniel

created 2016-03-14 15:09 UTC

m1_initial_schema::update_schema() B

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines	84
Code Lines	61

Duplication

Lines	0
Ratio	0 %

Importance

Changes	4
Bugs	1	Features	0

Metric	Value
c	4
b	1
f	0
dl	0
loc	84
rs	8.7169
cc	1
eloc	61
nc	1
nop	0

How to fix Long Method

<?php
/**
 *
 * @package sitemaker
 * @copyright (c) 2013 Daniel A. (blitze)
 * @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
 *
 */

namespace blitze\sitemaker\migrations\v20x;

class m1_initial_schema extends \phpbb\db\migration\migration
{
	/**
	 * @inheritdoc
	 */
	public static function depends_on()
	{
		return array(
			'\blitze\sitemaker\migrations\converter\c3_update_tables',
			'\blitze\sitemaker\migrations\v20x\m3_initial_permission',
		);
	}

	/**
	 * @inheritdoc
	 */
	public function update_schema()
	{
		return array(
			'add_tables'	=> array(
				$this->table_prefix . 'sm_block_routes' => array(
					'COLUMNS'		=> array(
						'route_id'		=> array('UINT', null, 'auto_increment'),
						'ext_name'		=> array('VCHAR:255', ''),
						'route'			=> array('XSTEXT_UNI', ''),
						'style'			=> array('USINT', 0),
						'hide_blocks'	=> array('BOOL', 0),
						'has_blocks'	=> array('BOOL', 0),
						'ex_positions'	=> array('VCHAR:255', ''),
					),

					'PRIMARY_KEY'	=> 'route_id'
				),

				$this->table_prefix . 'sm_blocks' => array(
					'COLUMNS'		=> array(
						'bid'			=> array('UINT', null, 'auto_increment'),
						'icon'			=> array('VCHAR:55', ''),
						'name'			=> array('VCHAR:55', ''),
						'title'			=> array('XSTEXT_UNI', ''),
						'route_id'		=> array('UINT', 0),
						'position'		=> array('VCHAR:55', ''),
						'weight'		=> array('USINT', 0),
						'style'			=> array('USINT', 0),
						'permission'	=> array('VCHAR:125', ''),
						'class'			=> array('VCHAR:125', ''),
						'status'		=> array('BOOL', 1),
						'type'			=> array('BOOL', 0),
						'no_wrap'		=> array('BOOL', 0),
						'hide_title'	=> array('BOOL', 0),
						'hash'			=> array('VCHAR:32', ''),
					),

					'PRIMARY_KEY'	=> 'bid',

					'KEYS'			=> array(
						'style'			=> array('INDEX', 'style'),
					),
				),

				$this->table_prefix . 'sm_menus' => array(
					'COLUMNS'        => array(
						'menu_id'			=> array('UINT', null, 'auto_increment'),
						'menu_name'			=> array('VCHAR:55', ''),
					),
					'PRIMARY_KEY'	=> 'menu_id',
					'KEYS'			=> array(
						'menu_name'			=> array('UNIQUE', 'menu_name'),
						'menu_id'			=> array('INDEX', 'menu_id'),
					),
				),

				$this->table_prefix . 'sm_menu_items' => array(
					'COLUMNS'        => array(
						'item_id'			=> array('UINT', null, 'auto_increment'),
						'menu_id'			=> array('UINT', 0),
						'group_id'			=> array('UINT', 0),
						'parent_id'			=> array('UINT', 0),
						'item_title'		=> array('XSTEXT_UNI', ''),
						'item_url'			=> array('VCHAR_UNI', ''),
						'item_icon'			=> array('VCHAR', ''),
						'item_desc'			=> array('VCHAR:55', ''),
						'item_target'		=> array('BOOL', 0),
						'left_id'			=> array('UINT', 0),
						'right_id'			=> array('UINT', 0),
						'depth'				=> array('UINT', 0),
					),
					'PRIMARY_KEY'	=> 'item_id',
					'KEYS'			=> array(
						'menu_id'		=> array('INDEX', 'menu_id'),
					),
				),
			),
			'add_columns'	=> array(
				$this->table_prefix . 'forums'	=> array(
					'hidden_forum'		=> array('BOOL', 0),
				),
			),
		);
	}

	/**
	 * @inheritdoc
	 */
	public function revert_schema()
	{
		return array(
			'drop_tables'	=> array(
				$this->table_prefix . 'sm_blocks',
				$this->table_prefix . 'sm_blocks_config',
				$this->table_prefix . 'sm_block_routes',
				$this->table_prefix . 'sm_menus',
				$this->table_prefix . 'sm_menu_items',
			),
		);
	}
}


1			<?php
2			/**
3			*
4			* @package sitemaker
5			* @copyright (c) 2013 Daniel A. (blitze)
6			* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
7			*
8			*/
9
10			namespace blitze\sitemaker\migrations\v20x;
11
12			class m1_initial_schema extends \phpbb\db\migration\migration
13			{
14			/**
15			* @inheritdoc
16			*/
17			public static function depends_on()
18			{
19			return array(
20			'\blitze\sitemaker\migrations\converter\c3_update_tables',
21			'\blitze\sitemaker\migrations\v20x\m3_initial_permission',
22			);
23			}
24
25			/**
26			* @inheritdoc
27			*/
28			public function update_schema()
29			{
30			return array(
31			'add_tables' => array(
32			$this->table_prefix . 'sm_block_routes' => array(
33			'COLUMNS' => array(
34			'route_id' => array('UINT', null, 'auto_increment'),
35			'ext_name' => array('VCHAR:255', ''),
36			'route' => array('XSTEXT_UNI', ''),
37			'style' => array('USINT', 0),
38			'hide_blocks' => array('BOOL', 0),
39			'has_blocks' => array('BOOL', 0),
40			'ex_positions' => array('VCHAR:255', ''),
41			),
42
43			'PRIMARY_KEY' => 'route_id'
44			),
45
46			$this->table_prefix . 'sm_blocks' => array(
47			'COLUMNS' => array(
48			'bid' => array('UINT', null, 'auto_increment'),
49			'icon' => array('VCHAR:55', ''),
50			'name' => array('VCHAR:55', ''),
51			'title' => array('XSTEXT_UNI', ''),
52			'route_id' => array('UINT', 0),
53			'position' => array('VCHAR:55', ''),
54			'weight' => array('USINT', 0),
55			'style' => array('USINT', 0),
56			'permission' => array('VCHAR:125', ''),
57			'class' => array('VCHAR:125', ''),
58			'status' => array('BOOL', 1),
59			'type' => array('BOOL', 0),
60			'no_wrap' => array('BOOL', 0),
61			'hide_title' => array('BOOL', 0),
62			'hash' => array('VCHAR:32', ''),
63			),
64
65			'PRIMARY_KEY' => 'bid',
66
67			'KEYS' => array(
68			'style' => array('INDEX', 'style'),
69			),
70			),
71
72			$this->table_prefix . 'sm_menus' => array(
73			'COLUMNS' => array(
74			'menu_id' => array('UINT', null, 'auto_increment'),
75			'menu_name' => array('VCHAR:55', ''),
76			),
77			'PRIMARY_KEY' => 'menu_id',
78			'KEYS' => array(
79			'menu_name' => array('UNIQUE', 'menu_name'),
80			'menu_id' => array('INDEX', 'menu_id'),
81			),
82			),
83
84			$this->table_prefix . 'sm_menu_items' => array(
85			'COLUMNS' => array(
86			'item_id' => array('UINT', null, 'auto_increment'),
87			'menu_id' => array('UINT', 0),
88			'group_id' => array('UINT', 0),
89			'parent_id' => array('UINT', 0),
90			'item_title' => array('XSTEXT_UNI', ''),
91			'item_url' => array('VCHAR_UNI', ''),
92			'item_icon' => array('VCHAR', ''),
93			'item_desc' => array('VCHAR:55', ''),
94			'item_target' => array('BOOL', 0),
95			'left_id' => array('UINT', 0),
96			'right_id' => array('UINT', 0),
97			'depth' => array('UINT', 0),
98			),
99			'PRIMARY_KEY' => 'item_id',
100			'KEYS' => array(
101			'menu_id' => array('INDEX', 'menu_id'),
102			),
103			),
104			),
105			'add_columns' => array(
106			$this->table_prefix . 'forums' => array(
107			'hidden_forum' => array('BOOL', 0),
108			),
109			),
110			);
111			}
112
113			/**
114			* @inheritdoc
115			*/
116			public function revert_schema()
117			{
118			return array(
119			'drop_tables' => array(
120			$this->table_prefix . 'sm_blocks',
121			$this->table_prefix . 'sm_blocks_config',
122			$this->table_prefix . 'sm_block_routes',
123			$this->table_prefix . 'sm_menus',
124			$this->table_prefix . 'sm_menu_items',
125			),
126			);
127			}
128			}
129

blitze / phpBB-ext-sitemaker

Push — master ( 02e5e5...868d49 )

m1_initial_schema::update_schema() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like