FriendsOfREDAXO / rexstan

redaxo phpstan addon
https://staabm.github.io/archive.html#rexstan
MIT License
35 stars 3 forks source link

Remove SF Command stub in simplify/phpstan-rules #721

Closed gharlan closed 2 months ago

gharlan commented 2 months ago

Die Stub-Datei kam in einem der letzten Updates des Vendors mit rein. Problem ist, dass der Rex-Autoloader aktuell alle Addon-Vendoren per Classmap lädt, somit auch die Stubs dort.

Es kommt in der Console dann zu diesem Fehler (z.b. redaxo/bin/console cache:clear):

Fatal error: Cannot declare class Symfony\Component\Console\Command\Command, because the name is already in use in /Users/gharlan/workspace/redaxo/redaxo/src/core/vendor/symfony/console/Command/Command.php on line 35 ErrorException: Cannot declare class Symfony\Component\Console\Command\Command, because the name is already in use in /Users/gharlan/workspace/redaxo/redaxo/src/core/vendor/symfony/console/Command/Command.php:35 Stack trace:

0 [internal function]: rex_error_handler::shutdown()

1 {main}

staabm commented 2 months ago

hmm, d.h. der redaxo loader lädt alles was er finden kann?

ggf. ist das was, was man mit redaxo6 besser machen kann (z.b. per convention nur lib, src ordner o.ä.)

tyrant88 commented 2 months ago

thx

gharlan commented 2 months ago

hmm, d.h. der redaxo loader lädt alles was er finden kann?

Beim Core-Vendor-Order wird der echte Composer-Autoloader verwendet, aber in den Addons werden die Vendor-Ordner per Classmap-Generator pauschal durchsucht.

ggf. ist das was, was man mit redaxo6 besser machen kann (z.b. per convention nur lib, src ordner o.ä.)

Ja, das soll da mit gelöst werden.