Seed::__construct() - Code Metrics - Inspection of "Merge pull request #11567 from mrclay/11563_routes" - Elgg/Elgg - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c0a3a7...3b84a4 )

by Jeroen

created 2018-01-08 10:08 UTC

Seed::__construct() A

↳ Parent: Seed

Complexity

Conditions	2
Paths	2

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
cc	2
eloc	2
nc	2
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0
ccs	0
cts	3
cp	0
crap	6

<?php

namespace Elgg\Database\Seeds;

/**
 * Abstract seed
 *
 * Plugins should extend this class to create their own seeders,
 * add use 'seeds','database' plugin hook to add their seed to the sequence.
 */
abstract class Seed implements Seedable {

	use Seeding;

	/**
	 * Seed constructor.
	 *
	 * @param null $limit Number of item to seed

	 */
	public function __construct($limit = null) {
		if (isset($limit)) {
			$this->limit = $limit;
		}
	}

	/**
	 * Populate database
	 * @return mixed
	 */
	abstract function seed();


	/**
	 * Removed seeded rows from database
	 * @return mixed
	 */
	abstract function unseed();


}


1			<?php
2
3			namespace Elgg\Database\Seeds;
4
5			/**
6			* Abstract seed
7			*
8			* Plugins should extend this class to create their own seeders,
9			* add use 'seeds','database' plugin hook to add their seed to the sequence.
10			*/
11			abstract class Seed implements Seedable {
12
13			use Seeding;
14
15			/**
16			* Seed constructor.
17			*
18			* @param null $limit Number of item to seed
			0 ignored issues – show Documentation Bug introduced 2017-10-16 06:19 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$limit` is correct as it would always require `null` to be passed? Loading history...
19			*/
20			public function __construct($limit = null) {
21			if (isset($limit)) {
22			$this->limit = $limit;
23			}
24			}
25
26			/**
27			* Populate database
28			* @return mixed
29			*/
30			abstract function seed();
			0 ignored issues – show Best Practice introduced 2017-10-16 06:19 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
31
32			/**
33			* Removed seeded rows from database
34			* @return mixed
35			*/
36			abstract function unseed();
			0 ignored issues – show Best Practice introduced 2017-10-16 06:19 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
37
38			}
39

Elgg / Elgg

Push — master ( c0a3a7...3b84a4 )

Seed::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like