BusinessContact::makePersonNameDescription() - Code Metrics - Inspection of "Dev6.4 new models" - linkeddatacenter/BOTK-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#8)

by Enrico

created 2017-08-09 14:18 UTC

BusinessContact::makePersonNameDescription() B

↳ Parent: BusinessContact

Complexity

Conditions	5
Paths	9

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	5

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	13
ccs	8
cts	8
cp	1
rs	8.8571
cc	5
eloc	8
nc	9
nop	0
crap	5

<?php
namespace BOTK\Model;


/**
 * An ibrid class that merge the semantic of schema:organization, schema:place and schema:geo, 
 * it is similar to schema:LocalBusiness.
 * Allows the bulk setup of properties
 */
class BusinessContact extends AbstractModel implements \BOTK\ModelInterface 
{

	protected static $DEFAULT_OPTIONS = array (
		'personType'		=> array(		
			// additional types  as extension of schema:Person
			'filter'    => FILTER_DEFAULT,
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'taxID'				=> array(	
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_TOKEN',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'alternateName'		=> array(
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'givenName'				=> array(
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'familyName'				=> array(
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'additionalName'			=> array(
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'jobTitle'	=> array(
			'filter'    => FILTER_DEFAULT,
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'honorificPrefix'	=> array(
			'filter'    => FILTER_DEFAULT,
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'honorificSuffix'	=> array(
			'filter'    => FILTER_DEFAULT,
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'gender'	=> array(	
			'filter'    => FILTER_CALLBACK,
			'options'    => '\BOTK\Filters::FILTER_SANITIZE_GENDER',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'worksFor'	=> array(		
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_VALIDATE_URI',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'telephone'			=> array(	
			'filter'    => FILTER_CALLBACK,	
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_TELEPHONE',
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'email'				=> array(	
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_EMAIL',
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'spokenLanguage'			=> array(	
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_LANGUAGE',
			'flags'  	=> FILTER_FORCE_ARRAY,
			),
		'hasOptInOptOutDate'				=> array(	
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_DATETIME',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		'privacyFlag'		=> array(	
			'filter'    => FILTER_CALLBACK,
			'options' 	=> '\BOTK\Filters::FILTER_SANITIZE_BOOLEAN',
			'flags'  	=> FILTER_REQUIRE_SCALAR,
			),
		);


	private function makePersonNameDescription()
	{	
		extract($this->data);
			
		if(empty($alternateName)){
			$alternateName ='';
			if(!empty($givenName)) { $alternateName.= "$givenName ";}						
			if(!empty($additionalName)) { $alternateName.= "$additionalName ";}		
			if(!empty($familyName)) { $alternateName.= "$familyName ";}
		}
		
		return \BOTK\Filters::FILTER_SANITIZE_PERSON_NAME($alternateName);
	}
	
	
	public function asTurtleFragment()
	{
		if(is_null($this->rdf)) {
			extract($this->data);

			// create uris
			$personUri = $this->getUri();
			
			// make a default for altenatename (can be empty)
			$alternateName = $this->makePersonNameDescription();
			
			$tripleCounter =0;
			
			// define $_ as a macro to write simple rdf
			$_= function($format, $var,$sanitize=true) use(&$turtleString, &$tripleCounter){
				foreach((array)$var as $v){
					if($var){
						$turtleString.= sprintf($format,$sanitize?\BOTK\Filters::FILTER_SANITIZE_TURTLE_STRING($v):$v);
						$tripleCounter++;
					}
				}
			};

	 		// serialize schema:LocalBusiness
	 		
			$turtleString= "<$personUri> ";			
$x = 1;
$callable = function() use ($x) {
    $x = 2; // Not visible in outer scope. If you would like this, how
            // about using a different variable name than $x?
};

$callable();
var_dump($x); // integer(1)
			!empty($id) 				&& $_('dct:identifier "%s";', $id); 
			!empty($disambiguatingDescription)&& $_('schema:disambiguatingDescription "%s";', $disambiguatingDescription);
			!empty($aggregateRatingValue)&& $_('schema:aggregateRating [a schema:AggregateRating; schema:ratingValue "%s"^^xsd:float];', $aggregateRatingValue);
			!empty($page) 				&& $_('foaf:page <%s>;', $page,false);
			!empty($homepage) 			&& $_('foaf:homepage <%s>;', $homepage,false);
			!empty($personType) 		&& $_('a %s;', $personType);
			!empty($taxID) 				&& $_('schema:taxID "%s";', $taxID);
			!empty($givenName)			&& $_('schema:givenName "%s";', $givenName);
			!empty($familyName)			&& $_('schema:familyName "%s";', $familyName);
			!empty($additionalName) 	&& $_('schema:additionalName "%s";', $additionalName);
			!empty($alternateName) 		&& $_('schema:alternateName "%s";', $alternateName);
			!empty($telephone) 			&& $_('schema:telephone "%s";', $telephone);
			!empty($faxNumber) 			&& $_('schema:faxNumber "%s";', $faxNumber);
			!empty($jobTitle)			&& $_('schema:jobTitle "%s";', $jobTitle);
			!empty($honorificPrefix)	&& $_('schema:honorificPrefix "%s";', $honorificPrefix);
			!empty($honorificSuffix)	&& $_('schema:honorificSuffix "%s";', $honorificSuffix);
			!empty($email) 				&& $_('schema:email "%s";', $email);
			!empty($gender)				&& $_('schema:gender "%s";', $gender);
			!empty($worksFor)			&& $_('schema:worksFor <%s> ;', $worksFor,false);
			!empty($spokenLanguage)		&& $_('botk:spokenLanguage "%s";', $spokenLanguage);
			!empty($hasOptInOptOutDate)	&& $_('botk:hasOptInOptOutDate "%s";', $hasOptInOptOutDate);
			!empty($privacyFlag)		&& $_('botk:privacyFlag <%s>;', $privacyFlag);		
			$_('a schema:Person .', $personUri);

			$this->rdf = $turtleString;
			$this->tripleCount = $tripleCounter;
		}

		return $this->rdf;
	}

}

1		<?php
2		namespace BOTK\Model;
3
4
5		/**
6		* An ibrid class that merge the semantic of schema:organization, schema:place and schema:geo,
7		* it is similar to schema:LocalBusiness.
8		* Allows the bulk setup of properties
9		*/
10		class BusinessContact extends AbstractModel implements \BOTK\ModelInterface
11		{
12
13		protected static $DEFAULT_OPTIONS = array (
14		'personType' => array(
15		// additional types as extension of schema:Person
16		'filter' => FILTER_DEFAULT,
17		'flags' => FILTER_FORCE_ARRAY,
18		),
19		'taxID' => array(
20		'filter' => FILTER_CALLBACK,
21		'options' => '\BOTK\Filters::FILTER_SANITIZE_TOKEN',
22		'flags' => FILTER_REQUIRE_SCALAR,
23		),
24		'alternateName' => array(
25		'filter' => FILTER_CALLBACK,
26		'options' => '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
27		'flags' => FILTER_FORCE_ARRAY,
28		),
29		'givenName' => array(
30		'filter' => FILTER_CALLBACK,
31		'options' => '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
32		'flags' => FILTER_REQUIRE_SCALAR,
33		),
34		'familyName' => array(
35		'filter' => FILTER_CALLBACK,
36		'options' => '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
37		'flags' => FILTER_REQUIRE_SCALAR,
38		),
39		'additionalName' => array(
40		'filter' => FILTER_CALLBACK,
41		'options' => '\BOTK\Filters::FILTER_SANITIZE_PERSON_NAME',
42		'flags' => FILTER_REQUIRE_SCALAR,
43		),
44		'jobTitle' => array(
45		'filter' => FILTER_DEFAULT,
46		'flags' => FILTER_FORCE_ARRAY,
47		),
48		'honorificPrefix' => array(
49		'filter' => FILTER_DEFAULT,
50		'flags' => FILTER_FORCE_ARRAY,
51		),
52		'honorificSuffix' => array(
53		'filter' => FILTER_DEFAULT,
54		'flags' => FILTER_FORCE_ARRAY,
55		),
56		'gender' => array(
57		'filter' => FILTER_CALLBACK,
58		'options' => '\BOTK\Filters::FILTER_SANITIZE_GENDER',
59		'flags' => FILTER_REQUIRE_SCALAR,
60		),
61		'worksFor' => array(
62		'filter' => FILTER_CALLBACK,
63		'options' => '\BOTK\Filters::FILTER_VALIDATE_URI',
64		'flags' => FILTER_REQUIRE_SCALAR,
65		),
66		'telephone' => array(
67		'filter' => FILTER_CALLBACK,
68		'options' => '\BOTK\Filters::FILTER_SANITIZE_TELEPHONE',
69		'flags' => FILTER_FORCE_ARRAY,
70		),
71		'email' => array(
72		'filter' => FILTER_CALLBACK,
73		'options' => '\BOTK\Filters::FILTER_SANITIZE_EMAIL',
74		'flags' => FILTER_FORCE_ARRAY,
75		),
76		'spokenLanguage' => array(
77		'filter' => FILTER_CALLBACK,
78		'options' => '\BOTK\Filters::FILTER_SANITIZE_LANGUAGE',
79		'flags' => FILTER_FORCE_ARRAY,
80		),
81		'hasOptInOptOutDate' => array(
82		'filter' => FILTER_CALLBACK,
83		'options' => '\BOTK\Filters::FILTER_SANITIZE_DATETIME',
84		'flags' => FILTER_REQUIRE_SCALAR,
85		),
86		'privacyFlag' => array(
87		'filter' => FILTER_CALLBACK,
88		'options' => '\BOTK\Filters::FILTER_SANITIZE_BOOLEAN',
89		'flags' => FILTER_REQUIRE_SCALAR,
90		),
91		);
92
93
94	4	private function makePersonNameDescription()
95		{
96	4	extract($this->data);
97
98	4	if(empty($alternateName)){
99	3	$alternateName ='';
100	3	if(!empty($givenName)) { $alternateName.= "$givenName ";}
101	3	if(!empty($additionalName)) { $alternateName.= "$additionalName ";}
102	3	if(!empty($familyName)) { $alternateName.= "$familyName ";}
103		}
104
105	4	return \BOTK\Filters::FILTER_SANITIZE_PERSON_NAME($alternateName);
106		}
107
108
109	4	public function asTurtleFragment()
110		{
111	4	if(is_null($this->rdf)) {
112	4	extract($this->data);
113
114		// create uris
115	4	$personUri = $this->getUri();
116
117		// make a default for altenatename (can be empty)
118	4	$alternateName = $this->makePersonNameDescription();
119
120	4	$tripleCounter =0;
121
122		// define $_ as a macro to write simple rdf
123	4	$_= function($format, $var,$sanitize=true) use(&$turtleString, &$tripleCounter){
124	4	foreach((array)$var as $v){
125	4	if($var){
126	4	$turtleString.= sprintf($format,$sanitize?\BOTK\Filters::FILTER_SANITIZE_TURTLE_STRING($v):$v);
127	4	$tripleCounter++;
128		}
129		}
130	4	};
131
132		// serialize schema:LocalBusiness
133
134		$turtleString= "<$personUri> ";
		0 ignored issues – show Bug introduced 2017-08-09 14:20 UTC by Report Bug Copy Issue Report Consider using a different name than the imported variable `$turtleString`, or did you forget to import by reference? It seems like you are assigning to a variable which was imported through a `use` statement which was not imported by reference. For clarity, we suggest to use a different name or import by reference depending on whether you would like to have the change visibile in outer-scope. Change not visible in outer-scope $x = 1; $callable = function() use ($x) { $x = 2; // Not visible in outer scope. If you would like this, how // about using a different variable name than $x? }; $callable(); var_dump($x); // integer(1) Change visible in outer-scope $x = 1; $callable = function() use (&$x) { $x = 2; }; $callable(); var_dump($x); // integer(2) Loading history...
135		!empty($id) && $_('dct:identifier "%s";', $id);
136		!empty($disambiguatingDescription)&& $_('schema:disambiguatingDescription "%s";', $disambiguatingDescription);
137		!empty($aggregateRatingValue)&& $_('schema:aggregateRating [a schema:AggregateRating; schema:ratingValue "%s"^^xsd:float];', $aggregateRatingValue);
138		!empty($page) && $_('foaf:page <%s>;', $page,false);
139		!empty($homepage) && $_('foaf:homepage <%s>;', $homepage,false);
140		!empty($personType) && $_('a %s;', $personType);
141		!empty($taxID) && $_('schema:taxID "%s";', $taxID);
142		!empty($givenName) && $_('schema:givenName "%s";', $givenName);
143		!empty($familyName) && $_('schema:familyName "%s";', $familyName);
144		!empty($additionalName) && $_('schema:additionalName "%s";', $additionalName);
145		!empty($alternateName) && $_('schema:alternateName "%s";', $alternateName);
146		!empty($telephone) && $_('schema:telephone "%s";', $telephone);
147		!empty($faxNumber) && $_('schema:faxNumber "%s";', $faxNumber);
148		!empty($jobTitle) && $_('schema:jobTitle "%s";', $jobTitle);
149		!empty($honorificPrefix) && $_('schema:honorificPrefix "%s";', $honorificPrefix);
150		!empty($honorificSuffix) && $_('schema:honorificSuffix "%s";', $honorificSuffix);
151		!empty($email) && $_('schema:email "%s";', $email);
152		!empty($gender) && $_('schema:gender "%s";', $gender);
153		!empty($worksFor) && $_('schema:worksFor <%s> ;', $worksFor,false);
154		!empty($spokenLanguage) && $_('botk:spokenLanguage "%s";', $spokenLanguage);
155		!empty($hasOptInOptOutDate) && $_('botk:hasOptInOptOutDate "%s";', $hasOptInOptOutDate);
156		!empty($privacyFlag) && $_('botk:privacyFlag <%s>;', $privacyFlag);
157		$_('a schema:Person .', $personUri);
158
159		$this->rdf = $turtleString;
160		$this->tripleCount = $tripleCounter;
161		}
162
163		return $this->rdf;
164		}
165
166		}

linkeddatacenter / BOTK-core

Pull Request — master (#8)

BusinessContact::makePersonNameDescription() B

Complexity

Size

Duplication

Code Coverage

Importance

Change not visible in outer-scope

Change visible in outer-scope

Duplication Side-by-Side

Filter issues like