Non-Static Podio class

[daniel-sc]

This discussion should help sort/organize/resolve issues with the proposed change from the static Podio class to the PodioClient instance.

PR: #228
Issue: #90

@rafaelmb @carlfredrikhero @bajarang-agarwal @timosillander @dylannadon @daviferreira @mgithens hope this discussion is the right format, please let me know your inputs

[daniel-sc]

TODOs

• Migration guide
• Update docs (https://podio-community.github.io/)

[daniel-sc]

Serialize/Unserialize model classes

During migration, I noticed that after the change the model classes (e.g. PodioItem, PodioCollection, ..) are not serializable any more. This is caused by the fact all models have a reference to the PodioClient, which contains a \GuzzleHttp\Client (and its handler stack). Potential further issues might arise with \PodioClient::$last_http_response which is a \GuzzleHttp\Psr7\Response (not tested yet).

I'm not sure how much of an issue this is, but I think it should at least be somehow possible to serialize (and unserialize) all models.

Potential solutions that come to my mind (please add further ideas!):

1. Make the reference to PodioClient in the models nullable and exclude the PodioClient from serialization. Then provide a PodioObject::setPodioClient(PodioClient) to allow manually (re-)setting the client after deserialization.
2. Make guzzle client serializable - possible?
3. Drop the "active record" pattern: use instances for data only and use static methods for all operations.

Any other ideas?

[daniel-sc]

@rafaelmb @carlfredrikhero @bajarang-agarwal @timosillander @dylannadon @daviferreira @mgithens any inputs for this?

[daniel-sc]

PSR namespaces

Should we namespace all classes/files?

It seems almost all modern libs are using PSR-4 namespaces. As we have a major breaking change with v7 anyway, we could add namespaces as well? Or is this just an unnecessary complication (for users migrating to v7)?

[daniel-sc]

@rafaelmb @carlfredrikhero @bajarang-agarwal @timosillander @dylannadon @daviferreira @mgithens any inputs here?