SubObject::toContainerSemanticData() - Code Metrics - Inspection of "Add sane way to construct subobjects" - JeroenDeDauw/Maps - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — SemanticGeoJson ( e5361a...e30e84 )

by Jeroen De

created 2020-02-13 23:40 UTC

SubObject::toContainerSemanticData() A

↳ Parent: SubObject

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	14
c	0
b	0
f	0
ccs	7
cts	7
cp	1
rs	9.7998
cc	3
nc	3
nop	1
crap	3

<?php

declare( strict_types = 1 );

namespace Maps\DataAccess\GeoJsonStore;

use SMW\DataModel\ContainerSemanticData;
use SMW\DIProperty;
use SMW\DIWikiPage;
use TitleValue;

class SubObject {

	private $name;
	private $values = [];

	public function __construct( string $name ) {
		$this->name = $name;
	}

	public function addPropertyValuePair( string $propertyName, \SMWDataItem $dataItem ) {
		$this->values[$propertyName][] = $dataItem;
	}

	public function toContainerSemanticData( TitleValue $subjectPage ): ContainerSemanticData {
		$container = $this->newContainerSemanticData( $subjectPage );

		foreach ( $this->values as $propertyName => $dataItems ) {
			foreach ( $dataItems as $dataItem ) {
				$container->addPropertyObjectValue(
					new DIProperty( $propertyName ),
					$dataItem
				);
			}
		}

		return $container;
	}

	private function newContainerSemanticData( TitleValue $subjectPage ): ContainerSemanticData {
		return new ContainerSemanticData(
			new DIWikiPage(
				$subjectPage->getDBkey(),
				$subjectPage->getNamespace(),
				$subjectPage->getInterwiki(),
				$this->name
			)
		);
	}

}


1		<?php
2
3		declare( strict_types = 1 );
4
5		namespace Maps\DataAccess\GeoJsonStore;
6
7		use SMW\DataModel\ContainerSemanticData;
8		use SMW\DIProperty;
9		use SMW\DIWikiPage;
10		use TitleValue;
11
12		class SubObject {
13
14		private $name;
15		private $values = [];
16
17	2	public function __construct( string $name ) {
18	2	$this->name = $name;
19	2	}
20
21	1	public function addPropertyValuePair( string $propertyName, \SMWDataItem $dataItem ) {
22	1	$this->values[$propertyName][] = $dataItem;
23	1	}
24
25	2	public function toContainerSemanticData( TitleValue $subjectPage ): ContainerSemanticData {
26	2	$container = $this->newContainerSemanticData( $subjectPage );
27
28	2	foreach ( $this->values as $propertyName => $dataItems ) {
29	1	foreach ( $dataItems as $dataItem ) {
30	1	$container->addPropertyObjectValue(
31	1	new DIProperty( $propertyName ),
32		$dataItem
33		);
34		}
35		}
36
37	2	return $container;
38		}
39
40	2	private function newContainerSemanticData( TitleValue $subjectPage ): ContainerSemanticData {
41	2	return new ContainerSemanticData(
42	2	new DIWikiPage(
43	2	$subjectPage->getDBkey(),
44	2	$subjectPage->getNamespace(),
45	2	$subjectPage->getInterwiki(),
46	2	$this->name
47		)
48		);
49		}
50
51		}
52

JeroenDeDauw / Maps

Push — SemanticGeoJson ( e5361a...e30e84 )

SubObject::toContainerSemanticData() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like