Symfony bekannter "src/" Ordner fehlt im Autoloader

[contao]

Symfony bekannter "src/" Ordner fehlt im Autoloader

[leofeyer]

Wir liefern kein AppBundle mit, daher auch kein Composer-Eintrag für src/. Sofern der Composer-Eintrag nicht zu Problemen führt, wenn es keinen Ordner src/ gibt, wäre ich allerdings dafür, ihn mit auszuliefern. @contao/developers Was denkt ihr?

[aschempp]

Ein Bundle muss im Kernel registriert werden, genauso wie in der composer.json. Ich finde nicht dass wir das standardmässig eintragen sollten. Ich trage jeweils auch nicht "psr-4": { "": "src/" } sondern "psr-4": { "AppBundle\\": "src/AppBundle" } ein, sonst sucht Composer alle Klassen in /src/.

[Toflar]

Seh ich auch so, jeder Entwickler ist selber für das Autoloading verantwortlich. Wenn man ein Bundle als Composer-Library zur Verfügung steht ist das ja auch so, wieso sollte es bei einem lokalen AppBundle anders sein?

Allerdings könnten wir die manuellen Includes für die AppKernel und AppCache weglassen. Die SF Standard Edition macht das auch so: symfony/symfony-standard@2c45777

[leofeyer]

Das ist neu in Symfony 2.8. Beim Upgrade übernehme ich das natürlich.

[Toflar]

Wir sollten sowieso auf 2.8 upgraden, aber das wäre ein generelles up for discussion issue. :)

[leofeyer]

Wir sollten eigentlich auf die 3 upgraden und uns eine Lösung für das Scopes-Problem einfallen lassen.

[aschempp]

Können wir nicht, aus BC-Gründen. Die 3 hat (noch) keine Vorteile gegenüber der 2.8er.

[leofeyer]

Genau, noch nicht. Aber das bleibt nicht so und deswegen sollten wir uns eine Lösung einfallen lassen :)

[aschempp]

Lösung: Contao 5. Gehört aber nicht in dieses Ticket.

[leofeyer]

Lösung: Contao 5.

Nein.

Gehört aber nicht in dieses Ticket.

Ja.

[discordier]

Ich bin dagegen einen autoloader key zu liefern wenn das zugehoerige directory nicht existiert.

AFAIR hatten wir die Diskussion ob wir ein app Bundle vorkonfigurieren sollen oder nicht doch bereits vor Monaten und haben uns dagegen entschieden. die Autoloader config eines selbigen zaehlt genauso als Konfiguration.