pradosoft / prado

framework/Collections/TCollectionSubscription.php

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
 	 * @param ?bool $autoSubscribe Default null for autoSubscribing when there is a
 	 *   key or item.
 	 */
-	public function __construct(?ArrayAccess $collection = null, mixed $key = null, mixed $item = null, null|int|float $priority = null, null|bool|int $isAssociative = 1, ?bool $autoSubscribe = null)
+	public function __construct(?ArrayAccess $collection = null, mixed $key = null, mixed $item = null, null | int | float $priority = null, null | bool | int $isAssociative = 1, ?bool $autoSubscribe = null)
 	{
 		parent::__construct($collection, $key, $item, $priority, $isAssociative, $autoSubscribe);
 	}
@@ -50,7 +50,7 @@ 
 	 * The ArrayAccess collection for getting the array without pass by reference.
 	 * @param bool $weak Return the collection in as a WeakReference.
 	 */
-	public function getCollection(bool $weak = false): null|ArrayAccess|WeakReference
+	public function getCollection(bool $weak = false): null | ArrayAccess | WeakReference
 	{
 		return $this->getArray($weak);
 	}

framework/Collections/TWeakList.php

Spacing +7 added lines, -7 removed lines

@@ -121,7 +121,7 @@ 
 	 */
 	protected function weakCustomAdd(object $object)
 	{
-		if($object instanceof TEventHandler) {
+		if ($object instanceof TEventHandler) {
 			$object = $object->getHandlerObject();
 			$this->_eventHandlerCount++;
 		}
@@ -137,7 +137,7 @@ 
 	 */
 	protected function weakCustomRemove(object $object)
 	{
-		if($object instanceof TEventHandler) {
+		if ($object instanceof TEventHandler) {
 			$object = $object->getHandlerObject();
 			$this->_eventHandlerCount--;
 		}
@@ -161,7 +161,7 @@ 
 				$item[$key] = $element;
 			}
 		} elseif (is_object($item)) {
-			if($item instanceof WeakReference) {
+			if ($item instanceof WeakReference) {
 				$item = $item->get();
 			} elseif (($item instanceof TEventHandler) && !$item->hasHandler()) {
 				$item = null;
@@ -207,7 +207,7 @@ 
 				if ($isEventHandler = ($object instanceof TEventHandler)) {
 					$object = $object->getHandlerObject(true);
 				}
-				if(($object instanceof WeakReference) && $object->get() === null) {
+				if (($object instanceof WeakReference) && $object->get() === null) {
 					$this->_c--;
 					if ($i === $this->_c) {
 						array_pop($this->_d);
@@ -247,7 +247,7 @@ 
 	 */
 	public function setDiscardInvalid($value): void
 	{
-		if($value === $this->_discardInvalid) {
+		if ($value === $this->_discardInvalid) {
 			return;
 		}
 		if ($this->_discardInvalid !== null && !Prado::isCallingSelf()) {
@@ -266,7 +266,7 @@ 
 						$object = $object->get();
 					}
 					if ($object === null) {
-						$this->_c--;	//on read only, parent::removeAt won't remove for scrub.
+						$this->_c--; //on read only, parent::removeAt won't remove for scrub.
 						if ($i === $this->_c) {
 							array_pop($this->_d);
 						} else {
@@ -445,7 +445,7 @@ 
 		$this->filterItemForInput($item);
 		if (($index = parent::indexOf($item)) === -1 && $this->_eventHandlerCount) {
 			$index = false;
-			foreach($this->_d as $index => $dItem) {
+			foreach ($this->_d as $index => $dItem) {
 				if (($dItem instanceof TEventHandler) && $dItem->isSameHandler($item, true)) {
 					break;
 				}

framework/Collections/TArraySubscription.php

Spacing +8 added lines, -8 removed lines

@@ -78,7 +78,7 @@ 
 	/**
 	 * @var null|int|string The key that the item is added to the array.
 	 */
-	private null|int|string $_key = null;
+	private null | int | string $_key = null;
 
 	/**
 	 * @var mixed The item subscribing to the array.
@@ -95,7 +95,7 @@ 
 	 * @var null|bool|int Is the array an associative array.  False for a "list" style array.
 	 *   null for discovery of the style of array on subscribe.  default true.
 	 */
-	protected null|bool|int $_isAssoc = true;
+	protected null | bool | int $_isAssoc = true;
 
 	/**
 	 * @var bool Is the item inserted into the collection.
@@ -135,7 +135,7 @@ 
 	 *   the item.  Default 1 for true except for TList.
 	 * @param bool $autoSubscribe Should the
 	 */
-	public function __construct(mixed &$array = null, mixed $key = null, mixed $item = null, null|int|float $priority = null, null|bool|int $isAssociative = 1, ?bool $autoSubscribe = null)
+	public function __construct(mixed &$array = null, mixed $key = null, mixed $item = null, null | int | float $priority = null, null | bool | int $isAssociative = 1, ?bool $autoSubscribe = null)
 	{
 		$this->setArray($array);
 		$this->setKey($key);
@@ -177,7 +177,7 @@ 
 	 * @param bool $weak
 	 * @return array|ArrayAccess The subscribed array-collection, passed by reference.
 	 */
-	public function &getArray(bool $weak = false): array|ArrayAccess|WeakReference|null
+	public function &getArray(bool $weak = false): array | ArrayAccess | WeakReference | null
 	{
 		if ($this->_array instanceof WeakReference) {
 			if ($weak) {
@@ -202,7 +202,7 @@ 
 	 * @throws TInvalidOperationException If the item is already subscribed.
 	 * @return static The current object.
 	 */
-	public function setArray(null|array|ArrayAccess &$value): static
+	public function setArray(null | array | ArrayAccess & $value): static
 	{
 		if ($this->_isSubscribed) {
 			throw new TInvalidOperationException('arraysubscription_no_change', 'Array');
@@ -223,7 +223,7 @@ 
 	 * with the TList (by {@see TList::indexOf()}) or array (by array_search).
 	 * @return null|int|string The key for the item subscription to the array.
 	 */
-	public function getKey(): null|int|string
+	public function getKey(): null | int | string
 	{
 		$collection = &$this->getArray();
 		if ($this->_isSubscribed && $this->_key === null) {
@@ -322,7 +322,7 @@ 
 	 *   as a list from (0, ..., count() - 1).  if null, where needed, the "list"ness
 	 *   of the array will be determined by {@see TArrayHelper::array_is_list()}.
 	 */
-	public function getIsAssociative(): null|bool|int
+	public function getIsAssociative(): null | bool | int
 	{
 		return $this->_isAssoc;
 	}
@@ -334,7 +334,7 @@ 
 	 * @throws TInvalidOperationException If the item is already subscribed.
 	 * @return static The current object.
 	 */
-	public function setIsAssociative(null|bool|int $value = null): static
+	public function setIsAssociative(null | bool | int $value = null): static
 	{
 		if ($this->_isSubscribed) {
 			throw new TInvalidOperationException('arraysubscription_no_change', 'Key');

framework/Collections/TWeakCallableCollection.php

Spacing +11 added lines, -11 removed lines

@@ -136,7 +136,7 @@ 
 	 */
 	protected function weakCustomAdd(object $object)
 	{
-		if($object instanceof TEventHandler) {
+		if ($object instanceof TEventHandler) {
 			$object = $object->getHandlerObject();
 			if (!$object) {
 				return;
@@ -155,7 +155,7 @@ 
 	 */
 	protected function weakCustomRemove(object $object)
 	{
-		if($object instanceof TEventHandler) {
+		if ($object instanceof TEventHandler) {
 			$object = $object->getHandlerObject();
 			if (!$object) {
 				return;
@@ -195,7 +195,7 @@ 
 				$handler = null;
 			}
 		} elseif (is_object($handler)) {
-			if($handler instanceof WeakReference) {
+			if ($handler instanceof WeakReference) {
 				$handler = $handler->get();
 			} elseif (($handler instanceof TEventHandler) && !$handler->hasHandler()) {
 				$handler = null;
@@ -294,7 +294,7 @@ 
 	 */
 	public function setDiscardInvalid($value): void
 	{
-		if($value === $this->_discardInvalid) {
+		if ($value === $this->_discardInvalid) {
 			return;
 		}
 		if ($this->_discardInvalid !== null && !Prado::isCallingSelf()) {
@@ -313,7 +313,7 @@ 
 							$obj = $obj->getHandlerObject(true);
 						}
 						if ($obj instanceof WeakReference) {
-							if($obj = $obj->get()) {
+							if ($obj = $obj->get()) {
 								$this->weakAdd($obj);
 							} else {
 								parent::removeAtIndexInPriority($i, $priority);
@@ -524,7 +524,7 @@ 
 			throw new TInvalidDataValueException('weakcallablecollection_callable_required');
 		}
 		$return = null;
-		foreach($items as $item) {
+		foreach ($items as $item) {
 			$itemPriority = null;
 			if (($isPriorityItem = ($item instanceof IPriorityItem)) && ($priority === null || !is_numeric($priority))) {
 				$itemPriority = $priority = $item->getPriority();
@@ -540,7 +540,7 @@ 
 			$result = parent::insertAtIndexInPriority($item, $index, $priority, $preserveCache);
 			if ($return === null) {
 				$return = $result;
-			} elseif(!is_array($return)) {
+			} elseif (!is_array($return)) {
 				$return = [$return, $result];
 			} else {
 				$return[] = $result;
@@ -713,7 +713,7 @@ 
 					continue;
 				} elseif ($p == $priority) {
 					$index = false;
-					foreach($this->_d[$p] as $index => $pItem) {
+					foreach ($this->_d[$p] as $index => $pItem) {
 						if ($item === $pItem || ($pItem instanceof TEventHandler) && $pItem->isSameHandler($item, true)) {
 							break;
 						}
@@ -731,7 +731,7 @@ 
 		$this->flattenPriorities();
 
 		if (($index = array_search($item, $this->_fd, true)) === false && $this->_eventHandlerCount) {
-			foreach($this->_fd as $index => $pItem) {
+			foreach ($this->_fd as $index => $pItem) {
 				if (($pItem instanceof TEventHandler) && $pItem->isSameHandler($item, true)) {
 					break;
 				}
@@ -783,8 +783,8 @@ 
 		$absindex = 0;
 		foreach (array_keys($this->_d) as $priority) {
 			$index = false;
-			foreach($this->_d[$priority] as $index => $pItem) {
-				if(($pItem instanceof TEventHandler) && $pItem->isSameHandler($item, true)) {
+			foreach ($this->_d[$priority] as $index => $pItem) {
+				if (($pItem instanceof TEventHandler) && $pItem->isSameHandler($item, true)) {
 					break;
 				}
 				$index = false;

framework/TEventSubscription.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 	 * @param ?bool $autoSubscribe
 	 * @param mixed $index
 	 */
-	public function __construct(?TComponent $component = null, mixed $event = null, mixed $handler = null, null|int|float $priority = null, ?bool $autoSubscribe = null, mixed $index = null)
+	public function __construct(?TComponent $component = null, mixed $event = null, mixed $handler = null, null | int | float $priority = null, ?bool $autoSubscribe = null, mixed $index = null)
 	{
 		if ($component) {
 			$this->setComponent($component);
@@ -65,7 +65,7 @@ 
 	 * @param bool $weak
 	 * @return null|array|ArrayAccess|WeakReference The Event Handler from the component.
 	 */
-	public function &getArray(bool $weak = false): array|ArrayAccess|WeakReference|null
+	public function &getArray(bool $weak = false): array | ArrayAccess | WeakReference | null
 	{
 		if ($handlers = parent::getArray($weak)) {
 			return $handlers;
@@ -100,7 +100,7 @@ 
 	 * @param null|array|ArrayAccess $value
 	 * @throws TInvalidOperationException This property cannot be set directly.
 	 */
-	public function setArray(null|array|ArrayAccess &$value): static
+	public function setArray(null | array | ArrayAccess & $value): static
 	{
 		if ($value !== null) {
 			throw new TInvalidOperationException('eventsubscription_no_setarray');

framework/Util/Helpers/TArrayHelper.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 	 * @return bool is the array a list.
 	 * @link https://www.php.net/manual/en/function.array-is-list.php
 	 */
-	public static function array_is_list(array|Traversable $array): bool
+	public static function array_is_list(array | Traversable $array): bool
 	{
 		if (function_exists('array_is_list') && !($array instanceof Traversable)) {
 			return array_is_list($array);