Nova у меня v3.6

Я чисто по логам смотрю. Чет не могу найти ее на packagist

ну нова врятли будет на pakagist т.к. она платаная

оу, спасибо

А это точно нормально вот так? Написать-то я написал, но мне самому очень тяжело прочесть написанное.  Меня упорно тянет взять голый SQL.
Можно привыкнуть к этим where(function ($query) ? И ведь это относительно простой SQL. А если бы сложнее был, там же вообще из-за этих where function будет не видно, что написано.
         
// created_by=auth_id OR (searchable AND (share_to IS NULL OR share_to RLIKE '[[:<:]]email[[:>:]]' OR share_ro RLIKE '[[:<:]]@domain[[:>:]]'))
       if($request->user()->id)
        $docs = $docs->where(
           function ($query) use ($request) {
                       $query->where('created_by', $request->user()->id)
                             ->orWhere(
               function ($q) use ($request) {
                          $q->where('searchable', true)
                            ->where(
                   function ($q2) use ($request) {
                          $q2->whereNull('share_to')
                             ->orWhere('share_to', 'RLIKE', '[[:<:]]' . $request->user()->email . '[[:>:]]')
                             ->orWhere('share_to', 'RLIKE', '[[:<:]]' . strrchr($request->user()->email, '@') . '[[:>:]]');
                   });
               });          
           });

вытащить переменную из реквеста.
вынести проверку на существование id выше запроса

всё таки я не обратил внимание, оказывается уже у нас обновили в репо nova до 3,9.1,

Какую переменную?
if($request->user()) — это я уже поправил.

Хотябы пользователя из реквеста достать, читаться лучше будет.
Можно какуюто часть условий вынести в скоуп.
https://laravel.com/docs/7.x/queries#conditional-clauses
Ну и самое главное чтобы это не в контроллере лежало )

Гайз, подскажите, впервые сталкиваюсь с такой хуйней в проекте. Пришел в проект, все довольно интересно, пока не залез в роуты.
Как объяснить упертому человеку, что подход генерации роутов по условиям плох?

Т.е. такой:

if ($user->hasRole(‘admin’)) {
  Route::get(‘/orders’, ‘Admin\OrderController@index’);
} 

If ($user->hasRole(‘courier’)) {
  Route::get(‘/orders’, ‘Courier\OrderController@index’);
}

If ($user->hasRole(‘client’)) {
  Route::get(‘/orders’, ‘Client\OrderController@index’);
}

Я назвал, как минимум, причину того, что нам надо давать роль клиентам и надо будет лезть в БД на каждую регистрацию, но это особо не остановило тимлида. Хочется ваше мнение услышать. Как работает кэширование роутов в ларе при таком подходе? Какие еще очевидные минусы я упускаю?

if ($user->hasRole(‘admin’)) {
  Route::get(‘/orders’, ‘Admin\OrderController@index’);
} 

If ($user->hasRole(‘courier’)) {
  Route::get(‘/orders’, ‘Courier\OrderController@index’);
}

If ($user->hasRole(‘client’)) {
  Route::get(‘/orders’, ‘Client\OrderController@index’);
}

как минимум $user всегда null при регистрации маршрута

В начале api.php идет $user = auth('jwt')->user();

Т.е. вроде юзер там есть

из консоли не будет

route:cache и посмотрите что он кэширует (bootstrap/cache/), там коллекция маршрутов, а не php код

еще как аргумент, это не laravel-way. В роутах никакой логики не должно быть, для этого есть миддлвары, гейты и политики

Парни, а реально ли laravel nova воткнуть в сущесвующий проект? И можно ли там как то приспособить существующие модели и т.д?

https://t.me/LaravelNovaRu
да, можно