Laravel页面与类别路由参数

Question

based on the example from here https://scotch.io/tutorials/simple-and-easy-laravel-routing#blog-pages-with-categories-route-parameters

I want to show entries for specific categories. By calling this Route:

Route::get('menues/{city?}', 'PagesController@menue');

I want to show all entries for a specific city.

This is my Controller:

public function menue($city = null) {

  if ($city) {
    $restaurants = User::with(['articles' => function ($q){
        $q->nowpublished();
    }])->where('city', '=', $city)->get();
  } else {
    $restaurants = User::with(['articles' => function ($q){
        $q->nowpublished();
    }])->where('city', '!=', $city)->get();
  }

  return view('pages.menues')
    ->withRestaurants($restaurants) 
    ->withCity($city);
}

The only thing that doesn't work is, by calling a url with a {city} that doesn't exist in the DB I want to display all entries. With the code above this doesn't happen. I get a blank page.

How can I fix this? My guess was that the code inside my else statement displays all entries, but this isn't the case.

Answer 1

Do the following:

public function menue($city = null) {
    $restaurants = User::with(['articles' => function ($q){
        $q->nowpublished();
    }]);

    if(if(!is_null($city) && !is_null(City::where('name', $city)->first())) {
        $restaurants->where('city', '=', $city);
    }

    $restaurants = $restaurants->get();
    return view('pages.menues')
        ->withRestaurants($restaurants) 
        ->withCity($city);
}

->where('city', '!=', $city) is the problem. If you want to get all articles, remove the condition.

Change the condition to:

if(!is_null($city) && !is_null(City::where('name', $city)->first())