Laravel Eloquent foreach i save Opcje

[markonix]

[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).

[markonix]

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 

Ten post edytował markonix 5.04.2017, 11:05:06