Hhxsv5\LaravelS\Swoole\Proxy\DynamicPropertiesTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__get()	0	4	1
A	__unset()	0	6	2
A	__set()	0	15	3
A	__isset()	0	8	2

<?php

Missing file doc comment

namespace Hhxsv5\LaravelS\Swoole\Proxy;

use Hhxsv5\LaravelS\Swoole\Server;

/**
 * DynamicPropertiesTrait solves the issue that PHP 8.2+ does not allow dynamic property creation.
 * Provides functionality for storing and accessing dynamic properties.
 */

Missing @category tag in class comment

Missing @package tag in class comment

Missing @author tag in class comment

Missing @license tag in class comment

Missing @link tag in class comment

trait DynamicPropertiesTrait
{
    /** @var array Stores dynamically created properties (e.g., tables) */

The open comment tag must be the only content on the line

Missing short description in doc comment

The close comment tag must be the only content on the line

    protected array $dynamicProperties = [];

    /**

Parameter $name should have a doc-comment as per coding-style.

     * Dynamic property access
     */

Missing @return tag in function comment

    public function __get($name)
    {
        // First check dynamic properties
        return $this->dynamicProperties[$name] ?? null;
    }

    /**

Parameter $name should have a doc-comment as per coding-style.

Parameter $value should have a doc-comment as per coding-style.

     * Dynamic property setter
     */

Missing @return tag in function comment

    public function __set($name, $value)
    {
        // If property name ends with 'Table', store it in dynamic properties
        if (str_ends_with($name, Server::TABLE_PROPERTY_SUFFIX)) {
            $this->dynamicProperties[$name] = $value;
            return;
        }

        // For other properties, try to set directly on parent class
        // (Swoole's C extension classes don't have __set, so it won't recurse)
        try {
            $this->{$name} = $value;
        } catch (\Error $e) {

catch (\Error $e) is not reachable.

            // When PHP 8.2+ doesn't allow dynamic property creation, store in dynamic properties
            $this->dynamicProperties[$name] = $value;
        }
    }

    /**

Parameter $name should have a doc-comment as per coding-style.

     * Check if property exists
     */

Missing @return tag in function comment

    public function __isset($name)
    {
        if (array_key_exists($name, $this->dynamicProperties)) {
            return true;
        }

        // Check parent class properties (Swoole's C extension class properties)
        return property_exists($this, $name);
    }

    /**

Parameter $name should have a doc-comment as per coding-style.

     * Unset property
     */

Missing @return tag in function comment

    public function __unset($name)
    {
        unset($this->dynamicProperties[$name]);

        if (property_exists($this, $name)) {
            unset($this->{$name});
        }
    }
}