FieldParserFixedText::getText() - Code Metrics - Inspection of "phpcs fixes" - graze/ciff-renderer - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — refactor-04-parser-tests ( dc4950...bd4663 )

by John

created 2016-12-02 12:30 UTC

FieldParserFixedText::getText() A

↳ Parent: FieldParserFixedText

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes

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

<?php

namespace Graze\CiffRenderer\Parser\FieldParser;

use Graze\CiffRenderer\Parser\FieldParser\FieldParserInterface;

class FieldParserFixedText extends AbstractFieldParser implements FieldParserInterface
{
    /**
     * @return float
     */
    public function getFontSize()
    {
        return (float) $this->xmlField->Text->Font->Pitch;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
    }

    /**
     * @return string
     */
    public function getFontFace()
    {
        return (string) $this->xmlField->Text->Font->Face;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
    }

    /**
     * @return int
     */
    public function getOrientation()
    {
        return (int) $this->xmlField->Orientation;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
    }

    /**
     * @return string
     */
    public function getText()
    {
        $text = '';
        foreach ($this->xmlField->Data->Object as $object) {
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            if ($object->SrcField) { // @codingStandardsIgnoreLine
                // Field is a merge field, fetch text from source field
                $sourceFieldName = (string) $object->SrcField->attributes()->SrcFieldName; // @codingStandardsIgnoreLine
                $sourceField = $this->fieldParserRegistry->getParser($sourceFieldName);
                $fieldText = $sourceField->getText();
            } else {
                $fieldText = (string) $object->Default; // @codingStandardsIgnoreLine
            }

            $text .= $fieldText;
        }

        return $text;
    }
}


Push — refactor-04-parser-tests ( dc4950...bd4663 )

FieldParserFixedText::getText() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1		<?php
2
3		namespace Graze\CiffRenderer\Parser\FieldParser;
4
5		use Graze\CiffRenderer\Parser\FieldParser\FieldParserInterface;
6
7		class FieldParserFixedText extends AbstractFieldParser implements FieldParserInterface
8		{
9		/**
10		* @return float
11		*/
12	1	public function getFontSize()
13		{
14	1	return (float) $this->xmlField->Text->Font->Pitch;
		0 ignored issues – show Bug introduced 2016-12-02 12:36 UTC by Report Bug Copy Issue Report Accessing `Text` on the interface `Graze\CiffRenderer\Parse...mpleXmlElementInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
15		}
16
17		/**
18		* @return string
19		*/
20	2	public function getFontFace()
21		{
22	2	return (string) $this->xmlField->Text->Font->Face;
		0 ignored issues – show Bug introduced 2016-12-02 12:36 UTC by Report Bug Copy Issue Report Accessing `Text` on the interface `Graze\CiffRenderer\Parse...mpleXmlElementInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
23		}
24
25		/**
26		* @return int
27		*/
28	3	public function getOrientation()
29		{
30	3	return (int) $this->xmlField->Orientation;
		0 ignored issues – show Bug introduced 2016-12-02 12:36 UTC by Report Bug Copy Issue Report Accessing `Orientation` on the interface `Graze\CiffRenderer\Parse...mpleXmlElementInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
31		}
32
33		/**
34		* @return string
35		*/
36	2	public function getText()
37		{
38	2	$text = '';
39	2	foreach ($this->xmlField->Data->Object as $object) {
		0 ignored issues – show Bug introduced 2016-12-02 12:36 UTC by Report Bug Copy Issue Report Accessing `Data` on the interface `Graze\CiffRenderer\Parse...mpleXmlElementInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
40	2	if ($object->SrcField) { // @codingStandardsIgnoreLine
41		// Field is a merge field, fetch text from source field
42	2	$sourceFieldName = (string) $object->SrcField->attributes()->SrcFieldName; // @codingStandardsIgnoreLine
43	2	$sourceField = $this->fieldParserRegistry->getParser($sourceFieldName);
44	2	$fieldText = $sourceField->getText();
45		} else {
46	2	$fieldText = (string) $object->Default; // @codingStandardsIgnoreLine
47		}
48
49	2	$text .= $fieldText;
50		}
51
52	2	return $text;
53		}
54		}
55

graze / ciff-renderer

Push — refactor-04-parser-tests ( dc4950...bd4663 )

FieldParserFixedText::getText() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like