You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently the proxy classes are used as a way to access protected methods in the SchemaStates and provide control over the way these are called. However, the way it is built right now does not give access to the originally used instance, but to the proxy deriving a new instance. That is fine for what it is used right now, but useless for any other case, and might even be problematic in the future.
Instead, we should consider removing the abstract proxies and have the proxy classes not extend the SchemaState, but just work with the SchemaState instance that is passed in through the constructor. In order to call the protected methods, it would be needed to use reflection.
This also means that we have to override or replace and method inside the SchemaStates that does method calls on itself, like we already did with the load() method, for instance.
The text was updated successfully, but these errors were encountered:
Currently the proxy classes are used as a way to access protected methods in the SchemaStates and provide control over the way these are called. However, the way it is built right now does not give access to the originally used instance, but to the proxy deriving a new instance. That is fine for what it is used right now, but useless for any other case, and might even be problematic in the future.
Instead, we should consider removing the abstract proxies and have the proxy classes not extend the SchemaState, but just work with the SchemaState instance that is passed in through the constructor. In order to call the protected methods, it would be needed to use reflection.
This also means that we have to override or replace and method inside the SchemaStates that does method calls on itself, like we already did with the
load()
method, for instance.The text was updated successfully, but these errors were encountered: