call getter for property if getter exists. about arrays HOT 5 CLOSED

One use case is objects validation via specifying path to the property i.e. owner.name to try calling $object->getOwner()->getName().

from arrays.

It's kind of black magic. Yii 3 usually avoids it.
IMHO ArrayHelper must not deal with object methods. It's not just an ability for user to shoot off his leg, it's a provocation.
In case of validation it's ok either to implement DataSet in the validatable object directly, or create a wrapper with this interface. But it is not ok to magically call object methods without a guarantee these methods don't change state or call some services like these:

class Test {
    private $prop = 1;

    public function getProp()
    {
        return $this->prop++;
    }
}

class Test2 {
    private $prop = null;

    public function getProp()
    {
        if ($this->prop === null) {
            $infrastructure = $this->httpService->deployInfrastructureFor($this);
            $this->prop = $this->infraRepository->save($infrastructure);
        }

        return $this->prop;
    }
}

from arrays.

It's kind of black magic. Yii 3 usually avoids it.
IMHO ArrayHelper must not deal with object methods. It's not just an ability for user to shoot off his leg, it's a provocation.
In case of validation it's ok either to implement DataSet in the validatable object directly, or create a wrapper with this interface. But it is not ok to magically call object methods without a guarantee these methods don't change state or call some services like these:

class Test {
    private $prop = 1;

    public function getProp()
    {
        return $this->prop++;
    }
}

class Test2 {
    private $prop = null;

    public function getProp()
    {
        if ($this->prop === null) {
            $infrastructure = $this->httpService->deployInfrastructureFor($this);
            $this->prop = $this->infraRepository->save($infrastructure);
        }

        return $this->prop;
    }
}

1. Yes i agree with you, and i suggest remove from ArrayHelper ability to deal with objects and make separated package for this.
2. Remove methods html* from ArrayHelper, because this methods dont deal with arrays. Maybe make HtmlHelper

from arrays.

I'm agree with you about html* methods, it's more about yiisoft/strings. But let's keep one issue for 1 theme :)

from arrays.

I'm agree with you about html* methods, it's more about yiisoft/strings. But let's keep one issue for 1 theme :)

Yes, you right!

from arrays.