[PHP] pobierz, plaintext 
        $i = 0;
        $trainings->each(function($training) use ($i) {
            dump($training->id);
            $training->topic = ++$i;
            $training->save();
        });
[PHP] pobierz, plaintext 

Czy użyje foreach czy each to nie zadziała. W $trainings mam ładną kolekcję obiektów Training zwróconą za pomocą ->get().
Samo $training->topic działa, nadpisze atrybuty ale gdy użyje to wraz save() to wszystko się psuje - dump zwraca za każdym razem id pierwszego obiektu.

Nie umiem zrozumieć dlaczego to nie działa? Jakieś rozwiązania na szybko bym znalazł (skorzystać z QB czy tworzyć w każdym obrocie pętli $obj = new Training::find($training->id) ale to by było mega słabe (na każdy obrót pętli 2 zapytania).

Generalnie najbardziej optymalnie by było coś w stylu:

[PHP] pobierz, plaintext 
        $i = 0;
        $trainings->each(function($training) use ($i) {
            dump($training->id);
            $training->topic = ++$i;
        });
 
            $trainings->save();
[PHP] pobierz, plaintext 

Ale to już bardziej rozumiem, że nie przechodzi (ale nie ukrywam, było by miło jakby był też sposób aby w ten sposób masowo updatejtnąć wszystkie obiekty).

A:

[PHP] pobierz, plaintext 
$i = 0;
$trainings->each(function($training) use ($i) {
    $training->update([
        'topic' => ++$i
    ]);
});
[PHP] pobierz, plaintext 

Gdy się położyłem to mi się dopiero żarówka zaświeciła.
Generalnie ta cała akcja służy reindeksacji pewnej kolumny przy delete/update/insert. Dla delete i insert nie ma problemu jednak problem przychodzi przy update.

[PHP] pobierz, plaintext 
        static::updated(function($training) {
            static::reindex($training->date);
        });
[PHP] pobierz, plaintext 

Używając eloquent spowoduje zapętlenie - update użytkownika odpala metodę reindex(), która odpala event. Podpowiedzią był error 500 zwracany przez ajax.
Pytanie jak do tego teraz fajnie podejść. Użycie Query buildera generalnie rozwiązuje problem, jako, że on nie odpala eventów.

Zacznij prosze zakladac tematy we wlasciwym dziale. Przenosze

Problem i tak się okazał "ponad frameworkowy". Zapętlenie się eventów na update to raczej problem logiczny

Nie mniej jednak mocno dotyczylo to jak dziala Laravel i eloquent

Możesz wywalić eventy dając `Training::flushEventListeners();`

Ostatecznie podszedłem do tego bardziej skomplikowanie.
Wyłuskałem z dokumentacji ciekawe metody isDirty() i getOriginal() i wykonuje tylko reIndex w momencie gdy zmieniła się godzina.
Przy okazji jest to też rozwiązanie na problem na który bym wcześniej nie zwrócił uwagi - reindeks działał każdorazowo tylko na NOWĄ datę, a przy zmianie daty jest on potrzebny także dla oryginalnego dnia.
Tak więc ostateczny kod:

[PHP] pobierz, plaintext 
        static::updated(function($training) {
            if ($training->isDirty('hour')) {
                static::reindexTrainings($training->date);
            }
            if ($training->isDirty('date')) {
                static::reindexTrainings($training->getOriginal('date'));
            }
        });
[PHP] pobierz, plaintext