-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
heimrichhannot/contao-news_categories as "suggest" #4
Comments
Also wenn dann muss es |
Meiner meinung nach ist es derzeit umgekehrt: // only import categories if news_categories extension is present
if (!in_array('news_categories', array_keys(System::getContainer()->getParameter('kernel.bundles'))))
{
return;
} Ist immer FALSE, da codefog/contao-news_categories den Bundlenamen "CodefogNewsCategoriesBundle" und nicht news_categories hat. heimrichhannot/contao-news_categories hat den Namen "news_categories", weil es eing. kein eichter Symfony-Bundle, sondern ein über Symlink gemappte Contao-Erweiterung ist. |
Die heißen nur zufällig gleich ;). Prinzipiell unterstützt die |
Die Änderung könnte zB so aussehen: $bundles = ['news_categories', \Codefog\NewsCategoriesBundle\CodefogNewsCategoriesBundle::class];
if (array_intersect($bundles, array_keys(System::getContainer()->getParameter('kernel.bundles'))) === 0)
{
return;
} Oder man verzichtet auf den Support älterer $bundle = \Codefog\NewsCategoriesBundle\CodefogNewsCategoriesBundle::class;
if (!in_array($bundle, array_keys(System::getContainer()->getParameter('kernel.bundles'))))
{
return;
} Und in der {
"conflict": {
"codefog/contao-news_categories": "<3.0"
}
} Und natürlich den suggest. |
Die 1. Version gefällt mir besser, weil es eig. mit dem heimrichhannot-Modul auch funktioniert. :) |
|
Oder gibt es einen Grund, warum du unbedingt den |
Nein, es gibt keinen bestimmten Grund, ich habe es nur deswegen installiert, dass es mit deiner Erweiterung zusammenarbeiten kann. :) |
Achso, in dem Fall würde ich eher empfehlen |
Hi,
Wäre es eine gute Idee das "heimrichhannot/contao-news_categories" als "suggest" ins composer.json reinzutun?
Nachdem ich mir den Code angeschaut habe, wollte ich es mit den Kategorien testen und deswegen musste ich die "contao-news_categories" über Composer installieren. Ich habe aber zuerst diese Repo gefunden:
https://github.com/codefog/contao-news_categories
Das Problem mit dieser Repo ist, dass sie einen anderen Bundlenamen bzw. anderen Namespace als die
https://github.com/heimrichhannot/contao-news_categories
repo hat.
The text was updated successfully, but these errors were encountered: