🚀 go-pugleaf

Piotr Czeszkiewicz wrote:

> Witam
> 
> Mógłby ktoś mi wytłumaczyć jak przy użyciu funkcji zwrotnych
> after_update lub podobnych wykonać poniższe zadanie.
> 
> 1. Jest sobie zgłoszenie,
> 2. Zmianiam stan zgłoszenia na "Zakończone"
> 3. Chciałbym, aby w tym momencie została wypełniona data zakończenia
> zgłoszenia.
> 
> Robię to tak, jak poniżej (czemu nie działa)?
> 
> class ZgloszeniaController < ApplicationController
> 
> def edit
>    @zgloszenie = Zgloszenie.find(params[:id])
>    if request.post?
>      if @zgloszenie.save
>        @zgloszenie = Zgloszenie.find(params[:id])
>        flash[:info] = 'Dane zostały zapisane.
>      end
>    end
>    render :action => 'edit', :id => @zgloszenie
> end

Po co dwa razy Zgloszenie.find(params[:id])?

> 
> 
> class Zgloszenie < ActiveRecord::Base
> 
> after_update :data_zamkniecia
> 
> def data_zamkniecia
>    if self.stan == 'Zakończone'
>      self.closed_at = Time.now
>    end
> end


Nie działa, bo after_update się co prawda odpali, ale nie zadziała już 
ponowne save na bieżącym obiekcie, a ty wyraźnie chcesz zmieniać wartość
zmiennej instancji _tego_ obiektu. Zrób to before_ - sprawdź w nim czy stan
== 'Zakończone' i ustaw datę.

A przy okazji, nie możesz mieć pola o nazwie updated_at? Rails by ci
automatycznie ustawiał datę, i skoro stan będzie miał wartość 'Zakończone'
to i tak wiadomo będzie, że data która aktualnie znajdować się będzie w
polu updated_at będzie dotyczyła Zakończenia, czyż nie?


-- 
.:: Piter // phlawski$gmail,com // gg: 4534287 ::.
Kiedy Bóg powiedział "Niech stanie się światłość", Chuck Norris
dodał "Powiedz, 'proszę'."

Piotr 'piter' Hlawski napisał(a):
> 
> Po co dwa razy Zgloszenie.find(params[:id])?

Błąd, w rzeczywistości jest tylko raz.

> 
> A przy okazji, nie możesz mieć pola o nazwie updated_at? Rails by ci
> automatycznie ustawiał datę, i skoro stan będzie miał wartość 'Zakończone'
> to i tak wiadomo będzie, że data która aktualnie znajdować się będzie w
> polu updated_at będzie dotyczyła Zakończenia, czyż nie?
> 

Faktycznie to najlepsze rozwiązanie. Dzięki.

-- 
Piotr Czeszkiewicz

Piotr Czeszkiewicz wrote:

>> 
>> A przy okazji, nie możesz mieć pola o nazwie updated_at? Rails by ci
>> automatycznie ustawiał datę, i skoro stan będzie miał wartość
>> 'Zakończone' to i tak wiadomo będzie, że data która aktualnie znajdować
>> się będzie w polu updated_at będzie dotyczyła Zakończenia, czyż nie?
>> 
> 
> Faktycznie to najlepsze rozwiązanie. Dzięki.

Dorób jeszcze pole created_at i będziesz miał komplet informacji :)

-- 
.:: Piter // phlawski$gmail,com // gg: 4534287 ::.
Nie próbujcie tłumaczyć Chuckowi Norrisowi słowa "aerodynamika". Nie
zrozumie. Nie żeby był tępy, po prostu powietrze jak dotąd nie odważyło się
stawiać mu oporu.

Piotr 'piter' Hlawski napisał(a):
> Piotr Czeszkiewicz wrote:
> 
> Dorób jeszcze pole created_at i będziesz miał komplet informacji :)
> 

To już miałem wcześniej

-- 
Piotrek