Move all preview providers which require Imagemagick to an app

[oparoz]

All the preview providers which require Imagemagick are disabled by default. I propose we move them to an app.

Phase 1

• Keep config in config/config.php
• Create app with all the current providers

Phase 2

• Move config to a config file in the app folder in order to decouple it from the server?
• Introduce some extra providers

Phase 3

• Enable/disable providers from the admin section?

---

@LukasReschke @nickvergessen @rullzer @icewind1991 @karlitschek

[nickvergessen]

Phase 1: yes
Phase 2: oc_appconfig, no config files for apps, those are deleted on update :-O

[oparoz]

those are deleted on update

Not if we use the fancy new filesystem in development which can store app related files, no?

[nickvergessen]

App configuration belongs in the named DB table, not in a file. config.php should also be migrated to the database. Otherwise you run into issues with load balancing if both servers have a different config file (happened in the past).
So please just use the DB and done.

[rullzer]

YES!!! 👍

And indeed keep it in the DB please :)

[karlitschek]

O.K. We can do this but the app should still stay in the core repo and should be shipped by default. Feature regressions are bad.

[oparoz]

O.K. We can do this but the app should still stay in the core repo and should be shipped by default. Feature regressions are bad.

Agreed

[rullzer]

Fine by me 😄

[szaimen]

I suppose this issue is still valid? If not, please close this issue!