Prado\Util\Behaviors\TNoUnserializeBehaviorTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	dyWakeUp()	0	7	2

<?php

/**
 * TNoUnserializeBehaviorTrait class file.
 *
 * @author Brad Anderson <belisoful@icloud.com>
 * @link https://github.com/pradosoft/prado
 * @license https://github.com/pradosoft/prado/blob/master/LICENSE
 */

namespace Prado\Util\Behaviors;

use Prado\Util\TCallChain;

/**
 * TNoUnserializeBehaviorTrait class.
 *
 * When this trait is used by an IBehavior, upon the owner being unserialized (via
 * magic method __wakeup and dyWakeUp) this trait removes itself from its owner.
 *
 * This trait is used to deprecate serialized objects' IBehavior.  By re-serializing
 * the object can be saved without the deprecated behavior.
 *
 * @author Brad Anderson <belisoful@icloud.com>
 * @since 4.2.3
 */
trait TNoUnserializeBehaviorTrait
{
	/**
	 * This is raised when an owner is completed its unserialize() method call to
	 * __wakeup.  This method removes it behavior from the owner.
	 * @param TCallChain $chain The chain of dynamic event method handlers.
	 */
	public function dyWakeUp(TCallChain $chain)
	{
		$owner = $this->getOwner();

It seems like getOwner() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		if ($index = array_search($this, $owner->getBehaviors())) {
			$owner->detachBehavior($index);
		}
		return $chain->dyWakeUp();

Are you sure the usage of $chain->dyWakeUp() targeting Prado\TComponent::dyWakeUp() seems to always return null.

	}
}