Laravel 8.x and Php 8.0 Support, fixes

[erikn69]

• Laravel 8.x and PHP 8.0 Support
• After #6 files with extension .json are not used in parseBonsai() because stripos('file.bonsai.json', '.js') !== false, so answer first if is a json file
• Added blade directive add , example
  @bonsai('add','files.bonsai.json')
  also removed unnecessary breaks; and " ' " from cases
  Closes #7
  (sorry for my english)

[erikn69]

@kaidesu Hi, maybe you has no time, but its a new bug the last merge

[kaidesu]

What's the use case for the add directive? This is meant to programmatically add assets to your application from the backend. For the frontend, you load your assets natively.

[erikn69]

@kaidesu of course in front end i can load my assets natively, but i'm using json files and need to load dinamically on blade views(i don´t wanna use @php @endphp), i means i need namespaces and depends from the bonsai.json, on updates is more easy update one json file that many blade views with the asset, is useful for me at least with json files(maybe nobody else are using bonsai.json because it's not in the documentation and the new bug went unnoticed), does not affect the above functionality, great addon by the way :)

[kaidesu]

I would argue that belongs in a view composer 👍 then you can natively use Bonsai as intended without muddying up your view files.

I'll accept this PR if you remove the add directive change - can open that as its own PR if you'd like to continue the discussion, but I do recommend using a view composer for this particular use case.

[erikn69]

They are partial views added with @include and components for example, i'm using view composer with the main views, but for that little piece of code I find it tedious

[PaolaRuby]

@kaidesu ping :)

[erikn69]

@kaidesu ready for merge, tested in Laravel 8.32 and PHP 8.0.2