laravel-self-diagnosis icon indicating copy to clipboard operation
laravel-self-diagnosis copied to clipboard

Check all translation messages present or not - New feature

Open knvpk opened this issue 6 years ago • 4 comments

A new feature that will check all the response messages, mail subjects which have used the trans function and check whether a message exists in resources\lang folder if not need to show error messages with the list of translations that are not available.

knvpk avatar Jul 19 '18 06:07 knvpk

I think the challenge is in finding all usage of trans function in the code

knvpk avatar Jul 19 '18 06:07 knvpk

In my opinion, this is not a task that should be solved by this package. It seems more like a static code analysis job to me and I doubt that static code analysis is one of the goals of this package (because there are already better tools out there). Personally, I think laravel-self-diagnosis should focus on making sure that the runtime environment (may it be local/dev, staging or production) is properly set up.

Namoshek avatar Jul 20 '18 14:07 Namoshek

I share the opinion of @Namoshek on this point. You have other packages like https://github.com/aXent/laravel-analyze-locale that will do this. Checking the translation string is an intensive job for bigger projects that can really slow down the deployment. You need not only check the values from the trans function but also from __() and @lang

tvbeek avatar Jul 26 '18 20:07 tvbeek

Thanks for package sharing @tvbeek

knvpk avatar Aug 03 '18 06:08 knvpk