Maslosoft\Mangan\Helpers\UnknownDocumentTypePanicker::tryHandle()

Complexity

Conditions	8
Paths	20

Size

Total Lines

40

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	72

Importance

Changes

0

<?php
	public static function tryHandle(&$data, $parent, $parentField)
	{
		$className = '';
		$handled = false;
		if(!empty($parent))
		{
			$event = new UnknownDocumentType();
			$event->setData($data);
			$event->parent = $parent;
			$event->field = $parentField;

			$handled = Event::handled($parent, UnknownDocumentType::EventName, $event);
			if($handled)
			{
				$data = $event->getData();
				if(empty($data['_class']))
				{
					$handled = false;
				}
				else
				{
					$className = $data['_class'];
				}
			}
		}
		if(!$handled)

The expression $handled of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

		{
			$params = [];
			if(!empty($parent) && is_object($parent))
			{
				$params[] = sprintf('on model `%s`', get_class($parent));
			}
			if(!empty($parentField))
			{
				$params[] = sprintf('on field `%s`', $parentField);
			}
			throw new TransformatorException('Could not determine document type ' . implode(', ', $params));
		}
		return $className;
	}
}