Arguments of the constructor were simplified taking advantage of some Laravel's helpers.
Now, an instance of the menu builder is part of the class, instead of holding the filters and the built menu that were already part of the menu builder.
Now, the logic to build the menu is easier to understand. Dispatching the building menu event and listening for it on the service provider to compile the static menu configuration was weird.
Better code documentation.
Refactor the AdminLteServiceProvider class.
Remove listener for the building menu event.
Simplifies logic to compose views with the AdminLte class instance.
Change and improve tests for the previous changes.
What's in this PR?
AdminLteclass.filtersand the builtmenuthat were already part of the menu builder.building menuevent and listening for it on the service provider to compile the static menu configuration was weird.AdminLteServiceProviderclass.building menuevent.AdminLteclass instance.Checklist