shall I do something about this warning?

ash3T

After I run this composer update --prefer-dist --no-dev -a --with-all-dependencies

I encounter the following,

Class Therealsujitk\GIFs\Controllers\ListGifController located in ./vendor/therealsujitk/flarum-ext-gifs/src/Controllers/ListGIFController.php does not comply with psr-4 autoloading standard. Skipping.
Warning: Ambiguous class resolution, "Flarum\Frontend\Frontend" was found 2x: in "/Applications/MAMP/htdocs/flarum/vendor/ianm/no-meta-title/Flarum/Frontend.php" and "/Applications/MAMP/htdocs/flarum/vendor/flarum/core/src/Frontend/Frontend.php", the first will be used.

Just wondering if I shall do something about it. I searched this topic did not find anything related.

luceos

@therealsujitk seems you have an issue with class loading. PSR-4 demands that the class name is identical to the php namespaced name.

So either make the file ListGifController or name the class inside it ListGIFController 😉

ash3T in theory you can ignore them. But sometimes these warning imply an issue that breaks extensions.

therealsujitk

luceos Thanks, it's been fixed in master, I'd like to fix another bug before releasing the next version.

ash3T

Thanks you both! 😁