Level 102
You are meant to pass data down. Not call controllers from a blade component. So send that data inside the component or pass it in as data
Feb 18, 2023
19
Level 1
Calling Controller functions in blade tempalte
Hello guys, I'm new to Laravel and I'm trying to call a Controller function from a blade template. I'm doing something definitely wrong.
Controller
class MenuController extends Controller
{
public function getMenu(){
dd(Prompt::all('service'));
}
}
Blade template
{{ action([\App\Http\Controllers\MenuController::class, 'getMenu']) }}
Level 1
@Sinnbeck thanks for the asnwer. can you explain a bit better? it's not clear to me what you're saying.
My idea is to dynamically create the menu with items from the database.
Level 102
@danilotrix that is fine. But you are meant to pass it down
For instance https://laravel.com/docs/9.x/views#passing-data-to-views
Level 80
@danilotrix You don’t call controllers from views. You pass data from controllers, to views.
You’d probably be better off actually watching some lessons on this site to get a handle on fundamentals like this.
Level 1
ok guys I understood how to do it thanks. However, I still have some issue
in my Route I use this
Route::get('/prompts/{cat}/{ser}', [DashboardController::class, 'index'])
->middleware(['auth', 'verified']);
in DashboardController
public function index()
{
$menu_from_service = new MenuController;
$menu = $menu_from_service->getMenu();
return view('pages.dashboards.index', $menu);
}
the problem is that, this method returns a blade template which is included in the master template. and $menu does not have scope in the master, where the menu should be.
hope it;s clear
Level 102
@danilotrix ok maybe I wasn't quite clear. You don't call a controller from another controller. You can move that logic to a model, a service or even use an action. Hard to give a best solution as I don't know the code. But a controller is meant to be called from a http request and return a response. Simple as that
Level 1
Level 102
@danilotrix so is that menu in your layout? If so, you might want to use a view composer to pass that data https://laravel.com/docs/9.x/views#view-composers
Level 80
@danilotrix Can you please watch some lessons on this site. Then you won’t keep making fundamental mistakes as above.
Level 1
@martinbean yes I did and fix this issue. thank you guys for your help
Level 80
@danilotrix You didn’t though, because your “lessons watched” count is still at 0.
Level 1
@martinbean I watched lessons on Youtube. They should be the same right?
Level 80
@danilotrix No, because Laracasts is more authoritative for learning Laravel versus random videos on YouTube that may be out of date or just plain incorrect.
Level 1
@martinbean thank you very much for your advice. actually didn't know laracast had lessons. I watched videos from developers that work at Laravel so I guess they were very accurate too.
Level 1
Not everything should be made from controller, cuz it's not effective if you want to show that data for every page. The solution is, you can make your own helpers function like this
laravel-news . com/creating-helpers
You can use it on Controllers, Models even Blades
Level 122
@hendro-dev There is already a mechanism for dealing with data you need on multiple pages. Its called View Composer
1 like
Level 1
@Snapey Thankyou
Level 1
@Snapey I figure out using view composer is a bad practice, it will make laravel calling multiple query for every included blade. I think a custom helpers is still the best option.
Level 122
@hendro-dev not if you target it correctly
What are you trying to do?
Please or to participate in this conversation.