CustomerRemoveLostUserDataLaravel::migrate() - Code Metrics - Inspection of "Remove left over address and list records before a..." - aimeos/ai-laravel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 384616...e3d84b )

by Aimeos

created 2018-07-06 09:30 UTC

CustomerRemoveLostUserDataLaravel::migrate() A

↳ Parent: CustomerRemoveLostUserDataLaravel

Complexity

Conditions	5
Paths	4

Size

Total Lines

Duplication

Lines	18
Ratio	66.67 %

Importance

Changes

Metric	Value
dl	18
loc	27
rs	9.1768
c	0
b	0
f	0
cc	5
nc	4
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 users entry
 */
class CustomerRemoveLostUserDataLaravel extends \Aimeos\MW\Setup\Task\Base
{
	private $sql = [
		'address' => 'DELETE FROM "users_address" WHERE NOT EXISTS ( SELECT "id" FROM "users" AS u WHERE "parentid"=u."id" )',
		'list' => 'DELETE FROM "users_list" WHERE NOT EXISTS ( SELECT "id" FROM "users" AS u WHERE "parentid"=u."id" )',
	];


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


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


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

		if( $this->schema->tableExists( 'users' ) && $this->schema->tableExists( 'users_address' ) )

		{
			$this->execute( $this->sql['address'] );
			$this->status( 'done' );
		}
		else
		{
			$this->status( 'OK' );
		}


		$this->msg( 'Remove left over Laravel user list records', 0 );

		if( $this->schema->tableExists( 'users' ) && $this->schema->tableExists( 'users_list' ) )

		{
			$this->execute( $this->sql['list'] );
			$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 users entry
14		*/
15		class CustomerRemoveLostUserDataLaravel extends \Aimeos\MW\Setup\Task\Base
16		{
17		private $sql = [
18		'address' => 'DELETE FROM "users_address" WHERE NOT EXISTS ( SELECT "id" FROM "users" AS u WHERE "parentid"=u."id" )',
19		'list' => 'DELETE FROM "users_list" WHERE NOT EXISTS ( SELECT "id" FROM "users" AS u WHERE "parentid"=u."id" )',
20		];
21
22
23		/**
24		* Returns the list of task names which this task depends on.
25		*
26		* @return array List of task names
27		*/
28		public function getPreDependencies()
29		{
30		return [];
31		}
32
33
34		/**
35		* Returns the list of task names which depends on this task.
36		*
37		* @return string[] List of task names
38		*/
39		public function getPostDependencies()
40		{
41		return array( 'TablesCreateMShop' );
42		}
43
44
45		/**
46		* Migrate database schema
47		*/
48		public function migrate()
49		{
50		$this->msg( 'Remove left over Laravel user address records', 0 );
51
52	View Code Duplication	if( $this->schema->tableExists( 'users' ) && $this->schema->tableExists( 'users_address' ) )
		0 ignored issues – show Deprecated Code introduced 2018-07-06 09:34 UTC by Report Bug Copy Issue Report The property `Aimeos\MW\Setup\Task\Base::$schema` has been deprecated with message: Use getSchema() instead This property has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the property will be removed from the class and what other property to use instead. Loading history... Duplication introduced 2018-07-06 09:34 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
53		{
54		$this->execute( $this->sql['address'] );
55		$this->status( 'done' );
56		}
57		else
58		{
59		$this->status( 'OK' );
60		}
61
62
63		$this->msg( 'Remove left over Laravel user list records', 0 );
64
65	View Code Duplication	if( $this->schema->tableExists( 'users' ) && $this->schema->tableExists( 'users_list' ) )
		0 ignored issues – show Deprecated Code introduced 2018-07-06 09:34 UTC by Report Bug Copy Issue Report The property `Aimeos\MW\Setup\Task\Base::$schema` has been deprecated with message: Use getSchema() instead This property has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the property will be removed from the class and what other property to use instead. Loading history... Duplication introduced 2018-07-06 09:34 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
66		{
67		$this->execute( $this->sql['list'] );
68		$this->status( 'done' );
69		}
70		else
71		{
72		$this->status( 'OK' );
73		}
74		}
75		}
76

aimeos / ai-laravel

Push — master ( 384616...e3d84b )

CustomerRemoveLostUserDataLaravel::migrate() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like