[Owncloud] AppFramework app

[Bernhard Posselt]

On 01/14/2013 07:17 AM, Thomas Müller wrote:
> Can you please elaborate why this is necessary and what the benefits are?
>
> I fine it suboptimal to have app dependencies for reusable code.
> And I assume the advanced-app-template will not get easier.  ;-)
>
> I like the idea of having an app framework, but this is something to 
> live in the core?
>
> Thx,
>
> Tom
Thanks for the feedback!

True, the developement wont get easier and there should be an API 
overview (which we should provide anyway, I'll try to do that once ive 
got more time).

The reason for the change was that people found it hard to synchronize 
their app with the changes from the advanced_apptemplate. If you wanted 
to incorporate changes you'd have to do the following:

* Compare all classloader classes for changes and adjust namespace 
changes when copy
* Compare the dependency injection container for changes
* Adjust the routes file for routing logic changes
* Copy all the files in lib/ and change the namespace of all the copied 
files

Other benefits include:
* Classloader is containing only app specific classes
* DI Container contains only app specific classes
* less code in general
* incentive for people to merge back improvements


The disadvantage is of course that people who use the appframework have 
to keep up to date when the framework changes in backwards incompatible 
ways. If people wanted to not depend on that, the code can be copied 
into ones app directory. This requires exactly as much work as keeping 
stuff up to date.

The reason why this is not yet in core is because first of all its a 
very new project and I dont know if people are satisified with all 
aspects meaning: I dont really consider it to have a fully stable API. 
It might also get harder to test if we have dependencies on core, 
especially finding out the requirepath for the parent classes could 
prove to be tricky in the unittests.

mfg

Bernhard


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/owncloud/attachments/20130114/430bd2e9/attachment.html>