[WIP] Pimple, Interfaces

[liuggio]

• [x] Gather feedback about what do to with \Pimple.
• [x] move all the examples and 'Simple*' classes into a 'example' folder.
• [x] Gather feedback about the objective of the ObjectLoaderFactory.
• [x] Improve CS.

I didn't change so much, I added an Abstract Repository, a simple Associator for class->worker, and few Interfaces

• IdentityMapSynchronizerInterface
• ObjectRepositoryInterface
• IdentityExtractorFactoryInterface
• ObjectLoaderFactoryInterface
• ObjectRepositoryFactoryInterface

let me know what do you think, thanks to the interfaces I created a stupid orm.

[Ocramius]

@liuggio I generally like the changes, but don't try to specialize the factories or make them smarter for now. The demos are supposed to work with only stdClass, and the factories should be completely replaced by a consuming library configuring a micro-DIC that provides all these components.

[coveralls]

Coverage decreased (-0%) when pulling 9d6c57827a4dfed9038825b2d33a941c8a8ab764 on liuggio:master into 782620bfbb93b222b92ad2dbb553bcdf7dd5bcac on Ocramius:master.

[liuggio]

totally agree with you I see now the future :)

[Ocramius]

@liuggio can you remove the associator stuff then? :)

[liuggio]

Yep!

On Wed, Jun 26, 2013 at 5:48 PM, Marco Pivetta notifications@github.comwrote:

@liuggio https://github.com/liuggio can you remove the associator stuff then? :)

— Reply to this email directly or view it on GitHubhttps://github.com/Ocramius/ChangeSet/pull/5#issuecomment-20057888 .

__liuggio_____

/|_ We reject kings, presidents and voting. /o ) \/ We believe in rough consensus ) v _/\ and running code (I.E.T.F. credo)

---

[liuggio]

Ok think about it as super WIP, I made this in half an hour and I'm in hurry (sorry) don't hate me, I added Pimple, and few containers, I'd like to change the names, and I removed the ObjectLoaderFactory only because I didn't not understand his objective.

[liuggio]

If you think that Pimple is not necessesary I'll remove it leaving only the repo and the 2 interfaces..

[coveralls]

Coverage decreased (-0%) when pulling 1e7040b05d93429a118cf4dc0a17a579d887e77c on liuggio:master into 782620bfbb93b222b92ad2dbb553bcdf7dd5bcac on Ocramius:master.

[coveralls]

Coverage decreased (-0%) when pulling 05da51aa634b7eb25fb88fd76944d5c617aa59a8 on liuggio:master into db04fc86a25fcd4c030e15183c9aa75ee07b9601 on Ocramius:master.

[coveralls]

Coverage decreased (-0%) when pulling 840bf70845d0548e2e0b9a7222f9094d58358f66 on liuggio:master into db04fc86a25fcd4c030e15183c9aa75ee07b9601 on Ocramius:master.

[liuggio]

Little recap:

• The Factory are called factory but they have an interface because they'll be injected and their logic 'd change.
• Object Loader Factory has been eliminated.

[coveralls]

Coverage decreased (-0%) when pulling feb324ae13963c7e38eeddc0da63f643909a8fcc on liuggio:master into db04fc86a25fcd4c030e15183c9aa75ee07b9601 on Ocramius:master.

[Ocramius]

Merging - hopefully going on with this tomorrow (I will try to integrate it with the existing ORM ;) )

[Ocramius]

And obviously: thanks!