Как в Laravel 5.5 правильно получить фиксированное количество hasMany связей?

Question

[tester_toster]

Приветствую. Реализую мультиязычный функционал новостей на сайте. Есть таблицы:

languages
--id
--iso_code
--name

news
--id
--alias
--published

news_languages
--id
--news_id
--language_id
....

В модели news добавил:

public function newsLanguages()
{
    return $this->hasMany('App\NewsLanguage');
}

public function getLanguage($id)
{
   if(!$this->languageBug){
      $this->languageBug = $this->newsLanguages->keyBy('language_id');
   }
   return $this->languageBug->get($id) ? $this->languageBug->get($id):  new NewsLanguage;
}

public function attachRequest($request)
{
    $lng = [];
    foreach(\App::make('Language')->getAll() as $language)
    {
        $l = $this->getLanguage($language->id);
        $l->language_id = $language->id;
        $l->title = $request->has('title.'.$language->iso_code) ? $request->title[$language->iso_code] : '';
        $l->meta_title = $request->has('meta_title.'.$language->iso_code) ? $request->meta_title[$language->iso_code] : '';
        $l->meta_description = $request->has('meta_description.'.$language->iso_code) ? $request->meta_description[$language->iso_code] : '';
        $l->content = $request->has('content.'.$language->iso_code) ? $request->content[$language->iso_code] : '';
        $lng[] = $l;
    }

    $this->alias = $request->alias;
    $this->published = $request->published;
    $this->save();
    $this->newsLanguages()->saveMany($lng);

}

Работа с моделью происходит через метод Route::resourse(...) На обновление и создание новости - один шаблон, один валидатор:

public function create(News $news, NewsRequest $request)
{
   $news->attachRequest($request);
}

public function update(News $news, NewsRequest $request)
{
   $news->attachRequest($request);
}

Форма отображается через laravel form collective:

{!! Form::model($news, ['route' => [ $news->id ? 'admin.news.update' : 'admin.news.store', $news], 'method' => $news->id ? 'PUT' : 'POST',  'files' => true]) !!}

...
{!!Form::label('alias', 'Алиас')!!}
{!!Form::text('alias', $news->alias, ['class' => 'form-control'])!!}
@if ($errors->has('alias'))
<span class="help-block">{{ $errors->first('alias') }}</span>
@endif
...
@foreach ($languages as $language)
...
<?php $lang = $news->getLanguage($language->id);?>
{!!Form::label('title'.$language->iso_code.'', 'Заголовок')!!}
{!!Form::text('title['.$language->iso_code.']', $lang->title, ['class' => 'form-control'])!!}
...
@endforeach
{!!Form::submit( $news->id ? 'Обновить' :'Создать',['class'  => 'btn btn-primary'])!!}
...

Все работает. Но решение как по мне - костыльное. Возможно ли как-нибудь правильнее получать пустые модели newsLanguages добавив в связи таблицу languages? Ведь при создании новости или если в будущем добавится язык - связи будут пусты. За любые комментарии, по теме или исправлению качества кода - буду благодарен.

Answer 1

[Александр Талалаев]

Попробуй перейти на spatie/laravel-translatable, у них решены многие проблемы, гораздо чище и прозрачнее код получается при создании мультиязычных штук.

Comments

[tester_toster]

Интересная реализация, как я понял - через json, без связей, что накладывает некоторые ограничения, например по поиску, или при большом размере полей, так же из-за постоянной сериализации строка может перейти в число и наоборот, например float - после обновления станет строкой, и если обновить число и не сохраняя модель - взять его, может прийти строка, если я не ошибаюсь, хотя проще в обращении. Посмотрел на github похожий проект, только с другой реализацией, почти как у меня - используется firstOrCreate. Но нет желания ставить для локализации отдельный пакет, так как локализация моделей потребуется лишь для двух/трех сущностей, поэтому пока останусь как есть, в крайнем случае - не трудно будет поправить на пакет второго типа)

[Александр Талалаев]

tester_toster, Ну прям сверх проблем не будет, все же в основном у нас у каждой отдельной сущности немного полей для перевода, у той же новости это (грубо говоря) название, содержание. Те же теги это уже другая сущность и тд.
Поиск тоже работает, ну и в целом, если нет какой-то специфичной области, в данном случае удобство и красота кода больше чем другие штуки. пока проблем не было.
PS так же советую на их другие пакеты глянуть, это клевая ребята из Бельгии, много клевых и простых штуки делают.