Partly initialization #380

AndreRudolph · 2017-07-21T08:14:06Z

Amazing and interesing project. I want to integrate your ProxyManager implementation into my project. However, I have a question about initialization of ValueHolderProxies. Is there any way to do a partly initialization depending on special use cases?

My use case:
I have a tree of objects.

    <?php
    
    class Object {
        private $parent;
        private $childs;

        public function getParent() {
            return $this->parent;
        }

        public function getChilds() {
            return $this->childs;
        }
   }`

An object within the tree can be obtained by creating a virtual proxy. Each object has parent / child relationships to other objects and some other properties which should be initialized immediately. At the moment when an object will be obtained, a proxy for the object itself and proxies for the directly parent and child objects are created. So my question is: Is there any way to create these parent / child proxies if they are explicitly needed (e.g. if getParent is called) ?

Best regards

The text was updated successfully, but these errors were encountered:

Ocramius · 2017-07-21T08:21:28Z

Partial loading is only possible with a ghost object, although I didn't implement it fully yet, and I'm unsure about the proper interface for now.

You probably want exactly what @lstrojny wants in #377

There is one way to achieve that, although it is very hacky and not formalised. As you can see in the initializer signature, a method and a parameter list is available. If you check for usages of __get, __set or __isset and only load the properties you want, then unset the properties that shouldn't have been initialized, that could work, although I didn't try it myself.

I will at some point have to implement partial loading, as I really want the feature for Doctrine ORM too.

AndreRudolph · 2017-07-21T08:34:16Z

Thank you for your fast reply. Apparently you have planned to implement partial loading in future, so I can wait with this performance issue because it has not a high priority. The important thing is that it works and it works like a charm.

Maybe it will reduces complexity in future if you could provide a way to use objects as initializer. Closures could be used for less complex and classes for more complex initialization logic.

Thank you very much.

Ocramius · 2017-07-21T08:38:24Z

classes for more complex initialization logic.

That's the plan for a later version: since PHP anonymous classes are now available, I want to interface the initializer.

Ref: #334

AndreRudolph · 2017-07-21T08:40:12Z

That's a good idea. Then I will use your library without worries. 👍

Ocramius · 2020-12-20T16:57:32Z

Note: the initializers are now interfaced correctly, although through psalm annotations.

ProxyManager/src/ProxyManager/Factory/LazyLoadingGhostFactory.php

Lines 78 to 84 in f65ae0f

    
                * @psalm-param Closure( 
        
                *   RealObjectType&GhostObjectInterface<RealObjectType>=, 
        
                *   string=, 
        
                *   array<string, mixed>=, 
        
                *   ?Closure=, 
        
                *   array<string, mixed>= 
        
                * ) : bool $initializer

Ocramius added enhancement question labels Jul 21, 2017

Ocramius closed this Dec 20, 2020

Dec	JAN	Feb
	21
2020	2021	2022

Ocramius / ProxyManager

Partly initialization #380

Partly initialization #380

AndreRudolph commented Jul 21, 2017 •

edited

Ocramius commented Jul 21, 2017

AndreRudolph commented Jul 21, 2017 •

edited

Ocramius commented Jul 21, 2017 •

edited

AndreRudolph commented Jul 21, 2017

Ocramius commented Dec 20, 2020

Ocramius / ProxyManager

Sponsor Ocramius/ProxyManager

Partly initialization #380

Partly initialization #380

Comments

AndreRudolph commented Jul 21, 2017 • edited

Ocramius commented Jul 21, 2017

AndreRudolph commented Jul 21, 2017 • edited

Ocramius commented Jul 21, 2017 • edited

AndreRudolph commented Jul 21, 2017

Ocramius commented Dec 20, 2020

AndreRudolph commented Jul 21, 2017 •

edited

AndreRudolph commented Jul 21, 2017 •

edited

Ocramius commented Jul 21, 2017 •

edited