lucidarch / laravel

[DEPRECATED] See https://github.com/lucidarch/lucid

Home Page:https://lucidarch.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

PHP Notice: Constant DS already defined

rfpdl opened this issue · comments

commented

I have a project that has 2 packages installed that are using this constant:

  • Lucid
  • Arcanedev log viewer

Even though it is just a notice, what should I do to prevent this?

PHP Notice:  Constant DS already defined in /var/www/html/vendor/lucid-arch/laravel-console/src/Finder.php on line 23
PHP Stack trace:
PHP   1. {main}() /var/www/html/vendor/lucid-arch/laravel-console/lucid:0
PHP   2. spl_autoload_call() /var/www/html/vendor/lucid-arch/laravel-console/lucid:19
PHP   3. Composer\Autoload\ClassLoader->loadClass() /var/www/html/vendor/lucid-arch/laravel-console/lucid:19
PHP   4. Composer\Autoload\includeFile() /var/www/html/vendor/composer/ClassLoader.php:322
PHP   5. include() /var/www/html/vendor/composer/ClassLoader.php:444
PHP   6. spl_autoload_call() /var/www/html/vendor/lucid-arch/laravel-console/src/Commands/ChangeSourceNamespaceCommand.php:27
PHP   7. Composer\Autoload\ClassLoader->loadClass() /var/www/html/vendor/lucid-arch/laravel-console/src/Commands/ChangeSourceNamespaceCommand.php:27
PHP   8. Composer\Autoload\includeFile() /var/www/html/vendor/composer/ClassLoader.php:322
PHP   9. include() /var/www/html/vendor/composer/ClassLoader.php:444
PHP  10. define() /var/www/html/vendor/lucid-arch/laravel-console/src/Finder.php:23

Note: Prior to updating Lucid to Laravel 6, this will cause PHP Error instead of just a notice. What I did before is I commented the constant value to make it work (inside the vendor. Yes I know it is not good, but I have to make it work)

Thanks for reporting, it's been covered in the new repo: https://github.com/lucidarch/lucid/blob/main/src/Finder.php#L14