Backoffice/Reference/Strategies/AbstractContentReferenceStrategy.php
Function extractContentsFromElement
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
Open
protected function extractContentsFromElement($element, array $references = array())
{
if ($this->isContentSearchAttribute($element)) {
$references[] = $element['contentId'];
- Read upRead up
- Create a ticketCreate a ticket
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
The method extractContentsFromElement() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10. Open
Open
protected function extractContentsFromElement($element, array $references = array())
{
if ($this->isContentSearchAttribute($element)) {
$references[] = $element['contentId'];
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
CyclomaticComplexity
Since: 0.1
Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.
Example
// Cyclomatic Complexity = 11
class Foo {
1 public function example() {
2 if ($a == $b) {
3 if ($a1 == $b1) {
fiddle();
4 } elseif ($a2 == $b2) {
fiddle();
} else {
fiddle();
}
5 } elseif ($c == $d) {
6 while ($c == $d) {
fiddle();
}
7 } elseif ($e == $f) {
8 for ($n = 0; $n < $h; $n++) {
fiddle();
}
} else {
switch ($z) {
9 case 1:
fiddle();
break;
10 case 2:
fiddle();
break;
11 case 3:
fiddle();
break;
default:
fiddle();
break;
}
}
}
}
Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity
Checking instanceof against undeclared class \OpenOrchestra\ModelInterface\Model\ContentAttributeInterface
Open
Open
$element instanceof ContentAttributeInterface
- Create a ticketCreate a ticket
- Exclude checks
Parameter $bbcodeParser
has undeclared type \OpenOrchestra\BBcodeBundle\Parser\BBcodeParserInterface
Open
Open
public function __construct(
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant TAG_NAME
from undeclared class \OpenOrchestra\DisplayBundle\BBcode\InternalLinkDefinition
Open
Open
$nodeTags = $parsedBBcode->getElementByTagName(InternalLinkDefinition::TAG_NAME);
- Create a ticketCreate a ticket
- Exclude checks
Parameter $contentRepository
has undeclared type \OpenOrchestra\ModelInterface\Repository\ContentRepositoryInterface
Open
Open
public function __construct(
- Create a ticketCreate a ticket
- Exclude checks
Call to method getValue
from undeclared class \OpenOrchestra\ModelInterface\Model\ContentAttributeInterface
Open
Open
$references = array_merge($references, $this->extractContentIdFromBBCode($element->getValue()));
- Create a ticketCreate a ticket
- Exclude checks
Call to method getType
from undeclared class \OpenOrchestra\ModelInterface\Model\ContentAttributeInterface
Open
Open
&& $element->getType() === 'tinymce'
- Create a ticketCreate a ticket
- Exclude checks
Checking instanceof against undeclared class \OpenOrchestra\ModelInterface\Model\ContentAttributeInterface
Open
Open
return $attributeValue instanceof ContentAttributeInterface
- Create a ticketCreate a ticket
- Exclude checks
Call to method parse
from undeclared class \OpenOrchestra\BBcodeBundle\Parser\BBcodeParserInterface
Open
Open
$parsedBBcode = $this->bbcodeParser->parse($str);
- Create a ticketCreate a ticket
- Exclude checks
Call to method getType
from undeclared class \OpenOrchestra\ModelInterface\Model\ContentAttributeInterface
Open
Open
&& $attributeValue->getType() === 'embedded_content';
- Create a ticketCreate a ticket
- Exclude checks
Call to method getValue
from undeclared class \OpenOrchestra\ModelInterface\Model\ContentAttributeInterface
Open
Open
&& $this->hasInternalLinkBBCode($element->getValue())
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant TAG_NAME
from undeclared class \OpenOrchestra\DisplayBundle\BBcode\InternalLinkDefinition
Open
Open
$internalLinkBBCode = '/\[' . InternalLinkDefinition::TAG_NAME . '[^]]*].*?\[\/'. InternalLinkDefinition::TAG_NAME . '\]/m';
- Create a ticketCreate a ticket
- Exclude checks
Line exceeds 120 characters; contains 132 characters Open
Open
$internalLinkBBCode = '/\[' . InternalLinkDefinition::TAG_NAME . '[^]]*].*?\[\/'. InternalLinkDefinition::TAG_NAME . '\]/m';
- Create a ticketCreate a ticket
- Exclude checks
Expected 0 spaces after opening bracket; newline found Open
Open
} elseif (
- Create a ticketCreate a ticket
- Exclude checks
Expected 0 spaces after opening bracket; newline found Open
Open
} elseif (
- Create a ticketCreate a ticket
- Exclude checks