Inspection of "Merge pull request #306 from robmachado/master" - nfephp-org/sped-esocial - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( ce467a...63d89e )

by Roberto

created 2021-05-14 14:21 UTC

Status

Spacing +19 added lines, -19 removed lines patch added patch discarded remove patch

@@ -59,7 +59,7 @@  discard block
 block discarded – undo
         $this->dom->addChild(
             $ideEstab,
             "fimValid",
-            ! empty($this->std->fimvalid) ? $this->std->fimvalid : null,
+            !empty($this->std->fimvalid) ? $this->std->fimvalid : null,
             false
         );
 
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
         }
         $node->appendChild($ideEstab);
 
-        if (! empty($this->std->dadosestab)) {
+        if (!empty($this->std->dadosestab)) {
             $dadosEstab = $this->dom->createElement("dadosEstab");
             $this->dom->addChild(
                 $dadosEstab,
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
                 $this->std->dadosestab->aliqgilrat->aliqrat,
                 true
             );
-            $fap = ! empty($this->std->dadosestab->aliqgilrat->fap) ? $this->std->dadosestab->aliqgilrat->fap : null;
+            $fap = !empty($this->std->dadosestab->aliqgilrat->fap) ? $this->std->dadosestab->aliqgilrat->fap : null;
             if ($fap) {
                 $fap = number_format($fap, 4, '.', '');
             }
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
                 $fap,
                 false
             );
-            $aliqrata = ! empty($this->std->dadosestab->aliqgilrat->aliqratajust)
+            $aliqrata = !empty($this->std->dadosestab->aliqgilrat->aliqratajust)
                 ? $this->std->dadosestab->aliqgilrat->aliqratajust
                 : null;
             if ($aliqrata) {
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
                 $aliqrata,
                 false
             );
-            if (! empty($this->std->dadosestab->aliqgilrat->procadmjudrat)) {
+            if (!empty($this->std->dadosestab->aliqgilrat->procadmjudrat)) {
                 $procAdmJudRat = $this->dom->createElement("procAdmJudRat");
                 $this->dom->addChild(
                     $procAdmJudRat,
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
                 );
                 $aliqGilrat->appendChild($procAdmJudRat);
             }
-            if (! empty($this->std->dadosestab->aliqgilrat->procadmjudfap)) {
+            if (!empty($this->std->dadosestab->aliqgilrat->procadmjudfap)) {
                 $procAdmJudFap = $this->dom->createElement("procAdmJudFap");
                 $this->dom->addChild(
                     $procAdmJudFap,
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
             }
             $dadosEstab->appendChild($aliqGilrat);
 
-            if (! empty($this->std->dadosestab->infocaepf)) {
+            if (!empty($this->std->dadosestab->infocaepf)) {
                 $infoCaepf = $this->dom->createElement("infoCaepf");
                 $this->dom->addChild(
                     $infoCaepf,
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
                 $dadosEstab->appendChild($infoCaepf);
             }
 
-            if (! empty($this->std->dadosestab->infoobra)) {
+            if (!empty($this->std->dadosestab->infoobra)) {
                 $infoObra = $this->dom->createElement("infoObra");
                 $this->dom->addChild(
                     $infoObra,
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
                 $this->std->dadosestab->infotrab->regpt,
                 true
             );
-            if (! empty($this->std->dadosestab->infoapr)) {
+            if (!empty($this->std->dadosestab->infoapr)) {
                 $infoApr = $this->dom->createElement("infoApr");
                 $this->dom->addChild(
                     $infoApr,
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
                 $this->dom->addChild(
                     $infoApr,
                     "nrProcJud",
-                    ! empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
+                    !empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
                         ? $this->std->dadosestab->infotrab->infoapr->nrprocjud
                         : null,
                     false
@@ -203,13 +203,13 @@  discard block
 block discarded – undo
                 $this->dom->addChild(
                     $infoApr,
                     "contEntEd",
-                    ! empty($this->std->dadosestab->infotrab->infoapr->contented)
+                    !empty($this->std->dadosestab->infotrab->infoapr->contented)
                         ? $this->std->dadosestab->infotrab->infoapr->contented
                         : null,
                     false
                 );
 
-                if (! empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
+                if (!empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
                     foreach ($this->std->dadosestab->infotrab->infoapr->infoenteduc as $edu) {
                         $infoEntEduc = $this->dom->createElement("infoEntEduc");
                         $this->dom->addChild(
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
 
                 $infoTrab->appendChild($infoApr);
             }
-            if (! empty($this->std->dadosestab->infotrab->infopdc)) {
+            if (!empty($this->std->dadosestab->infotrab->infopdc)) {
                 $infoPCD = $this->dom->createElement("infoPCD");
                 $this->dom->addChild(
                     $infoPCD,
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
                 $this->dom->addChild(
                     $infoPCD,
                     "nrProcJud",
-                    ! empty($this->std->dadosestab->infotrab->infopdc->nrprocjud)
+                    !empty($this->std->dadosestab->infotrab->infopdc->nrprocjud)
                         ? $this->std->dadosestab->infotrab->infopdc->nrprocjud
                         : null,
                     false
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
             $node->appendChild($dadosEstab);
         }
 
-        if (! empty($this->std->novavalidade) && $this->std->modo == 'ALT') {
+        if (!empty($this->std->novavalidade) && $this->std->modo == 'ALT') {
             $newVal = $this->std->novavalidade;
             $novaValidade = $this->dom->createElement("novaValidade");
             $this->dom->addChild(
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
             $this->dom->addChild(
                 $novaValidade,
                 "fimValid",
-                ! empty($newVal->fimvalid) ? $newVal->fimvalid : null,
+                !empty($newVal->fimvalid) ? $newVal->fimvalid : null,
                 false
             );
             $node->appendChild($novaValidade);
@@ -421,7 +421,7 @@  discard block
 block discarded – undo
                 );
                 $dadosEstab->appendChild($infoCaepf);
             }
-            if (! empty($this->std->dadosestab->infoobra)) {
+            if (!empty($this->std->dadosestab->infoobra)) {
                 $infoObra = $this->dom->createElement("infoObra");
                 $this->dom->addChild(
                     $infoObra,
@@ -438,12 +438,12 @@  discard block
 block discarded – undo
                     $this->dom->addChild(
                         $infoApr,
                         "nrProcJud",
-                        ! empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
+                        !empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
                             ? $this->std->dadosestab->infotrab->infoapr->nrprocjud
                             : null,
                         false
                     );
-                    if (! empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
+                    if (!empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
                         foreach ($this->std->dadosestab->infotrab->infoapr->infoenteduc as $edu) {
                             $infoEntEduc = $this->dom->createElement("infoEntEduc");
                             $this->dom->addChild(

Please login to merge, or discard this patch.

		@@ -59,7 +59,7 @@ discard block
		block discarded – undo
59	59	$this->dom->addChild(
60	60	$ideEstab,
61	61	"fimValid",
62		- ! empty($this->std->fimvalid) ? $this->std->fimvalid : null,
	62	+ !empty($this->std->fimvalid) ? $this->std->fimvalid : null,
63	63	false
64	64	);
65	65
		@@ -72,7 +72,7 @@ discard block
		block discarded – undo
72	72	}
73	73	$node->appendChild($ideEstab);
74	74
75		- if (! empty($this->std->dadosestab)) {
	75	+ if (!empty($this->std->dadosestab)) {
76	76	$dadosEstab = $this->dom->createElement("dadosEstab");
77	77	$this->dom->addChild(
78	78	$dadosEstab,
		@@ -87,7 +87,7 @@ discard block
		block discarded – undo
87	87	$this->std->dadosestab->aliqgilrat->aliqrat,
88	88	true
89	89	);
90		- $fap = ! empty($this->std->dadosestab->aliqgilrat->fap) ? $this->std->dadosestab->aliqgilrat->fap : null;
	90	+ $fap = !empty($this->std->dadosestab->aliqgilrat->fap) ? $this->std->dadosestab->aliqgilrat->fap : null;
91	91	if ($fap) {
92	92	$fap = number_format($fap, 4, '.', '');
93	93	}
		@@ -97,7 +97,7 @@ discard block
		block discarded – undo
97	97	$fap,
98	98	false
99	99	);
100		- $aliqrata = ! empty($this->std->dadosestab->aliqgilrat->aliqratajust)
	100	+ $aliqrata = !empty($this->std->dadosestab->aliqgilrat->aliqratajust)
101	101	? $this->std->dadosestab->aliqgilrat->aliqratajust
102	102	: null;
103	103	if ($aliqrata) {
		@@ -109,7 +109,7 @@ discard block
		block discarded – undo
109	109	$aliqrata,
110	110	false
111	111	);
112		- if (! empty($this->std->dadosestab->aliqgilrat->procadmjudrat)) {
	112	+ if (!empty($this->std->dadosestab->aliqgilrat->procadmjudrat)) {
113	113	$procAdmJudRat = $this->dom->createElement("procAdmJudRat");
114	114	$this->dom->addChild(
115	115	$procAdmJudRat,
		@@ -131,7 +131,7 @@ discard block
		block discarded – undo
131	131	);
132	132	$aliqGilrat->appendChild($procAdmJudRat);
133	133	}
134		- if (! empty($this->std->dadosestab->aliqgilrat->procadmjudfap)) {
	134	+ if (!empty($this->std->dadosestab->aliqgilrat->procadmjudfap)) {
135	135	$procAdmJudFap = $this->dom->createElement("procAdmJudFap");
136	136	$this->dom->addChild(
137	137	$procAdmJudFap,
		@@ -155,7 +155,7 @@ discard block
		block discarded – undo
155	155	}
156	156	$dadosEstab->appendChild($aliqGilrat);
157	157
158		- if (! empty($this->std->dadosestab->infocaepf)) {
	158	+ if (!empty($this->std->dadosestab->infocaepf)) {
159	159	$infoCaepf = $this->dom->createElement("infoCaepf");
160	160	$this->dom->addChild(
161	161	$infoCaepf,
		@@ -166,7 +166,7 @@ discard block
		block discarded – undo
166	166	$dadosEstab->appendChild($infoCaepf);
167	167	}
168	168
169		- if (! empty($this->std->dadosestab->infoobra)) {
	169	+ if (!empty($this->std->dadosestab->infoobra)) {
170	170	$infoObra = $this->dom->createElement("infoObra");
171	171	$this->dom->addChild(
172	172	$infoObra,
		@@ -184,7 +184,7 @@ discard block
		block discarded – undo
184	184	$this->std->dadosestab->infotrab->regpt,
185	185	true
186	186	);
187		- if (! empty($this->std->dadosestab->infoapr)) {
	187	+ if (!empty($this->std->dadosestab->infoapr)) {
188	188	$infoApr = $this->dom->createElement("infoApr");
189	189	$this->dom->addChild(
190	190	$infoApr,
		@@ -195,7 +195,7 @@ discard block
		block discarded – undo
195	195	$this->dom->addChild(
196	196	$infoApr,
197	197	"nrProcJud",
198		- ! empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
	198	+ !empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
199	199	? $this->std->dadosestab->infotrab->infoapr->nrprocjud
200	200	: null,
201	201	false
		@@ -203,13 +203,13 @@ discard block
		block discarded – undo
203	203	$this->dom->addChild(
204	204	$infoApr,
205	205	"contEntEd",
206		- ! empty($this->std->dadosestab->infotrab->infoapr->contented)
	206	+ !empty($this->std->dadosestab->infotrab->infoapr->contented)
207	207	? $this->std->dadosestab->infotrab->infoapr->contented
208	208	: null,
209	209	false
210	210	);
211	211
212		- if (! empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
	212	+ if (!empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
213	213	foreach ($this->std->dadosestab->infotrab->infoapr->infoenteduc as $edu) {
214	214	$infoEntEduc = $this->dom->createElement("infoEntEduc");
215	215	$this->dom->addChild(
		@@ -224,7 +224,7 @@ discard block
		block discarded – undo
224	224
225	225	$infoTrab->appendChild($infoApr);
226	226	}
227		- if (! empty($this->std->dadosestab->infotrab->infopdc)) {
	227	+ if (!empty($this->std->dadosestab->infotrab->infopdc)) {
228	228	$infoPCD = $this->dom->createElement("infoPCD");
229	229	$this->dom->addChild(
230	230	$infoPCD,
		@@ -235,7 +235,7 @@ discard block
		block discarded – undo
235	235	$this->dom->addChild(
236	236	$infoPCD,
237	237	"nrProcJud",
238		- ! empty($this->std->dadosestab->infotrab->infopdc->nrprocjud)
	238	+ !empty($this->std->dadosestab->infotrab->infopdc->nrprocjud)
239	239	? $this->std->dadosestab->infotrab->infopdc->nrprocjud
240	240	: null,
241	241	false
		@@ -246,7 +246,7 @@ discard block
		block discarded – undo
246	246	$node->appendChild($dadosEstab);
247	247	}
248	248
249		- if (! empty($this->std->novavalidade) && $this->std->modo == 'ALT') {
	249	+ if (!empty($this->std->novavalidade) && $this->std->modo == 'ALT') {
250	250	$newVal = $this->std->novavalidade;
251	251	$novaValidade = $this->dom->createElement("novaValidade");
252	252	$this->dom->addChild(
		@@ -258,7 +258,7 @@ discard block
		block discarded – undo
258	258	$this->dom->addChild(
259	259	$novaValidade,
260	260	"fimValid",
261		- ! empty($newVal->fimvalid) ? $newVal->fimvalid : null,
	261	+ !empty($newVal->fimvalid) ? $newVal->fimvalid : null,
262	262	false
263	263	);
264	264	$node->appendChild($novaValidade);
		@@ -421,7 +421,7 @@ discard block
		block discarded – undo
421	421	);
422	422	$dadosEstab->appendChild($infoCaepf);
423	423	}
424		- if (! empty($this->std->dadosestab->infoobra)) {
	424	+ if (!empty($this->std->dadosestab->infoobra)) {
425	425	$infoObra = $this->dom->createElement("infoObra");
426	426	$this->dom->addChild(
427	427	$infoObra,
		@@ -438,12 +438,12 @@ discard block
		block discarded – undo
438	438	$this->dom->addChild(
439	439	$infoApr,
440	440	"nrProcJud",
441		- ! empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
	441	+ !empty($this->std->dadosestab->infotrab->infoapr->nrprocjud)
442	442	? $this->std->dadosestab->infotrab->infoapr->nrprocjud
443	443	: null,
444	444	false
445	445	);
446		- if (! empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
	446	+ if (!empty($this->std->dadosestab->infotrab->infoapr->infoenteduc)) {
447	447	foreach ($this->std->dadosestab->infotrab->infoapr->infoenteduc as $edu) {
448	448	$infoEntEduc = $this->dom->createElement("infoEntEduc");
449	449	$this->dom->addChild(

nfephp-org / sped-esocial

Push — master ( ce467a...63d89e )

Status

Category

Spacing +19 added lines, -19 removed lines patch added patch discarded remove patch