BaseFillableElement::hasFillOpacity() - Code Metrics - Inspection of "Reorganize namespaces" - JeroenDeDauw/Maps - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 81538e...c2bf57 )

by Jeroen De

created 2020-04-13 00:34 UTC

BaseFillableElement::hasFillOpacity() A

↳ Parent: BaseFillableElement

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	3
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	0

<?php

declare( strict_types = 1 );

namespace Maps\LegacyModel;

/**
 * @since 2.0
 */
class BaseFillableElement extends BaseStrokableElement {

	protected $fillColor;
	protected $fillOpacity;

	public function getJSONObject( string $defText = '', string $defTitle = '' ): array {
		$parentArray = parent::getJSONObject( $defText, $defTitle );
		$array = [
			'fillColor' => $this->hasFillColor() ? $this->getFillColor() : '#FF0000',
			'fillOpacity' => $this->hasFillOpacity() ? $this->getFillOpacity() : '0.5',
		];
		return array_merge( $parentArray, $array );
	}

	public function hasFillColor() {
		return !is_null( $this->fillColor ) && $this->fillColor !== '';
	}

	public function getFillColor() {
		return $this->fillColor;
	}

	public function setFillColor( $fillColor ) {
		$this->fillColor = trim( $fillColor );
	}

	public function hasFillOpacity() {
		return !is_null( $this->fillOpacity ) && $this->fillOpacity !== '';
	}

	public function getFillOpacity() {
		return $this->fillOpacity;
	}

	public function setFillOpacity( $fillOpacity ) {
		$this->fillOpacity = trim( $fillOpacity );
	}
}


1			<?php
2
3			declare( strict_types = 1 );
4
5			namespace Maps\LegacyModel;
6
7			/**
8			* @since 2.0
9			*/
10			class BaseFillableElement extends BaseStrokableElement {
11
12			protected $fillColor;
13			protected $fillOpacity;
14
15			public function getJSONObject( string $defText = '', string $defTitle = '' ): array {
16			$parentArray = parent::getJSONObject( $defText, $defTitle );
17			$array = [
18			'fillColor' => $this->hasFillColor() ? $this->getFillColor() : '#FF0000',
19			'fillOpacity' => $this->hasFillOpacity() ? $this->getFillOpacity() : '0.5',
20			];
21			return array_merge( $parentArray, $array );
22			}
23
24			public function hasFillColor() {
25			return !is_null( $this->fillColor ) && $this->fillColor !== '';
26			}
27
28			public function getFillColor() {
29			return $this->fillColor;
30			}
31
32			public function setFillColor( $fillColor ) {
33			$this->fillColor = trim( $fillColor );
34			}
35
36			public function hasFillOpacity() {
37			return !is_null( $this->fillOpacity ) && $this->fillOpacity !== '';
38			}
39
40			public function getFillOpacity() {
41			return $this->fillOpacity;
42			}
43
44			public function setFillOpacity( $fillOpacity ) {
45			$this->fillOpacity = trim( $fillOpacity );
46			}
47			}
48

JeroenDeDauw / Maps

Push — master ( 81538e...c2bf57 )

BaseFillableElement::hasFillOpacity() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like