remove_database_or::update_data() - Code Metrics - Inspection of "Merge pull request #19 from iMattPro/develop" - iMattPro/dbtool - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (af0060)

by Matt

created 2022-06-19 06:13 UTC

remove_database_or::update_data() A

↳ Parent: remove_database_or

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	0	Features	0

Metric	Value
cc	1
eloc	8
c	5
b	0
f	0
nc	1
nop	0
dl	0
loc	14
rs	10

<?php
/**
*
* Database Optimize & Repair Tool
*
* @copyright (c) 2013 Matt Friedman
* @license GNU General Public License, version 2 (GPL-2.0)
*
*/

namespace vse\dbtool\migrations;

class remove_database_or extends \phpbb\db\migration\container_aware_migration
filter:
    dependency_paths: ["lib/*"]
{
	public function effectively_installed()
	{
		return !isset($this->config['database_or_version']);
	}

	public static function depends_on()
	{
		return array('\phpbb\db\migration\data\v310\rc2');
	}

	public function update_data()
	{
		// use module tool explicitly since module.exists does not work in 'if'
		$module_tool = $this->container->get('migrator.tool.module');

		return array(
			array('if', array(
				$module_tool->exists('acp', false, 'ACP_DATABASE_OR', true),
				array('module.remove', array('acp', false, 'ACP_DATABASE_OR')),
			)),

			array('if', array(
				$this->config->offsetExists('database_or_version'),
				array('config.remove', array('database_or_version')),
			)),
		);
	}
}


1			<?php
2			/**
3			*
4			* Database Optimize & Repair Tool
5			*
6			* @copyright (c) 2013 Matt Friedman
7			* @license GNU General Public License, version 2 (GPL-2.0)
8			*
9			*/
10
11			namespace vse\dbtool\migrations;
12
13			class remove_database_or extends \phpbb\db\migration\container_aware_migration
			0 ignored issues – show Bug introduced 2022-06-19 06:17 UTC by Report Bug Copy Issue Report The type `phpbb\db\migration\container_aware_migration` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
14			{
15			public function effectively_installed()
16			{
17			return !isset($this->config['database_or_version']);
18			}
19
20			public static function depends_on()
21			{
22			return array('\phpbb\db\migration\data\v310\rc2');
23			}
24
25			public function update_data()
26			{
27			// use module tool explicitly since module.exists does not work in 'if'
28			$module_tool = $this->container->get('migrator.tool.module');
29
30			return array(
31			array('if', array(
32			$module_tool->exists('acp', false, 'ACP_DATABASE_OR', true),
33			array('module.remove', array('acp', false, 'ACP_DATABASE_OR')),
34			)),
35
36			array('if', array(
37			$this->config->offsetExists('database_or_version'),
38			array('config.remove', array('database_or_version')),
39			)),
40			);
41			}
42			}
43

iMattPro / dbtool

Branch — master (af0060)

remove_database_or::update_data() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like