CustomerRemoveLostUserDataFosuser::migrate() - Code Metrics - Inspection of "Improved cleanup and fixed tests" - aimeos/ai-fosuser - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5df6aa...ddc494 )

by Aimeos

created 2019-02-19 12:53 UTC

CustomerRemoveLostUserDataFosuser::migrate() A

↳ Parent: CustomerRemoveLostUserDataFosuser

Complexity

Conditions	4
Paths	3

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

/**
 * @license LGPLv3, http://opensource.org/licenses/LGPL-3.0
 * @copyright Aimeos (aimeos.org), 2018
 */


namespace Aimeos\MW\Setup\Task;


/**
 * Removes address and list records without user entry
 */
class CustomerRemoveLostUserDataFosuser extends \Aimeos\MW\Setup\Task\Base
{
	private $sql = [
		'fos_user_address' => 'DELETE FROM "fos_user_address" WHERE NOT EXISTS ( SELECT "id" FROM "fos_user" AS u WHERE "parentid"=u."id" )',
		'fos_user_list' => 'DELETE FROM "fos_user_list" WHERE NOT EXISTS ( SELECT "id" FROM "fos_user" AS u WHERE "parentid"=u."id" )',
		'fos_user_property' => 'DELETE FROM "fos_user_property" WHERE NOT EXISTS ( SELECT "id" FROM "fos_user" AS u WHERE "parentid"=u."id" )',
	];


	/**
	 * Returns the list of task names which depends on this task.
	 *
	 * @return string[] List of task names
	 */
	public function getPostDependencies()
	{
		return ['TablesCreateMShop'];
	}


	/**
	 * Migrate database schema
	 */
	public function migrate()
	{
		$this->msg( 'Remove left over FOS user references', 0, '' );

		foreach( $this->sql as $table => $stmt )
		{
			$this->msg( sprintf( 'Remove unused %1$s records', $table ), 1 );

			if( $this->schema->tableExists( 'fos_user' ) && $this->schema->tableExists( $table ) )
			{
				$this->execute( $stmt );
				$this->status( 'done' );
			}
			else
			{
				$this->status( 'OK' );
			}
		}
	}
}


1			<?php
2
3			/**
4			* @license LGPLv3, http://opensource.org/licenses/LGPL-3.0
5			* @copyright Aimeos (aimeos.org), 2018
6			*/
7
8
9			namespace Aimeos\MW\Setup\Task;
10
11
12			/**
13			* Removes address and list records without user entry
14			*/
15			class CustomerRemoveLostUserDataFosuser extends \Aimeos\MW\Setup\Task\Base
16			{
17			private $sql = [
18			'fos_user_address' => 'DELETE FROM "fos_user_address" WHERE NOT EXISTS ( SELECT "id" FROM "fos_user" AS u WHERE "parentid"=u."id" )',
19			'fos_user_list' => 'DELETE FROM "fos_user_list" WHERE NOT EXISTS ( SELECT "id" FROM "fos_user" AS u WHERE "parentid"=u."id" )',
20			'fos_user_property' => 'DELETE FROM "fos_user_property" WHERE NOT EXISTS ( SELECT "id" FROM "fos_user" AS u WHERE "parentid"=u."id" )',
21			];
22
23
24			/**
25			* Returns the list of task names which depends on this task.
26			*
27			* @return string[] List of task names
28			*/
29			public function getPostDependencies()
30			{
31			return ['TablesCreateMShop'];
32			}
33
34
35			/**
36			* Migrate database schema
37			*/
38			public function migrate()
39			{
40			$this->msg( 'Remove left over FOS user references', 0, '' );
41
42			foreach( $this->sql as $table => $stmt )
43			{
44			$this->msg( sprintf( 'Remove unused %1$s records', $table ), 1 );
45
46			if( $this->schema->tableExists( 'fos_user' ) && $this->schema->tableExists( $table ) )
47			{
48			$this->execute( $stmt );
49			$this->status( 'done' );
50			}
51			else
52			{
53			$this->status( 'OK' );
54			}
55			}
56			}
57			}
58

aimeos / ai-fosuser

Push — master ( 5df6aa...ddc494 )

CustomerRemoveLostUserDataFosuser::migrate() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like